-
Notifications
You must be signed in to change notification settings - Fork 3.4k
HBASE-26071: Document HBASE-26021 and upgrade considerations for 1.7.… #3469
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
Conversation
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
| version. 1.7.0 was withdrawn and removed from the Apache sites. | ||
| * If you already installed a 1.7.0 cluster from scratch and are looking to migrate to 1.7.1+, you | ||
| cannot follow the regular rolling upgrade procedures due to broken compatibility contracts. | ||
| Instead shutdown the cluster and reboot with 1.7.1+ binaries. Newer versions detect any existing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tiny nit: double space between "the" and "cluster"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ack.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For 1.7.0 to 1.7.1, do clients have to shutdown the cluster? I thought it was about restarting master only.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In 1.7.0, client expects TableState (faulty serialization pb message) in GetTableStateResponse. Given that serialization has been undone (as we do not want to expose TableState anymore), client is not compatible (certain RPCs) with restarted Master. So it is safe to shutdown, update binaries everywhere and reboot the cluster.
I think this is ok given it is impossible that any serious user has upgraded an existing cluster to 1.7.0. Its only about new installations.
|
|
||
| * If you are considering an upgrade to 1.7.x version, skip 1.7.0 completely and upgrade to 1.7.1+ | ||
| version. 1.7.0 was withdrawn and removed from the Apache sites. | ||
| * If you already installed a 1.7.0 cluster from scratch and are looking to migrate to 1.7.1+, you |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What if I upgraded from 1.6 to 1.7.0 and am now running 1.7.0?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You cannot, the bug doesn't let you upgrade. During the upgrade, 1.7.0 binaries try to parse the table descriptors and it blows up (META to begin with).
Reidddddd
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left one Q.
| version. 1.7.0 was withdrawn and removed from the Apache sites. | ||
| * If you already installed a 1.7.0 cluster from scratch and are looking to migrate to 1.7.1+, you | ||
| cannot follow the regular rolling upgrade procedures due to broken compatibility contracts. | ||
| Instead shutdown the cluster and reboot with 1.7.1+ binaries. Newer versions detect any existing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For 1.7.0 to 1.7.1, do clients have to shutdown the cluster? I thought it was about restarting master only.
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
|
🎊 +1 overall
This message was automatically generated. |
Reidddddd
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
virajjasani
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1, one question: we can merge this only after withdrawing and removing mirrors of 1.7.0 release?
Correct, waiting for them to cleaned up everywhere. Will merge once done. |
…0/1.7.1