Skip to content
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

[Remove] CircuitBreaker Accounting #2056

Merged

Conversation

nknize
Copy link
Collaborator

@nknize nknize commented Feb 4, 2022

RAM Accounting of segments is removed so remove the associated Accounting
circuit breaker.

relates #2029

@nknize nknize added the v2.0.0 Version 2.0.0 label Feb 4, 2022
@nknize nknize requested a review from a team as a code owner February 4, 2022 20:22
@opensearch-ci-bot
Copy link
Collaborator

Can one of the admins verify this patch?

@opensearch-ci-bot
Copy link
Collaborator

❌   Gradle Check failure 6e8dbe94a45a925a66a6f5ec819cd83ad90a1de5
Log 2232

Reports 2232

@nknize
Copy link
Collaborator Author

nknize commented Feb 4, 2022

restarting gradle check. bwcVersions did not have the 1.2.5 bugfix version recently pushed

@nknize
Copy link
Collaborator Author

nknize commented Feb 4, 2022

start gradle check

@opensearch-ci-bot
Copy link
Collaborator

❌   Gradle Check failure 6e8dbe94a45a925a66a6f5ec819cd83ad90a1de5
Log 2234

Reports 2234

@nknize
Copy link
Collaborator Author

nknize commented Feb 4, 2022

start gradle check

@opensearch-ci-bot
Copy link
Collaborator

❌   Gradle Check failure 6e8dbe94a45a925a66a6f5ec819cd83ad90a1de5
Log 2235

Reports 2235

@nknize nknize force-pushed the remove/circuitbreakerAccounting branch from 6e8dbe9 to 3bdc492 Compare February 4, 2022 23:48
@opensearch-ci-bot
Copy link
Collaborator

❌   Gradle Check failure 3bdc492aef493ff4d23ebadb16242112f79369ef
Log 2236

Reports 2236

RAM Accounting of segments is removed so remove the associated Accounting
circuit breaker.

Signed-off-by: Nicholas Walter Knize <nknize@apache.org>
@nknize nknize force-pushed the remove/circuitbreakerAccounting branch from 3bdc492 to ed5f262 Compare February 5, 2022 00:26
@opensearch-ci-bot
Copy link
Collaborator

✅   Gradle Check success ed5f262
Log 2237

Reports 2237

@nknize nknize merged commit 8288c90 into opensearch-project:main Feb 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
v2.0.0 Version 2.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants