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

HLRC: ClearScroll API ignores 404 #58558

Closed
wants to merge 1 commit into from

Conversation

tumile
Copy link
Contributor

@tumile tumile commented Jun 25, 2020

Add 404 to ignores in clearScroll and clearScollAsync.

Resolves #56896.

Copy link

@xtermi2 xtermi2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. This should close #56896

@LiuCao0614
Copy link

LiuCao0614 commented May 5, 2021

Is this change already in master branch? We are experiencing the same issue and we'd like to have this fix in newer version.

@xtermi2
Copy link

xtermi2 commented May 6, 2021

I can confirm, that this issue is not fixed in any released version, don't know why this PR is not merged.
We catch the exception as a workaround.

@LiuCao0614
Copy link

Gotcha. We can probably use the same workaround at this point. Do we know if it is possible to merge this change before next release? It would be great if we can have this fixed.

@dakrone
Copy link
Member

dakrone commented Mar 8, 2024

Closing this as we've removed the high level rest client in favor of the Java client.

@dakrone dakrone closed this Mar 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants