-
Notifications
You must be signed in to change notification settings - Fork 285
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
Updates to APQ #923
Updates to APQ #923
Conversation
Ping @berngp |
This pull request has been marked as stale because it has been open 1 year with no activity. Remove stale label or comment or this will be closed in 7 days |
@berngp is there still interest from Netflix in integrating this? If yes I'll update it one last time Otherwise, I'll close it :) |
@jord1e - Thanks so much for all the work. We won't be investing in this effort at this time due to other internal priorities. Internally, we use APQ differently and do not require APQ support on our DGSs at the moment. Feel free to close the PR and we will reopen at a more convenient time. |
I've run back my original issue, while upgrading to spring boot 3 and bringing in these dependencies:
Any chance we could get this merged in? Or is it going to break your integration? @srinivasankavitha |
@Ancient-Dragon - we won't be merging this PR at this time since we are not using this feature and have conflicting internal priorities to invest in. |
Pull request checklist
first
Pull Request type
Changes in this PR
Fixes problems arising from #905 (will update description later)
Screenshots:
WebFlux smoke test (without Caffeine support):
WebMvc smoke test (with Caffeine support):
Alternatives considered
N/A
cc @berngp