dimas-b commented on code in PR #1287: URL: https://github.com/apache/polaris/pull/1287#discussion_r2047504459
########## README.md: ########## @@ -55,6 +55,7 @@ Apache Polaris is organized into the following modules: - Persistence modules: - `polaris-jpa-model` - The JPA entity definitions - `polaris-eclipselink` - The Eclipselink implementation of the MetaStoreManager interface + - `polaris-relational-jdbc` - The JDBC implementation of BasePersistence to be used via AtomicMetaStoreManager Review Comment: IMHO, `NoSQL JDBC` is a misnomer :sweat_smile: We can talk about `Multi-row Transaction JDBC` (this impl.) vs. `CAS-based JDBC` (a possible extension of the NoSQL Persistence code). "Relational" here means we actually have entity relations expressed as foreign keys, which is not the case in this PR, as far as I can tell. How about `polaris-transactional-jdbc`? -- 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]
