-
Notifications
You must be signed in to change notification settings - Fork 24
Replace stargate.jsonapi to data-api
#2208
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
base: main
Are you sure you want to change the base?
Conversation
|
I think there is also need to fix "env" versions of these settings:
|
Good point, thanks! Fixed. |
amorton
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approved, we can add a comment and ticket about something we are not changing, see comments.
| import io.smallrye.config.WithDefault; | ||
|
|
||
| @ConfigMapping(prefix = "stargate.jsonapi.http") | ||
| @ConfigMapping(prefix = "data-api.http") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add a comment in ApiConstants that we are leaving the constant as is there and create a ticket to remove the OfflineMode complely, I dont think we use it and we need to check that. It was for the sstable loader.
Code Coverage Report
|
What this PR does:
Replace
stargate.jsonapitodata-apiWhich issue(s) this PR fixes:
Fixes #2209
Checklist