Remove legacy columns

[rlamy]

This removes the following attributes or columns from various internal classes and tables: vtype, dir_type, owner_name, owner_id. Note that Node.dir_type is still used though.

The publicly visible, backwards-incompatible changes are:

• File.vtype is removed
• Existing listings table become invalid.

Part of #36

[cloudflare-workers-and-pages]

Deploying datachain-documentation with    Cloudflare Pages

Latest commit:	fafe4b5
Status:	✅  Deploy successful!
Preview URL:	https://2b2468dd.datachain-documentation.pages.dev
Branch Preview URL:	https://rm-legacy-columns.datachain-documentation.pages.dev

View logs

[codecov]

Codecov Report

Attention: Patch coverage is 66.66667% with 6 lines in your changes missing coverage. Please review.

Project coverage is 87.09%. Comparing base (48a0de5) to head (fafe4b5).
Report is 4 commits behind head on main.

Files with missing lines	Patch %	Lines
src/datachain/cli.py	25.00%	3 Missing ⚠️
src/datachain/listing.py	0.00%	0 Missing and 2 partials ⚠️
src/datachain/node.py	75.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #263      +/-   ##
==========================================
+ Coverage   87.07%   87.09%   +0.01%     
==========================================
  Files          92       92              
  Lines        9952     9925      -27     
  Branches     2037     2031       -6     
==========================================
- Hits         8666     8644      -22     
+ Misses        931      927       -4     
+ Partials      355      354       -1     

Flag	Coverage Δ
datachain	87.04% <66.66%> (+0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[shcheklein]

Looks good to me.

Do we need to invalidate listing on the Studio side or remove them? Does it also invalidate datasets (do they have these file fields inherited from listings)?

[rlamy]

@shcheklein This makes existing listing tables invalid, but I think that's irrelevant after #294. For datasets, the only change is that File.vtype is removed, but it wasn't used before so I don't think it'll cause any issue. Old dataset tables will just have an extra "file__vtype" column that should be ignored everywhere (UI, queries, ...).

[shcheklein]

@rlamy are we good to go here? :)

[rlamy]

@shcheklein The datachain part works, but we need the Studio tests to pass...