Skip to content

HBASE-28861 Use hasReferences api of SFT in RegionSplitter and remove… - #6428

Merged
apurtell merged 1 commit into
apache:masterfrom
gvprathyusha6:HBASE-28861
Jan 23, 2025
Merged

HBASE-28861 Use hasReferences api of SFT in RegionSplitter and remove…#6428
apurtell merged 1 commit into
apache:masterfrom
gvprathyusha6:HBASE-28861

Conversation

@gvprathyusha6

Copy link
Copy Markdown
Contributor

… Reference.read(), HRegionFS.hasReferences()

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

@Apache-HBase

This comment has been minimized.

…ve Reference.read(), HRegionFS.hasReferences()
@Apache-HBase

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 27sDocker 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.
_ master Compile Tests _
+1 💚mvninstall3m 5smaster passed
+1 💚compile3m 5smaster passed
+1 💚checkstyle0m 37smaster passed
+1 💚spotbugs1m 34smaster passed
+1 💚spotless0m 45sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+1 💚mvninstall2m 53sthe patch passed
+1 💚compile3m 1sthe patch passed
+1 💚javac3m 1sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
-0 ⚠️checkstyle0m 35s/results-checkstyle-hbase-server.txthbase-server: The patch generated 1 new + 22 unchanged - 2 fixed = 23 total (was 24)
+1 💚spotbugs1m 39sthe patch passed
+1 💚hadoopcheck10m 57sPatch does not cause any errors with Hadoop 3.3.6 3.4.0.
+1 💚spotless0m 42spatch has no errors when running spotless:check.
_ Other Tests _
+1 💚asflicense0m 10sThe patch does not generate ASF License warnings.
36m 26s
SubsystemReport/Notes
DockerClientAPI=1.43 ServerAPI=1.43 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6428/4/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#6428
Optional Testsdupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless
unameLinux a2eff6fd31e6 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 revisionmaster / 6febdd6
Default JavaEclipse Adoptium-17.0.11+9
Max. process+thread count85 (vs. ulimit of 30000)
modulesC: hbase-server U: hbase-server
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6428/4/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

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 43sDocker mode activated.
-0 ⚠️yetus0m 3sUnprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --author-ignore-list --blanks-eol-ignore-file --blanks-tabs-ignore-file --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+1 💚mvninstall2m 42smaster passed
+1 💚compile0m 53smaster passed
+1 💚javadoc0m 29smaster passed
+1 💚shadedjars4m 55sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+1 💚mvninstall2m 39sthe patch passed
+1 💚compile0m 52sthe patch passed
+1 💚javac0m 52sthe patch passed
+1 💚javadoc0m 26sthe patch passed
+1 💚shadedjars4m 52spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit185m 7shbase-server in the patch passed.
208m 2s
SubsystemReport/Notes
DockerClientAPI=1.47 ServerAPI=1.47 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6428/4/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#6428
Optional Testsjavac javadoc unit compile shadedjars
unameLinux 28abae5ebc1c 5.4.0-200-generic #220-Ubuntu SMP Fri Sep 27 13:19:16 UTC 2024 x86_64 x86_64 x86_64 GNU/Linux
Build toolmaven
Personalitydev-support/hbase-personality.sh
git revisionmaster / 6febdd6
Default JavaEclipse Adoptium-17.0.11+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6428/4/testReport/
Max. process+thread count5683 (vs. ulimit of 30000)
modulesC: hbase-server U: hbase-server
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6428/4/console
versionsgit=2.34.1 maven=3.9.8
Powered byApache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@apurtell
apurtell merged commit 89d5f4e into apache:masterJan 23, 2025
asfgit pushed a commit that referenced this pull request Jan 23, 2025
…ve Reference.read(), HRegionFS.hasReferences() (#6428)
Signed-off-by: Andrew Purtell <apurtell@apache.org>
Co-authored-by: Prathyusha Garre <pgarre@pgarre-ltmqft8.internal.salesforce.com>
mnpoonia pushed a commit to mnpoonia/hbase that referenced this pull request Jan 24, 2025
…ve Reference.read(), HRegionFS.hasReferences() (apache#6428)
Signed-off-by: Andrew Purtell <apurtell@apache.org>
Co-authored-by: Prathyusha Garre <pgarre@pgarre-ltmqft8.internal.salesforce.com>
mnpoonia pushed a commit to mnpoonia/hbase that referenced this pull request Jan 24, 2025
…ve Reference.read(), HRegionFS.hasReferences() (apache#6428)
Signed-off-by: Andrew Purtell <apurtell@apache.org>
Co-authored-by: Prathyusha Garre <pgarre@pgarre-ltmqft8.internal.salesforce.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@gvprathyusha6@Apache-HBase@apurtell@mnpoonia