-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
HBASE-24657 add unit test for JSONBean.java #2004
Conversation
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
hbase-http/src/test/java/org/apache/hadoop/hbase/util/TestJSONBean.java
Outdated
Show resolved
Hide resolved
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
🎊 +1 overall
This message was automatically generated. |
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
Closes #2004 Signed-off-by: Viraj Jasani <vjasani@apache.org>
Closes #2004 Signed-off-by: Viraj Jasani <vjasani@apache.org>
Closes #2004 Signed-off-by: Viraj Jasani <vjasani@apache.org>
Closes #2004 Signed-off-by: Viraj Jasani <vjasani@apache.org>
Closes apache#2004 Signed-off-by: Viraj Jasani <vjasani@apache.org>
Closes apache#2004 Signed-off-by: Viraj Jasani <vjasani@apache.org> (cherry picked from commit d1c3df8) Change-Id: I149ebda19e1c1b296a238194d4adf626c65f75fd
HBASE-24657
A bug existed in JSONBean.java for branch-1. This PR migrates the test which covers the scenario into master. There is no product fix required other than in branch-1, PR for that fix is #1999.
I tried to keep the tests mostly in sync between branch-1 and master, but there are variations between branches which required changes.