You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@polaris.apache.org by "snazy (via GitHub)" <gi...@apache.org> on 2025/01/15 08:49:14 UTC
[I] Eclipselink module not prepared for multiple realms [polaris]
snazy opened a new issue, #760:
URL: https://github.com/apache/polaris/issues/760
### Describe the bug
The current way of how Polaris uses databases for persistence is to encode the realm-ID in the JDBC connection URL - aka: one isolated database per realm.
On the other side, `org.apache.polaris.extension.persistence.impl.eclipselink.PolarisSequenceUtil` is global and static and it's `initialize()` function does not account the fact that there can be multiple realms.
### To Reproduce
_No response_
### Actual Behavior
_No response_
### Expected Behavior
_No response_
### Additional context
_No response_
### System information
_No response_
--
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: issues-unsubscribe@polaris.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Eclipselink module not prepared for multiple realms [polaris]
Posted by "flyrain (via GitHub)" <gi...@apache.org>.
flyrain commented on issue #760:
URL: https://github.com/apache/polaris/issues/760#issuecomment-2722706332
Each method in `PolarisSequenceUtil` gets a session object which is passed from an object of `PolarisEclipseLinkMetaStoreSessionImpl`. And the object is initialized with `realmContext`. With that, I don't think it's an issue.
```
public static void initialize(EntityManager session)
public static Long getNewId(EntityManager session)
```
cc @eric-maynard
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Eclipselink module not prepared for multiple realms [polaris]
Posted by "flyrain (via GitHub)" <gi...@apache.org>.
flyrain commented on issue #760:
URL: https://github.com/apache/polaris/issues/760#issuecomment-2840578421
I don't think it is a 1.0 blocker, as we are going to deprecate EclipseLink.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Eclipselink module not prepared for multiple realms [polaris]
Posted by "snazy (via GitHub)" <gi...@apache.org>.
snazy commented on issue #760:
URL: https://github.com/apache/polaris/issues/760#issuecomment-5021668911
Eclipselink is no longer used, closing
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Eclipselink module not prepared for multiple realms [polaris]
Posted by "snazy (via GitHub)" <gi...@apache.org>.
snazy commented on issue #760:
URL: https://github.com/apache/polaris/issues/760#issuecomment-2729995343
@flyrain maybe you haven't seen the `private static final AtomicBoolean initialized`? I think there is still an issue.
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [I] Eclipselink module not prepared for multiple realms [polaris]
Posted by "snazy (via GitHub)" <gi...@apache.org>.
snazy closed issue #760: Eclipselink module not prepared for multiple realms
URL: https://github.com/apache/polaris/issues/760
--
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: issues-unsubscribe@polaris.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org