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

Correctly set X-Partial-Results #2348

Open
wants to merge 8 commits into
base: integration
Choose a base branch
from
Open

Conversation

foster33
Copy link
Collaborator

Fixes #879

Reworks #1195 against the current integration branch... it was originally merged into feature/accumulo-2.0 but is missing in integration.

avgAGB
avgAGB previously approved these changes Apr 24, 2024
ivakegg
ivakegg previously approved these changes May 8, 2024
@foster33 foster33 dismissed stale reviews from ivakegg and avgAGB via 7a9633b June 26, 2024 12:47
@foster33 foster33 requested a review from jwomeara as a code owner June 26, 2024 12:47
@foster33
Copy link
Collaborator Author

foster33 commented Jun 27, 2024

LongRunningQueryTest.testLongRunningQueryWithSmallPageSize() seems to be a flaky test due to short circuit timeouts. Looking into this. Flaky test fixed.

@foster33 foster33 requested review from ivakegg and avgAGB July 2, 2024 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Correctly set X-Partial-Results
3 participants