Skip to content

Conversation

@gvprathyusha6
Copy link
Contributor

@gvprathyusha6 gvprathyusha6 commented Nov 12, 2024

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 gvprathyusha6 force-pushed the HBASE-28969 branch 2 times, most recently from 689010e to bc9a60b Compare December 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

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 44s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+0 🆗 codespell 0m 1s codespell was not available.
+0 🆗 detsecrets 0m 1s detect-secrets was not available.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
+1 💚 hbaseanti 0m 0s Patch does not have any anti-patterns.
_ master Compile Tests _
+1 💚 mvninstall 2m 54s master passed
+1 💚 compile 2m 36s master passed
+1 💚 checkstyle 0m 33s master passed
+1 💚 spotbugs 1m 28s master passed
+1 💚 spotless 0m 41s branch has no errors when running spotless:check.
_ Patch Compile Tests _
+1 💚 mvninstall 2m 38s the patch passed
+1 💚 compile 2m 35s the patch passed
+1 💚 javac 2m 35s the patch passed
+1 💚 blanks 0m 0s The patch has no blanks issues.
-0 ⚠️ checkstyle 0m 33s /results-checkstyle-hbase-server.txt hbase-server: The patch generated 1 new + 58 unchanged - 1 fixed = 59 total (was 59)
+1 💚 spotbugs 1m 32s the patch passed
+1 💚 hadoopcheck 9m 26s Patch does not cause any errors with Hadoop 3.3.6 3.4.0.
+1 💚 spotless 0m 42s patch has no errors when running spotless:check.
_ Other Tests _
+1 💚 asflicense 0m 9s The patch does not generate ASF License warnings.
32m 15s
Subsystem Report/Notes
Docker ClientAPI=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 Tests dupname asflicense javac spotbugs checkstyle codespell detsecrets compile hadoopcheck hbaseanti spotless
uname Linux 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 tool maven
Personality dev-support/hbase-personality.sh
git revision master / 81bda01
Default Java Eclipse Adoptium-17.0.11+9
Max. process+thread count 85 (vs. ulimit of 30000)
modules C: hbase-server U: hbase-server
Console output https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6459/5/console
versions git=2.34.1 maven=3.9.8 spotbugs=4.7.3
Powered by Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Logfile Comment
+0 🆗 reexec 0m 47s Docker mode activated.
-0 ⚠️ yetus 0m 3s Unprocessed 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 💚 mvninstall 4m 6s master passed
+1 💚 compile 1m 29s master passed
+1 💚 javadoc 0m 41s master passed
+1 💚 shadedjars 7m 2s branch has no errors when building our shaded downstream artifacts.
_ Patch Compile Tests _
+1 💚 mvninstall 3m 46s the patch passed
+1 💚 compile 1m 16s the patch passed
+1 💚 javac 1m 16s the patch passed
-0 ⚠️ javadoc 0m 43s /results-javadoc-javadoc-hbase-server.txt hbase-server generated 2 new + 61 unchanged - 0 fixed = 63 total (was 61)
+1 💚 shadedjars 7m 50s patch has no errors when building our shaded downstream artifacts.
_ Other Tests _
+1 💚 unit 209m 21s hbase-server in the patch passed.
241m 32s
Subsystem Report/Notes
Docker ClientAPI=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 Tests javac javadoc unit compile shadedjars
uname Linux 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 tool maven
Personality dev-support/hbase-personality.sh
git revision master / 81bda01
Default Java Eclipse Adoptium-17.0.11+9
Test Results https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6459/5/testReport/
Max. process+thread count 4528 (vs. ulimit of 30000)
modules C: hbase-server U: hbase-server
Console output https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-6459/5/console
versions git=2.34.1 maven=3.9.8
Powered by Apache Yetus 0.15.0 https://yetus.apache.org

This message was automatically generated.

@apurtell apurtell merged commit a680795 into apache:master Jan 13, 2025
1 check passed
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>
Sign up for free to 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