Skip to content
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.

Sign up for GitHub

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

Jump to bottom

HBASE-26166 Fix a minor bug of table list on master ui #3554

Merged
merged 1 commit into from
Aug 4, 2021
Merged

HBASE-26166 Fix a minor bug of table list on master ui #3554

merged 1 commit into from
Aug 4, 2021

Conversation

frostruan
Copy link
Contributor

@frostruan frostruan commented Aug 3, 2021 •
edited by guangxuCheng
Loading

before
image
after
image

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 1m 0s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ master Compile Tests _
+1 💚 mvninstall 5m 8s master passed
_ Patch Compile Tests _
+1 💚 mvninstall 4m 42s the patch passed
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
_ Other Tests _
+1 💚 asflicense 0m 15s The patch does not generate ASF License warnings.
12m 54s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3554/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR #3554
Optional Tests dupname asflicense javac
uname Linux 92c4824f149e 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision master / de84c17
Default Java AdoptOpenJDK-1.8.0_282-b08
Max. process+thread count 78 (vs. ulimit of 30000)
modules C: hbase-server U: hbase-server
Console output https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3554/1/console
versions git=2.17.1 maven=3.6.3
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 32s Docker mode activated.
-0 ⚠️ yetus 0m 4s Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+1 💚 mvninstall 4m 31s master passed
+1 💚 javadoc 0m 43s master passed
_ Patch Compile Tests _
+1 💚 mvninstall 4m 17s the patch passed
+1 💚 javadoc 0m 40s the patch passed
_ Other Tests _
+1 💚 unit 138m 15s hbase-server in the patch passed.
151m 10s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3554/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR #3554
Optional Tests javac javadoc unit
uname Linux 80c3e6f347b2 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision master / de84c17
Default Java AdoptOpenJDK-11.0.10+9
Test Results https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3554/1/testReport/
Max. process+thread count 4335 (vs. ulimit of 30000)
modules C: hbase-server U: hbase-server
Console output https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3554/1/console
versions git=2.17.1 maven=3.6.3
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 4m 12s Docker mode activated.
-0 ⚠️ yetus 0m 3s Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+1 💚 mvninstall 4m 0s master passed
+1 💚 javadoc 0m 42s master passed
_ Patch Compile Tests _
+1 💚 mvninstall 3m 45s the patch passed
+1 💚 javadoc 0m 37s the patch passed
_ Other Tests _
+1 💚 unit 146m 47s hbase-server in the patch passed.
162m 26s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3554/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR #3554
Optional Tests javac javadoc unit
uname Linux 4dab558dc9e2 4.15.0-151-generic #157-Ubuntu SMP Fri Jul 9 23:07:57 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision master / de84c17
Default Java AdoptOpenJDK-1.8.0_282-b08
Test Results https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3554/1/testReport/
Max. process+thread count 4029 (vs. ulimit of 30000)
modules C: hbase-server U: hbase-server
Console output https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3554/1/console
versions git=2.17.1 maven=3.6.3
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

guangxuCheng
guangxuCheng approved these changes Aug 3, 2021
View reviewed changes
Copy link
Member

@guangxuCheng guangxuCheng left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM +1

@shahrs87
Copy link
Contributor

shahrs87 commented Aug 3, 2021

@frostruan We are not seeing the table attributes after the change ?

@frostruan
Copy link
Contributor Author

@frostruan We are not seeing the table attributes after the change ?
it's different cluster sir.

shahrs87
shahrs87 approved these changes Aug 3, 2021
View reviewed changes
Copy link
Contributor

@shahrs87 shahrs87 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 ltgm

@guangxuCheng guangxuCheng changed the title HBASE-26166 fix a minor table display bug in master ui HBASE-26166 Fix a minor bug of table list on master ui Aug 4, 2021
@guangxuCheng guangxuCheng merged commit 73a0411 into apache:master Aug 4, 2021
guangxuCheng pushed a commit that referenced this pull request Aug 4, 2021
@frostruan @guangxuCheng
HBASE-26166 Fix a minor bug of table list on master ui (#3554)
fcc67c2 
Signed-off-by: Guangxu Cheng <gxcheng@apache.org>
Signed-off-by: Rushabh Shah <shahrs87@gmail.com>
guangxuCheng pushed a commit that referenced this pull request Aug 4, 2021
@frostruan @guangxuCheng
HBASE-26166 Fix a minor bug of table list on master ui (#3554)
348c392 
Signed-off-by: Guangxu Cheng <gxcheng@apache.org>
Signed-off-by: Rushabh Shah <shahrs87@gmail.com>
guangxuCheng pushed a commit that referenced this pull request Aug 4, 2021
@frostruan @guangxuCheng
HBASE-26166 Fix a minor bug of table list on master ui (#3554)
04d3c7a 
Signed-off-by: Guangxu Cheng <gxcheng@apache.org>
Signed-off-by: Rushabh Shah <shahrs87@gmail.com>
guangxuCheng pushed a commit to guangxuCheng/hbase that referenced this pull request Mar 28, 2022
HBASE-26166 Fix a minor bug of table list on master ui (apache#3554)
422b9ed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@shahrs87 shahrs87 shahrs87 approved these changes

@guangxuCheng guangxuCheng guangxuCheng approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@frostruan @Apache-HBase @shahrs87 @guangxuCheng