-
Notifications
You must be signed in to change notification settings - Fork 199
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add JDBC persistence backend #649
Comments
Based on #766 , the JDBC connection needs to support SERIALIZABLE isolation. This would allow a few scalable distributed backend options like CockroachDB and Google Spanner beyond single node SQL engines that could become a bottleneck. Unfortunately the equivalent product in AWS - Amazon Aurora DSQL, only supports REPEATABLE READ as of today... 😢 |
That's a good point. If it should work fine for JDBC, I don't think we should limit/focus only on JDBC in Polaris. We have to support also NoSQL. |
Can you explain more about the background on this? If we want to support both SQL and noSQL, we probably need to create a broad interface that can be implemented by both SQL and noSQL in drastically different ways. I was a bit out of date with where the community is right now, are we acceptable to such change? |
@jackye1995 I mean that the Polaris data model should allow us to support both SQL and NoSQL backend. The purpose of this issue, the #650 and #844 is to identify the changes needed on the current data model.
The reason why I think it's important to discuss this today is because it's central to Polaris and any change later will mean to "refactore" the backend plugins. |
Related to #985 |
cc @singhpk234 |
@singhpk234 will take this per discussion. I couldn't assign this issue to him somehow. |
I added @singhpk234 as collaborator on Polaris. He can now assign issue to himself. |
As discussed during the Polaris Community Meeting, I will add a new plugin implementing JDBC storage backend, without Eclipselink and focusing on PostgreSQL.
It will use "purely" JDBC DataSources with DBCP2 connection pooling.
The text was updated successfully, but these errors were encountered: