MonkeyCanCode commented on PR #426:
URL: https://github.com/apache/polaris/pull/426#issuecomment-2458590487

   Hello @fivetran-arunsuri ,
   
   That is expected as Polaris is not enable eclipse-link by default. It is 
possible to enable it as part of the build. See following for detail (same can 
be found in https://polaris.apache.org/in-dev/unreleased/metastores/):
   ```
   Apache Polaris supports the following optional build options:
   
   -PeclipseLink=true – Enables the EclipseLink extension.
   -PeclipseLinkDeps=[groupId]:[artifactId]:[version],... – Specifies one or 
more additional dependencies for EclipseLink (e.g., JDBC drivers) separated by 
commas.
   ```
   
   This is archived by the same file you put the PR with following snippet:
   ```
   if (project.properties.get("eclipseLink") == "true") {
     dependencies { implementation(project(":polaris-eclipselink")) }
   }
   ```
   
   Reasoning for this is by default eclipselink is configured to use h2 
database which is not very useful for prod graded deployment. Due to licensing 
concern as well as various diff backends end-users may preferred, none of the 
other database drivers are included as of now. There is a purpose to use 
quarkus to close this gap (https://github.com/apache/polaris/issues/392), 
however, that is not yet completed as of today. 
   
   Thanks,
   Yong Zheng


-- 
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]

Reply via email to