Uh oh!
There was an error while loading. Please reload this page.
HBASE-24877 Add option to avoid aborting RS process upon uncaught exceptions happen on replication source - #2399
Merged
Conversation
…eptions happen on replication source
Apache9
commented
Sep 15, 2020
Contributor
Please create addendum first and then backport it to branch-2? |
Apache9
commented
Sep 15, 2020
Contributor
And while reading the current code of master, the loop in the startup method does not have a sleep? And yes the retryStartup flag has to be AtomicBoolean as it will be set in another thread, though it is only used in the startup method... And now, seems we will block in the startup method forever? I think a better solution is to do the retry in the initThread so we will not block the startup call? I think the reason we use a initThread is to avoid blocking the startup call, if this is not the case, why not just move the initialize call into the startup method... Thanks. |
Apache-HBase
commented
Sep 15, 2020
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Sep 15, 2020
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Sep 15, 2020
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Sep 15, 2020
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Sep 15, 2020
🎊 +1 overall
This message was automatically generated. |
wchevreuil
commented
Sep 15, 2020
ContributorAuthor
Sure, addendum is created here: #2400
Holy cow, you're right @Apache9 . That's not the original intention, we should not block startup here, I was so eager to fix the race condition issue and didn't notice the side effect.
Definitely, yeah. Let me do this. |
…ace control in the initialize loop (apache#2400) Signed-off-by: Duo Zhang <zhangduo@apache.org> Signed-off-by: Josh Elser <elserj@apache.org>
Apache-HBase
commented
Oct 2, 2020
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Oct 5, 2020
💔 -1 overall
This message was automatically generated. |
2 similar comments
Apache-HBase
commented
Oct 5, 2020
💔 -1 overall
This message was automatically generated. |
Apache-HBase
commented
Oct 5, 2020
💔 -1 overall
This message was automatically generated. |
wchevreuil
commented
Oct 5, 2020
ContributorAuthor
ping @Apache9 , had tried cherry-picking the changes of the master addendum in here, noticed first UTs never finishing on the first pre-commit run (could be related to this PR), but never found the patch-unit-hbase-server.txt artifact was not available, then tried submit the build again and got this docker issue:
Was talking to @busbey , he mentioned that you had done some recent work for queuing up builds in the new jenkins nodes, would you know this should avoid this kind of errors? |
Apache9
commented
Oct 6, 2020
Contributor
The pre commit job is still on the old Hadoop nodes, so it should not be effected by the new nodes. Maybe the problem is that our replication related tests are all hanging recently and generate too many logs which cause the surefire plugin to cache a lot of stdout and stderr files under /tmp and make the nodes unavailable. The new jenkins node has a small root disk so it is easy to be full, but finally, we also blow up the H nodes? |
Apache-HBase
commented
Oct 7, 2020
💔 -1 overall
This message was automatically generated. |
Apache-HBase
commented
Oct 7, 2020
💔 -1 overall
This message was automatically generated. |
Apache-HBase
commented
Oct 7, 2020
💔 -1 overall
This message was automatically generated. |
Apache-HBase
commented
Oct 12, 2020
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Oct 12, 2020
🎊 +1 overall
This message was automatically generated. |
Apache-HBase
commented
Oct 12, 2020
🎊 +1 overall
This message was automatically generated. |
saintstack
approved these changes
Oct 14, 2020
clarax pushed a commit
to clarax/hbase
that referenced
this pull request
Nov 15, 2020
…eptions happen on replication source (apache#2399) Signed-off-by: stack <stack@apache.org>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PR for branch-2, after cherry-picking and resolving conflicts. Found one extra possible race condition while testing this, I think we need an addendum for the master one.