Build #5,612

License check with RAT and Checkstyle

Build result summary


5 minutes
667ff642b75f1f210d7ea73f058ce20d5d88e81c 667ff642b75f1f210d7ea73f058ce20d5d88e81c
Successful since
#5446 ()

Code commits

Author Commit Message Commit date
Terence Yim <> Terence Yim <> 667ff642b75f1f210d7ea73f058ce20d5d88e81c 667ff642b75f1f210d7ea73f058ce20d5d88e81c Merge pull request #13715 from cdapio/feature/CDAP-18573-storage-spi-ext
Revamp the storage SPI to support new storage engine as an extension
Terence Yim Terence Yim 3ab151146657e795fad43a0d4f846c66025a8eea 3ab151146657e795fad43a0d4f846c66025a8eea Revamp the storage SPI to support new storage engine as an extension
- Added StorageProvider to storage SPI and implementations to support extension based storage engine.
- Removed StructuredTableRegistry from SPI as it is the implementation details of the current Table and PostgreSQL implementations.
- Changed the StructureTableAdmin API to replace the getSpecification method with exists and getSchema methods.
- Added method to check for compatibility between StructuredTableSchema and StructuredTableSpecification.
  - The schema of the actual table might be different than the specification in terms of data type. However, as long as the
    table can store data correctly without losing precision, it is considered as compatible.
- Refactored and code cleanup around StoreDefinition class and its usages.
- Unify StructuredTable related unit tests to use always Guice injector for creating StructuredTableAdmin and TransactionRunner.