adutra commented on PR #1758:
URL: https://github.com/apache/polaris/pull/1758#issuecomment-2931860502
> We actually only need the realm, not even the whole call context. This PR
currently introduces the new method, i will have follow up PRs to remove the
other usages step by step, the whole refactor is pretty big and touches
persistent, i need to move step by step to make thing simplier
OK, in this case for consistency with other components, maybe we could pass
`RealmContext`? Something like:
```java
public <T> @Nullable T getConfiguration(@Nonnull RealmContext realmCtx,
String configName) {
```
This could also answer @dimas-b concerns voiced here:
https://github.com/apache/polaris/pull/1758#discussion_r2121777758
--
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-unsubscr...@polaris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
