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
[client] read submission for readonly handle should bypass OSE threads. #1804
Conversation
*Motivation* apache#1791 improves performance by bypassing OSE threads for read request submission. however the change doesn't handle `tailing` ReadOnlyLedgerHandle *Changes* ReadOnlyLedgerHandle overrides `isHandleWritable` and return `false` always.
@nicmichael can you please review this PR to see if that matches your intention at #1791 ? |
In client tests there are a lot of failure around fencing. |
retest this please |
1 similar comment
retest this please |
there are still a lot of errors |
@sijie Yes that was the intention. |
maybe the errors on recovery are due to the fact that reads should strictly respect the order |
During recovery I suppose they should... |
yeah. I think we don't this change. we should just close it. |
I think what we need is something like this:
It passes tests on my workstation. I'm just not quite sure whether this check needs any synchronization or can go without. |
@nicmichael I think it is a bit risky to take |
Sounds good to me. The most common case is probably addressed by the earlier change. |
Okay, we don't need this. |
Descriptions of the changes in this PR:
Motivation
#1791 improves performance by bypassing OSE threads for read request submission.
however the change doesn't handle
tailing
ReadOnlyLedgerHandleChanges
ReadOnlyLedgerHandle overrides
isHandleWritable
and returnfalse
always.