ashvina opened a new pull request, #653: URL: https://github.com/apache/incubator-xtable/pull/653
Fixes #652 The main goal of this pull request is to establish a common structure for storage files and introduces an abstraction for all types of storage files associated to a table. This should facilitate future extensions and ensure stability of APIs. The abstraction will serve as the base for various file types such as data files, delete files, etc. Different table formats support various storage files, and as these evolve, more file types are expected to be added. The abstraction provided by the base class is helpful for maintaining the stability of core XTable APIs while supporting extensibility. For instance if a new storage file for statistics is introduced, XTable code for detecting file changes between commits need not change. While the APIs need not change, handling of new types will be needed in the source and target implementations. For e.g., a target may not support deletion type files. By default the target format should throw an error if a deletion file is detected and not handled. This may not be true for in all cases. Additionally, minor refactoring has been performed to rename certain elements for better clarity and consistency. This PR does not add any new functionality of change in behavior. Tests have been updated. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@xtable.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
