-
Notifications
You must be signed in to change notification settings - Fork 3.4k
HBASE-22962 Fix typo in javadoc description #569
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
Hi, Thanks for this PR. The HBase project mainly use The ASF JIRA for tracking issues, would you mind filing a jira and describing your problem there? For how to contribute to HBase, please refer to Get involved for more details. And feel free to email your questions or doubts to dev@hbase.apache.org for help. |
|
🎊 +1 overall
This message was automatically generated. |
LGTM +1 (non binding) |
(I reviewed the patch, and then realized it's a HBase patch, not a Hadoop patch. Still, since I invested my time in this review, I thought to post a patch review to save other reviewers' time) |
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
Merged to master. Thanks @jojochuang for reviewing. |
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Signed-off-by: Duo Zhang <zhangduo@apache.org>
We love reviews from not-yet-committers. Please feel free to keep doing them. It definitely helps me if someone else has taken a look at a change request. 😸 |
Signed-off-by: Duo Zhang <zhangduo@apache.org>
Signed-off-by: Duo Zhang <zhangduo@apache.org> (cherry picked from commit 1f9d650) Change-Id: I67e7751ab45611b8f49388424f16698a07eb4b97
No description provided.