cc5566 commented on PR #9779: URL: https://github.com/apache/gravitino/pull/9779#issuecomment-3829306891
@jerqi Thanks for the feedback! I've addressed all the points in the latest commit: 1. Created issue #9838 and updated the PR title accordingly 2. Added the partition_statistic_meta table to the upgrade scripts (upgrade-1.1.0-to-1.2.0-mysql.sql, postgresql, and h2) 3. Changed the default storage to JDBC. It now reuses the entity store's JDBC config by default (jdbcUrl, jdbcDriver, jdbcUser, jdbcPassword, pool settings), but users can still override with partition-specific configs if needed 4. Fixed the code style issue - using ifPresent() now instead of isPresent()/get() Tests are all passing. Let me know if anything else needs adjustment! -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
