Skip to content

API, Core: Support column files through TrackedFileAdapter for data files#17127

Open
gaborkaszab wants to merge 2 commits into
apache:mainfrom
gaborkaszab:main_column_file_in_trackedfile_adapter
Open

API, Core: Support column files through TrackedFileAdapter for data files#17127
gaborkaszab wants to merge 2 commits into
apache:mainfrom
gaborkaszab:main_column_file_in_trackedfile_adapter

Conversation

@gaborkaszab

Copy link
Copy Markdown
Contributor

No description provided.

This change introduces the basic structs for column files and also
integrates them to the schema for TrackedFile and Tracking.
return null;
}

default List<ColumnFile> columnFiles() {

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

equalityFieldIds override the function from ContentFile. Following the same pattern, should columnFiles be also defined there?


/** Information about a column file. */
interface ColumnFile {
public interface ColumnFile {

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To return List from DataFile, we have to move this class to API and make it public.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

2 participants