Skip to content

HBASE-28969 Move HFileLink file creations to SFT - #6459

Merged
apurtell merged 1 commit into
apache:masterfrom
gvprathyusha6:HBASE-28969
Jan 13, 2025
Merged

HBASE-28969 Move HFileLink file creations to SFT#6459
apurtell merged 1 commit into
apache:masterfrom
gvprathyusha6:HBASE-28969

Conversation

@gvprathyusha6

@gvprathyusha6gvprathyusha6 commented Nov 12, 2024

Copy link
Copy Markdown
Contributor

Post HBASE-28564, HStoreFile creation is SFT aware and Reference file creations are moved to SFT.
Current change refactors all HFileLink file creations to SFT, so as to support Virtual links creation via FSFT in HBASE-27826

@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.

@gvprathyusha6
gvprathyusha6force-pushed the HBASE-28969 branch 2 times, most recently from 689010e to bc9a60bCompareDecember 19, 2024 14:34
@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

Copy link
Copy Markdown

🎊 +1 overall

VoteSubsystemRuntimeLogfileComment
+0 🆗reexec0m 44sDocker mode activated.
_ Prechecks _
+1 💚dupname0m 0sNo case conflicting files found.
+0 🆗codespell0m 1scodespell was not available.
+0 🆗detsecrets0m 1sdetect-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 💚mvninstall2m 54smaster passed
+1 💚compile2m 36smaster passed
+1 💚checkstyle0m 33smaster passed
+1 💚spotbugs1m 28smaster passed
+1 💚spotless0m 41sbranch has no errors when running spotless:check.
_ Patch Compile Tests _
+1 💚mvninstall2m 38sthe patch passed
+1 💚compile2m 35sthe patch passed
+1 💚javac2m 35sthe patch passed
+1 💚blanks0m 0sThe patch has no blanks issues.
-0 ⚠️checkstyle0m 33s/results-checkstyle-hbase-server.txthbase-server: The patch generated 1 new + 58 unchanged - 1 fixed = 59 total (was 59)
+1 💚spotbugs1m 32sthe patch passed
+1 💚hadoopcheck9m 26sPatch 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 9sThe patch does not generate ASF License warnings.
32m 15s
SubsystemReport/Notes
DockerClientAPI=1.47 ServerAPI=1.47 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6459/5/artifact/yetus-general-check/output/Dockerfile
GITHUB PR#6459
Optional Testsdupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless
unameLinux 4711c59b7caa 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 / 81bda01
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-6459/5/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 47sDocker 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 💚mvninstall4m 6smaster passed
+1 💚compile1m 29smaster passed
+1 💚javadoc0m 41smaster passed
+1 💚shadedjars7m 2sbranch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+1 💚mvninstall3m 46sthe patch passed
+1 💚compile1m 16sthe patch passed
+1 💚javac1m 16sthe patch passed
-0 ⚠️javadoc0m 43s/results-javadoc-javadoc-hbase-server.txthbase-server generated 2 new + 61 unchanged - 0 fixed = 63 total (was 61)
+1 💚shadedjars7m 50spatch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚unit209m 21shbase-server in the patch passed.
241m 32s
SubsystemReport/Notes
DockerClientAPI=1.47 ServerAPI=1.47 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6459/5/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
GITHUB PR#6459
Optional Testsjavac javadoc unit compile shadedjars
unameLinux 31b8b37daddc 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 / 81bda01
Default JavaEclipse Adoptium-17.0.11+9
Test Resultshttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6459/5/testReport/
Max. process+thread count4528 (vs. ulimit of 30000)
modulesC: hbase-server U: hbase-server
Console outputhttps://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6459/5/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 a680795 into apache:masterJan 13, 2025
asfgit pushed a commit that referenced this pull request Jan 13, 2025
Signed-off-by: Andrew Purtell <apurtell@apache.org>
Co-authored-by: Prathyusha Garre <pgarre@pgarre-ltmqft8.internal.salesforce.com>
gvprathyusha6 added a commit to gvprathyusha6/hbase that referenced this pull request Oct 13, 2025
Signed-off-by: Andrew Purtell <apurtell@apache.org>
Co-authored-by: Prathyusha Garre <pgarre@pgarre-ltmqft8.internal.salesforce.com>
gvprathyusha6 added a commit to gvprathyusha6/hbase that referenced this pull request Dec 19, 2025
Signed-off-by: Andrew Purtell <apurtell@apache.org>
Co-authored-by: Prathyusha Garre <pgarre@pgarre-ltmqft8.internal.salesforce.com>
gvprathyusha6 added a commit to gvprathyusha6/hbase that referenced this pull request Dec 19, 2025
Signed-off-by: Andrew Purtell <apurtell@apache.org>
Co-authored-by: Prathyusha Garre <pgarre@pgarre-ltmqft8.internal.salesforce.com>
gvprathyusha6 added a commit to gvprathyusha6/hbase that referenced this pull request Dec 22, 2025
Signed-off-by: Andrew Purtell <apurtell@apache.org>
Co-authored-by: Prathyusha Garre <pgarre@pgarre-ltmqft8.internal.salesforce.com>
apurtell pushed a commit that referenced this pull request Mar 12, 2026
Signed-off-by: Andrew Purtell <apurtell@apache.org>
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.

3 participants

@gvprathyusha6@Apache-HBase@apurtell