Skip to content

HBASE-27746 Check if the file system supports storage policy before invoking setStoragePolicy() - #6172

Closed
jojochuang wants to merge 1 commit into
apache:branch-2from
jojochuang:HBASE-27746-branch-2
Closed

HBASE-27746 Check if the file system supports storage policy before invoking setStoragePolicy()#6172
jojochuang wants to merge 1 commit into
apache:branch-2from
jojochuang:HBASE-27746-branch-2

Conversation

@jojochuang

@jojochuangjojochuang commented Aug 21, 2024

Copy link
Copy Markdown
Contributor

No description provided.

@jojochuang
HBASE-27746 Check if the file system supports storage policy before i…
d93c0c0
…nvoking setStoragePolicy() (apache#5189)
(cherry picked from commit 0646151)
(cherry picked from commit 35b11eb)
@Apache-HBase

Apache-HBase commented Aug 21, 2024

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 45sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗codespell0m 0scodespell was not available.
+0 🆗detsecrets0m 0sdetect-secrets was not available.
+1 💚@author0m 0sThe patch does not contain any @author tags.
+1 💚hbaseanti0m 0sPatch does not have any anti-patterns.
_ branch-2 Compile Tests _
+0 🆗mvndep0m 12sMaven dependency ordering for branch
+1 💚mvninstall3m 18sbranch-2 passed
+1 💚compile3m 28sbranch-2 passed
+1 💚checkstyle0m 52sbranch-2 passed
+1 💚spotbugs2m 9sbranch-2 passed
+1 💚spotless0m 47sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+0 🆗mvndep0m 16sMaven dependency ordering for patch
+1 💚mvninstall3m 8sthe patch passed
+1 💚compile3m 30sthe patch passed
+1 💚javac3m 30sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
+1 💚checkstyle0m 52sthe patch passed
+1 💚spotbugs2m 21sthe patch passed
-1 ❌hadoopcheck0m 31sThe patch causes 22 errors with Hadoop v2.10.2.
+1 💚spotless0m 45spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 19sThe patch does not generate ASF License warnings.
37m 30s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6172/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#6172
Optional Testsdupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless
unameLinux 04685c823b1a 5.4.0-1103-aws #111~18.04.1-Ubuntu SMP Tue May 23 20:04:10 UTC 2023 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / d93c0c0
Default JavaEclipse Adoptium-11.0.23+9
hadoopcheckhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6172/1/artifact/yetus-general-check/output/patch-javac-2.10.2.txt
Max. process+thread count77 (vs. ulimit of 30000)
modulesC: hbase-common hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6172/1/console
versionsgit=2.34.1 maven=3.9.8 spotbugs=4.7.3
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Apache-HBase commented Aug 21, 2024

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 56sDocker mode activated.
-0 ⚠️yetus0m 5sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+0 🆗mvndep0m 10sMaven dependency ordering for branch
+1 💚mvninstall3m 13sbranch-2 passed
+1 💚compile1m 32sbranch-2 passed
+1 💚javadoc0m 46sbranch-2 passed
+1 💚shadedjars5m 39sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 19sMaven dependency ordering for patch
+1 💚mvninstall3m 6sthe patch passed
+1 💚compile1m 12sthe patch passed
+1 💚javac1m 12sthe patch passed
+1 💚javadoc0m 45sthe patch passed
+1 💚shadedjars5m 34spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit2m 46shbase-common in the patch passed.
+1 💚unit239m 31shbase-server in the patch passed.
270m 52s
SubsystemReport/Notes
DockerClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6172/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR#6172
Optional Testsjavac javadoc unit compile shadedjars
unameLinux 1bd86db8543c 5.4.0-186-generic #206-Ubuntu SMP Fri Apr 26 12:31:10 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / d93c0c0
Default JavaEclipse Adoptium-11.0.23+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6172/1/testReport/
Max. process+thread count4326 (vs. ulimit of 30000)
modulesC: hbase-common hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6172/1/console
versionsgit=2.34.1 maven=3.9.8
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Apache-HBase commented Aug 22, 2024

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 49sDocker mode activated.
-0 ⚠️yetus0m 5sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+0 🆗mvndep0m 11sMaven dependency ordering for branch
+1 💚mvninstall4m 53sbranch-2 passed
+1 💚compile1m 58sbranch-2 passed
+1 💚javadoc1m 6sbranch-2 passed
+1 💚shadedjars8m 53sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 31sMaven dependency ordering for patch
+1 💚mvninstall4m 31sthe patch passed
+1 💚compile1m 43sthe patch passed
+1 💚javac1m 43sthe patch passed
+1 💚javadoc1m 11sthe patch passed
+1 💚shadedjars8m 39spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit3m 51shbase-common in the patch passed.
-1 ❌unit285m 41s/patch-unit-hbase-server.txthbase-server in the patch failed.
329m 32s
SubsystemReport/Notes
DockerClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6172/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#6172
Optional Testsjavac javadoc unit compile shadedjars
unameLinux 052dce34564b 5.4.0-192-generic #212-Ubuntu SMP Fri Jul 5 09:47:39 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / d93c0c0
Default JavaEclipse Adoptium-17.0.11+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6172/1/testReport/
Max. process+thread count4473 (vs. ulimit of 30000)
modulesC: hbase-common hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6172/1/console
versionsgit=2.34.1 maven=3.9.8
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase

Apache-HBase commented Aug 22, 2024

Copy link
Copy Markdown

💔 -1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec1m 3sDocker mode activated.
-0 ⚠️yetus0m 5sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ branch-2 Compile Tests _
+0 🆗mvndep0m 10sMaven dependency ordering for branch
+1 💚mvninstall3m 50sbranch-2 passed
+1 💚compile1m 18sbranch-2 passed
+1 💚javadoc0m 54sbranch-2 passed
+1 💚shadedjars5m 55sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+0 🆗mvndep0m 14sMaven dependency ordering for patch
-1 ❌mvninstall0m 43s/patch-mvninstall-root.txtroot in the patch failed.
-1 ❌compile0m 15s/patch-compile-hbase-common.txthbase-common in the patch failed.
-0 ⚠️javac0m 15s/patch-compile-hbase-common.txthbase-common in the patch failed.
-0 ⚠️javadoc0m 18s/results-javadoc-javadoc-hbase-common.txthbase-common generated 2 new + 2 unchanged - 0 fixed = 4 total (was 2)
-1 ❌shadedjars1m 19spatch has 22 errors when building our shaded downstream artifacts.
_ Other Tests _
-1 ❌unit0m 22s/patch-unit-hbase-common.txthbase-common in the patch failed.
-1 ❌unit327m 34s/patch-unit-hbase-server.txthbase-server in the patch failed.
351m 43s
SubsystemReport/Notes
DockerClientAPI=1.46 ServerAPI=1.46 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6172/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile
GITHUB PR#6172
Optional Testsjavac javadoc unit compile shadedjars
unameLinux b400786e28ff 5.4.0-186-generic #206-Ubuntu SMP Fri Apr 26 12:31:10 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionbranch-2 / d93c0c0
Default JavaTemurin-1.8.0_412-b08
shadedjarshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6172/1/artifact/yetus-jdk8-hadoop2-check/output/patch-shadedjars.txt
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6172/1/testReport/
Max. process+thread count3955 (vs. ulimit of 30000)
modulesC: hbase-common hbase-server U: .
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6172/1/console
versionsgit=2.34.1 maven=3.9.8
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@jojochuang

jojochuang commented Aug 22, 2024

Copy link
Copy Markdown
ContributorAuthor

Forgot this is not supported by Hadoop 2. Drop the PR for now.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jojochuang@Apache-HBase