Re: [PR] HBASE-28994 Update backup describe command for continuous backup [hbase]
Apache-HBase commented on PR #7045:
URL: https://github.com/apache/hbase/pull/7045#issuecomment-3004412957
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexec | 0m 32s | | Docker mode activated. |
_ Prechecks _ |
| +1 :green_heart: | dupname | 0m 0s | | No case conflicting files
found. |
| +0 :ok: | codespell | 0m 0s | | codespell was not available. |
| +0 :ok: | detsecrets | 0m 0s | | detect-secrets was not available.
|
| +0 :ok: | buf | 0m 0s | | buf was not available. |
| +0 :ok: | buf | 0m 0s | | buf was not available. |
| +1 :green_heart: | @author | 0m 0s | | The patch does not contain
any @author tags. |
| +1 :green_heart: | hbaseanti | 0m 0s | | Patch does not have any
anti-patterns. |
_ HBASE-28957 Compile Tests _ |
| +0 :ok: | mvndep | 0m 11s | | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 2m 57s | | HBASE-28957 passed |
| +1 :green_heart: | compile | 1m 8s | | HBASE-28957 passed |
| -0 :warning: | checkstyle | 0m 9s |
[/buildtool-branch-checkstyle-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/6/artifact/yetus-general-check/output/buildtool-branch-checkstyle-hbase-backup.txt)
| The patch fails to run checkstyle in hbase-backup |
| +1 :green_heart: | spotbugs | 2m 40s | | HBASE-28957 passed |
| +1 :green_heart: | spotless | 0m 41s | | branch has no errors when
running spotless:check. |
| -0 :warning: | patch | 0m 54s | | Used diff version of patch file.
Binary files and potentially other changes not applied. Please rebase and
squash commits if necessary. |
_ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 11s | | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 2m 52s | | the patch passed |
| +1 :green_heart: | compile | 1m 4s | | the patch passed |
| +1 :green_heart: | cc | 1m 4s | | the patch passed |
| -0 :warning: | javac | 0m 30s |
[/results-compile-javac-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/6/artifact/yetus-general-check/output/results-compile-javac-hbase-backup.txt)
| hbase-backup generated 1 new + 116 unchanged - 0 fixed = 117 total (was
116) |
| +1 :green_heart: | blanks | 0m 0s | | The patch has no blanks
issues. |
| -0 :warning: | checkstyle | 0m 8s |
[/buildtool-patch-checkstyle-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/6/artifact/yetus-general-check/output/buildtool-patch-checkstyle-hbase-backup.txt)
| The patch fails to run checkstyle in hbase-backup |
| +1 :green_heart: | spotbugs | 2m 53s | | the patch passed |
| +1 :green_heart: | hadoopcheck | 10m 57s | | Patch does not cause any
errors with Hadoop 3.3.6 3.4.0. |
| +1 :green_heart: | hbaseprotoc | 1m 3s | | the patch passed |
| +1 :green_heart: | spotless | 0m 42s | | patch has no errors when
running spotless:check. |
_ Other Tests _ |
| +1 :green_heart: | asflicense | 0m 16s | | The patch does not
generate ASF License warnings. |
| | | 36m 10s | | |
| Subsystem | Report/Notes |
|--:|:-|
| Docker | ClientAPI=1.43 ServerAPI=1.43 base:
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/6/artifact/yetus-general-check/output/Dockerfile
|
| GITHUB PR | https://github.com/apache/hbase/pull/7045 |
| Optional Tests | dupname asflicense javac spotbugs checkstyle codespell
detsecrets compile hadoopcheck hbaseanti spotless cc buflint bufcompat
hbaseprotoc |
| uname | Linux cefefc5db9bb 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 tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | HBASE-28957 / e6414821aac8cf3fdbbb0c5bc5ebdd838194f06f |
| Default Java | Eclipse Adoptium-17.0.11+9 |
| Max. process+thread count | 86 (vs. ulimit of 3) |
| modules | C: hbase-protocol-shaded hbase-backup U: . |
| Console output |
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/6/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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Re: [PR] HBASE-28994 Update backup describe command for continuous backup [hbase]
Apache-HBase commented on PR #7045:
URL: https://github.com/apache/hbase/pull/7045#issuecomment-3004441904
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexec | 0m 31s | | Docker mode activated. |
| -0 :warning: | 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 _ |
_ HBASE-28957 Compile Tests _ |
| +0 :ok: | mvndep | 0m 11s | | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 3m 6s | | HBASE-28957 passed |
| +1 :green_heart: | compile | 0m 55s | | HBASE-28957 passed |
| +1 :green_heart: | javadoc | 0m 24s | | HBASE-28957 passed |
| +1 :green_heart: | shadedjars | 6m 4s | | branch has no errors when
building our shaded downstream artifacts. |
| -0 :warning: | patch | 6m 20s | | Used diff version of patch file.
Binary files and potentially other changes not applied. Please rebase and
squash commits if necessary. |
_ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 13s | | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 3m 13s | | the patch passed |
| +1 :green_heart: | compile | 0m 54s | | the patch passed |
| +1 :green_heart: | javac | 0m 54s | | the patch passed |
| +1 :green_heart: | javadoc | 0m 25s | | the patch passed |
| +1 :green_heart: | shadedjars | 6m 9s | | patch has no errors when
building our shaded downstream artifacts. |
_ Other Tests _ |
| +1 :green_heart: | unit | 0m 34s | | hbase-protocol-shaded in the
patch passed. |
| -1 :x: | unit | 23m 12s |
[/patch-unit-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/6/artifact/yetus-jdk17-hadoop3-check/output/patch-unit-hbase-backup.txt)
| hbase-backup in the patch failed. |
| | | 47m 3s | | |
| Subsystem | Report/Notes |
|--:|:-|
| Docker | ClientAPI=1.43 ServerAPI=1.43 base:
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/6/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
|
| GITHUB PR | https://github.com/apache/hbase/pull/7045 |
| Optional Tests | javac javadoc unit compile shadedjars |
| uname | Linux a10554e6c718 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 tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | HBASE-28957 / e6414821aac8cf3fdbbb0c5bc5ebdd838194f06f |
| Default Java | Eclipse Adoptium-17.0.11+9 |
| Test Results |
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/6/testReport/
|
| Max. process+thread count | 3776 (vs. ulimit of 3) |
| modules | C: hbase-protocol-shaded hbase-backup U: . |
| Console output |
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/6/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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Re: [PR] HBASE-28994 Update backup describe command for continuous backup [hbase]
Apache-HBase commented on PR #7045:
URL: https://github.com/apache/hbase/pull/7045#issuecomment-3003531336
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexec | 0m 31s | | Docker mode activated. |
| -0 :warning: | 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 _ |
_ HBASE-28957 Compile Tests _ |
| +0 :ok: | mvndep | 0m 11s | | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 3m 17s | | HBASE-28957 passed |
| +1 :green_heart: | compile | 0m 53s | | HBASE-28957 passed |
| +1 :green_heart: | javadoc | 0m 25s | | HBASE-28957 passed |
| +1 :green_heart: | shadedjars | 6m 11s | | branch has no errors when
building our shaded downstream artifacts. |
| -0 :warning: | patch | 6m 27s | | Used diff version of patch file.
Binary files and potentially other changes not applied. Please rebase and
squash commits if necessary. |
_ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 12s | | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 3m 13s | | the patch passed |
| +1 :green_heart: | compile | 0m 53s | | the patch passed |
| +1 :green_heart: | javac | 0m 53s | | the patch passed |
| +1 :green_heart: | javadoc | 0m 24s | | the patch passed |
| +1 :green_heart: | shadedjars | 6m 9s | | patch has no errors when
building our shaded downstream artifacts. |
_ Other Tests _ |
| +1 :green_heart: | unit | 0m 34s | | hbase-protocol-shaded in the
patch passed. |
| -1 :x: | unit | 23m 13s |
[/patch-unit-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/5/artifact/yetus-jdk17-hadoop3-check/output/patch-unit-hbase-backup.txt)
| hbase-backup in the patch failed. |
| | | 47m 23s | | |
| Subsystem | Report/Notes |
|--:|:-|
| Docker | ClientAPI=1.43 ServerAPI=1.43 base:
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/5/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
|
| GITHUB PR | https://github.com/apache/hbase/pull/7045 |
| Optional Tests | javac javadoc unit compile shadedjars |
| uname | Linux 48da5444fbf1 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 tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | HBASE-28957 / 58a57dce09473ad25d8140683f157daa2496a50d |
| Default Java | Eclipse Adoptium-17.0.11+9 |
| Test Results |
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/5/testReport/
|
| Max. process+thread count | 3780 (vs. ulimit of 3) |
| modules | C: hbase-protocol-shaded hbase-backup U: . |
| Console output |
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Re: [PR] HBASE-28994 Update backup describe command for continuous backup [hbase]
Apache-HBase commented on PR #7045:
URL: https://github.com/apache/hbase/pull/7045#issuecomment-3003493274
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexec | 0m 32s | | Docker mode activated. |
_ Prechecks _ |
| +1 :green_heart: | dupname | 0m 0s | | No case conflicting files
found. |
| +0 :ok: | codespell | 0m 0s | | codespell was not available. |
| +0 :ok: | detsecrets | 0m 0s | | detect-secrets was not available.
|
| +0 :ok: | buf | 0m 0s | | buf was not available. |
| +0 :ok: | buf | 0m 0s | | buf was not available. |
| +1 :green_heart: | @author | 0m 0s | | The patch does not contain
any @author tags. |
| +1 :green_heart: | hbaseanti | 0m 0s | | Patch does not have any
anti-patterns. |
_ HBASE-28957 Compile Tests _ |
| +0 :ok: | mvndep | 0m 12s | | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 2m 50s | | HBASE-28957 passed |
| +1 :green_heart: | compile | 1m 7s | | HBASE-28957 passed |
| -0 :warning: | checkstyle | 0m 8s |
[/buildtool-branch-checkstyle-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/5/artifact/yetus-general-check/output/buildtool-branch-checkstyle-hbase-backup.txt)
| The patch fails to run checkstyle in hbase-backup |
| +1 :green_heart: | spotbugs | 2m 43s | | HBASE-28957 passed |
| +1 :green_heart: | spotless | 0m 42s | | branch has no errors when
running spotless:check. |
| -0 :warning: | patch | 0m 54s | | Used diff version of patch file.
Binary files and potentially other changes not applied. Please rebase and
squash commits if necessary. |
_ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 11s | | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 2m 49s | | the patch passed |
| +1 :green_heart: | compile | 1m 2s | | the patch passed |
| +1 :green_heart: | cc | 1m 2s | | the patch passed |
| -0 :warning: | javac | 0m 28s |
[/results-compile-javac-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/5/artifact/yetus-general-check/output/results-compile-javac-hbase-backup.txt)
| hbase-backup generated 1 new + 116 unchanged - 0 fixed = 117 total (was
116) |
| +1 :green_heart: | blanks | 0m 0s | | The patch has no blanks
issues. |
| -0 :warning: | checkstyle | 0m 9s |
[/buildtool-patch-checkstyle-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/5/artifact/yetus-general-check/output/buildtool-patch-checkstyle-hbase-backup.txt)
| The patch fails to run checkstyle in hbase-backup |
| +1 :green_heart: | spotbugs | 2m 53s | | the patch passed |
| +1 :green_heart: | hadoopcheck | 11m 2s | | Patch does not cause any
errors with Hadoop 3.3.6 3.4.0. |
| +1 :green_heart: | hbaseprotoc | 1m 2s | | the patch passed |
| +1 :green_heart: | spotless | 0m 43s | | patch has no errors when
running spotless:check. |
_ Other Tests _ |
| +1 :green_heart: | asflicense | 0m 17s | | The patch does not
generate ASF License warnings. |
| | | 35m 58s | | |
| Subsystem | Report/Notes |
|--:|:-|
| Docker | ClientAPI=1.43 ServerAPI=1.43 base:
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/5/artifact/yetus-general-check/output/Dockerfile
|
| GITHUB PR | https://github.com/apache/hbase/pull/7045 |
| Optional Tests | dupname asflicense javac spotbugs checkstyle codespell
detsecrets compile hadoopcheck hbaseanti spotless cc buflint bufcompat
hbaseprotoc |
| uname | Linux d942b8a91afd 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 tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | HBASE-28957 / 58a57dce09473ad25d8140683f157daa2496a50d |
| Default Java | Eclipse Adoptium-17.0.11+9 |
| Max. process+thread count | 85 (vs. ulimit of 3) |
| modules | C: hbase-protocol-shaded hbase-backup U: . |
| Console output |
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Re: [PR] HBASE-28994 Update backup describe command for continuous backup [hbase]
Apache-HBase commented on PR #7045:
URL: https://github.com/apache/hbase/pull/7045#issuecomment-3003189534
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexec | 0m 30s | | Docker mode activated. |
| -0 :warning: | 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 _ |
_ HBASE-28957 Compile Tests _ |
| +0 :ok: | mvndep | 0m 32s | | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 3m 28s | | HBASE-28957 passed |
| +1 :green_heart: | compile | 0m 54s | | HBASE-28957 passed |
| +1 :green_heart: | javadoc | 0m 25s | | HBASE-28957 passed |
| +1 :green_heart: | shadedjars | 6m 9s | | branch has no errors when
building our shaded downstream artifacts. |
| -0 :warning: | patch | 6m 26s | | Used diff version of patch file.
Binary files and potentially other changes not applied. Please rebase and
squash commits if necessary. |
_ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 13s | | Maven dependency ordering for patch |
| -1 :x: | mvninstall | 3m 11s |
[/patch-mvninstall-root.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/4/artifact/yetus-jdk17-hadoop3-check/output/patch-mvninstall-root.txt)
| root in the patch failed. |
| -1 :x: | compile | 0m 18s |
[/patch-compile-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/4/artifact/yetus-jdk17-hadoop3-check/output/patch-compile-hbase-backup.txt)
| hbase-backup in the patch failed. |
| -0 :warning: | javac | 0m 18s |
[/patch-compile-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/4/artifact/yetus-jdk17-hadoop3-check/output/patch-compile-hbase-backup.txt)
| hbase-backup in the patch failed. |
| -0 :warning: | javadoc | 0m 14s |
[/results-javadoc-javadoc-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/4/artifact/yetus-jdk17-hadoop3-check/output/results-javadoc-javadoc-hbase-backup.txt)
| hbase-backup generated 3 new + 0 unchanged - 0 fixed = 3 total (was 0) |
| +1 :green_heart: | shadedjars | 6m 5s | | patch has no errors when
building our shaded downstream artifacts. |
_ Other Tests _ |
| +1 :green_heart: | unit | 0m 34s | | hbase-protocol-shaded in the
patch passed. |
| -1 :x: | unit | 0m 18s |
[/patch-unit-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/4/artifact/yetus-jdk17-hadoop3-check/output/patch-unit-hbase-backup.txt)
| hbase-backup in the patch failed. |
| | | 24m 44s | | |
| Subsystem | Report/Notes |
|--:|:-|
| Docker | ClientAPI=1.43 ServerAPI=1.43 base:
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/4/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
|
| GITHUB PR | https://github.com/apache/hbase/pull/7045 |
| Optional Tests | javac javadoc unit compile shadedjars |
| uname | Linux 19edb8192c8d 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 tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | HBASE-28957 / 5cb1a98bb8830fe56fe4d4583feef0fabb6151dd |
| Default Java | Eclipse Adoptium-17.0.11+9 |
| Test Results |
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/4/testReport/
|
| Max. process+thread count | 83 (vs. ulimit of 3) |
| modules | C: hbase-protocol-shaded hbase-backup U: . |
| Console output |
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/4/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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Re: [PR] HBASE-28994 Update backup describe command for continuous backup [hbase]
Apache-HBase commented on PR #7045:
URL: https://github.com/apache/hbase/pull/7045#issuecomment-3003188285
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexec | 0m 34s | | Docker mode activated. |
_ Prechecks _ |
| +1 :green_heart: | dupname | 0m 0s | | No case conflicting files
found. |
| +0 :ok: | codespell | 0m 0s | | codespell was not available. |
| +0 :ok: | detsecrets | 0m 0s | | detect-secrets was not available.
|
| +0 :ok: | buf | 0m 0s | | buf was not available. |
| +0 :ok: | buf | 0m 0s | | buf was not available. |
| +1 :green_heart: | @author | 0m 0s | | The patch does not contain
any @author tags. |
| +1 :green_heart: | hbaseanti | 0m 0s | | Patch does not have any
anti-patterns. |
_ HBASE-28957 Compile Tests _ |
| +0 :ok: | mvndep | 0m 10s | | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 3m 3s | | HBASE-28957 passed |
| +1 :green_heart: | compile | 1m 9s | | HBASE-28957 passed |
| -0 :warning: | checkstyle | 0m 9s |
[/buildtool-branch-checkstyle-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/4/artifact/yetus-general-check/output/buildtool-branch-checkstyle-hbase-backup.txt)
| The patch fails to run checkstyle in hbase-backup |
| +1 :green_heart: | spotbugs | 2m 45s | | HBASE-28957 passed |
| +1 :green_heart: | spotless | 0m 42s | | branch has no errors when
running spotless:check. |
| -0 :warning: | patch | 0m 55s | | Used diff version of patch file.
Binary files and potentially other changes not applied. Please rebase and
squash commits if necessary. |
_ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 11s | | Maven dependency ordering for patch |
| -1 :x: | mvninstall | 2m 51s |
[/patch-mvninstall-root.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/4/artifact/yetus-general-check/output/patch-mvninstall-root.txt)
| root in the patch failed. |
| -1 :x: | compile | 0m 17s |
[/patch-compile-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/4/artifact/yetus-general-check/output/patch-compile-hbase-backup.txt)
| hbase-backup in the patch failed. |
| -1 :x: | cc | 0m 17s |
[/patch-compile-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/4/artifact/yetus-general-check/output/patch-compile-hbase-backup.txt)
| hbase-backup in the patch failed. |
| -0 :warning: | javac | 0m 17s |
[/patch-compile-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/4/artifact/yetus-general-check/output/patch-compile-hbase-backup.txt)
| hbase-backup in the patch failed. |
| +1 :green_heart: | blanks | 0m 0s | | The patch has no blanks
issues. |
| -0 :warning: | checkstyle | 0m 8s |
[/buildtool-patch-checkstyle-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/4/artifact/yetus-general-check/output/buildtool-patch-checkstyle-hbase-backup.txt)
| The patch fails to run checkstyle in hbase-backup |
| -1 :x: | spotbugs | 0m 15s |
[/patch-spotbugs-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/4/artifact/yetus-general-check/output/patch-spotbugs-hbase-backup.txt)
| hbase-backup in the patch failed. |
| -1 :x: | hadoopcheck | 2m 34s | | The patch causes 18 errors with
Hadoop v3.3.6. |
| -1 :x: | hadoopcheck | 5m 10s | | The patch causes 18 errors with
Hadoop v3.4.0. |
| -1 :x: | hbaseprotoc | 0m 16s |
[/patch-hbaseprotoc-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/4/artifact/yetus-general-check/output/patch-hbaseprotoc-hbase-backup.txt)
| hbase-backup in the patch failed. |
| +1 :green_heart: | spotless | 0m 40s | | patch has no errors when
running spotless:check. |
_ Other Tests _ |
| +1 :green_heart: | asflicense | 0m 15s | | The patch does not
generate ASF License warnings. |
| | | 24m 16s | | |
| Subsystem | Report/Notes |
|--:|:-|
| Docker | ClientAPI=1.43 ServerAPI=1.43 base:
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/4/artifact/yetus-general-check/output/Dockerfile
|
| GITHUB PR | https://github.com/apache/hbase/pull/7045 |
| Optional Tests | dupname asflicense javac spotbugs checkstyle codespell
detsecrets compile hadoopcheck hbaseanti spotless cc buflint bufcompat
hbaseprotoc |
| uname | Linux dca4b57c88bd 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 tool | maven |
| Pe
Re: [PR] HBASE-28994 Update backup describe command for continuous backup [hbase]
Apache-HBase commented on PR #7045: URL: https://github.com/apache/hbase/pull/7045#issuecomment-3002814733 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexec | 0m 0s | | Docker mode activated. | | -1 :x: | patch | 0m 7s | | https://github.com/apache/hbase/pull/7045 does not apply to HBASE-28957. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help. | | Subsystem | Report/Notes | |--:|:-| | GITHUB PR | https://github.com/apache/hbase/pull/7045 | | Console output | https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/3/console | | versions | git=2.17.1 | | Powered by | Apache Yetus 0.15.0 https://yetus.apache.org | This message was automatically generated. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-28994 Update backup describe command for continuous backup [hbase]
Apache-HBase commented on PR #7045: URL: https://github.com/apache/hbase/pull/7045#issuecomment-3002814618 :broken_heart: **-1 overall** | Vote | Subsystem | Runtime | Logfile | Comment | |::|--:|:|::|:---:| | +0 :ok: | reexec | 0m 0s | | Docker mode activated. | | -1 :x: | patch | 0m 6s | | https://github.com/apache/hbase/pull/7045 does not apply to HBASE-28957. Rebase required? Wrong Branch? See https://yetus.apache.org/documentation/in-progress/precommit-patchnames for help. | | Subsystem | Report/Notes | |--:|:-| | GITHUB PR | https://github.com/apache/hbase/pull/7045 | | Console output | https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/3/console | | versions | git=2.17.1 | | Powered by | Apache Yetus 0.15.0 https://yetus.apache.org | This message was automatically generated. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-28994 Update backup describe command for continuous backup [hbase]
Copilot commented on code in PR #7045:
URL: https://github.com/apache/hbase/pull/7045#discussion_r2140630273
##
hbase-backup/src/test/java/org/apache/hadoop/hbase/backup/TestBackupDescribe.java:
##
@@ -107,4 +112,50 @@ public void testBackupDescribeCommand() throws Exception {
table.close();
assertTrue(response.indexOf(desc) >= 0);
}
+
+ @Test
+ public void testBackupDescribeCommandForContinuousBackup() throws Exception {
+LOG.info("test backup describe on a single table with data: command-line");
+BackupSystemTable table = new BackupSystemTable(TEST_UTIL.getConnection());
+
+Path root = TEST_UTIL.getDataTestDirOnTestFS();
+Path backupWalDir = new Path(root, "testBackupDescribeCommand");
+FileSystem fs = FileSystem.get(conf1);
+fs.mkdirs(backupWalDir);
+conf1.set(CONF_CONTINUOUS_BACKUP_WAL_DIR, backupWalDir.toString());
+
+String[] backupArgs = new String[] { "create", BackupType.FULL.name(),
BACKUP_ROOT_DIR, "-t",
+ table1.getNameAsString(), "-" + OPTION_ENABLE_CONTINUOUS_BACKUP };
+int ret = ToolRunner.run(conf1, new BackupDriver(), backupArgs);
+assertEquals("Backup should succeed", 0, ret);
+String backupId = table.getBackupHistory().get(0).getBackupId();
+assertTrue(checkSucceeded(backupId));
+LOG.info("backup complete");
+
+BackupInfo info = getBackupAdmin().getBackupInfo(backupId);
+assertTrue(info.getState() == BackupState.COMPLETE);
+
+ByteArrayOutputStream baos = new ByteArrayOutputStream();
+System.setOut(new PrintStream(baos));
+
+String[] args = new String[] { "describe", backupId };
+// Run backup
+ret = ToolRunner.run(conf1, new BackupDriver(), args);
+assertTrue(ret == 0);
+String response = baos.toString();
+assertTrue(response.indexOf(backupId) > 0);
+assertTrue(response.indexOf("COMPLETE") > 0);
+assertTrue(response.indexOf("IsContinuous=true") > 0);
+
+BackupInfo status = table.readBackupInfo(backupId);
+String desc = status.getShortDescription();
+table.close();
+assertTrue(response.indexOf(desc) >= 0);
+
Review Comment:
Capture the original System.out (e.g., PrintStream original = System.out)
and restore it in a finally block or @After method to avoid side effects on
other tests.
```suggestion
PrintStream originalOut = System.out; // Capture the original System.out
System.setOut(new PrintStream(baos));
try {
String[] args = new String[] { "describe", backupId };
// Run backup
ret = ToolRunner.run(conf1, new BackupDriver(), args);
assertTrue(ret == 0);
String response = baos.toString();
assertTrue(response.indexOf(backupId) > 0);
assertTrue(response.indexOf("COMPLETE") > 0);
assertTrue(response.indexOf("IsContinuous=true") > 0);
BackupInfo status = table.readBackupInfo(backupId);
String desc = status.getShortDescription();
table.close();
assertTrue(response.indexOf(desc) >= 0);
} finally {
System.setOut(originalOut); // Restore the original System.out
}
```
##
hbase-backup/src/test/java/org/apache/hadoop/hbase/backup/TestBackupDescribe.java:
##
@@ -107,4 +112,50 @@ public void testBackupDescribeCommand() throws Exception {
table.close();
assertTrue(response.indexOf(desc) >= 0);
}
+
+ @Test
+ public void testBackupDescribeCommandForContinuousBackup() throws Exception {
+LOG.info("test backup describe on a single table with data: command-line");
+BackupSystemTable table = new BackupSystemTable(TEST_UTIL.getConnection());
+
+Path root = TEST_UTIL.getDataTestDirOnTestFS();
+Path backupWalDir = new Path(root, "testBackupDescribeCommand");
+FileSystem fs = FileSystem.get(conf1);
+fs.mkdirs(backupWalDir);
+conf1.set(CONF_CONTINUOUS_BACKUP_WAL_DIR, backupWalDir.toString());
+
+String[] backupArgs = new String[] { "create", BackupType.FULL.name(),
BACKUP_ROOT_DIR, "-t",
+ table1.getNameAsString(), "-" + OPTION_ENABLE_CONTINUOUS_BACKUP };
+int ret = ToolRunner.run(conf1, new BackupDriver(), backupArgs);
+assertEquals("Backup should succeed", 0, ret);
+String backupId = table.getBackupHistory().get(0).getBackupId();
+assertTrue(checkSucceeded(backupId));
+LOG.info("backup complete");
+
+BackupInfo info = getBackupAdmin().getBackupInfo(backupId);
+assertTrue(info.getState() == BackupState.COMPLETE);
+
+ByteArrayOutputStream baos = new ByteArrayOutputStream();
+System.setOut(new PrintStream(baos));
+
+String[] args = new String[] { "describe", backupId };
+// Run backup
+ret = ToolRunner.run(conf1, new BackupDriver(), args);
+assertTrue(ret == 0);
+String response = baos.toString();
+assertTrue(response.indexOf(backupId) > 0);
+assertTrue(response.indexOf("COMPLETE") > 0);
+assertTrue(response.indexOf("IsContinuous=true") > 0);
Review Commen
Re: [PR] HBASE-28994 Update backup describe command for continuous backup [hbase]
NihalJain commented on PR #7045: URL: https://github.com/apache/hbase/pull/7045#issuecomment-2963479724 I see a few issues around resource/connection cleanup/reset etc. in the new test, lets see if copilot catches them. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
Re: [PR] HBASE-28994 Update backup describe command for continuous backup [hbase]
NihalJain commented on code in PR #7045:
URL: https://github.com/apache/hbase/pull/7045#discussion_r2140647823
##
hbase-backup/src/test/java/org/apache/hadoop/hbase/backup/TestBackupDescribe.java:
##
@@ -107,4 +112,50 @@ public void testBackupDescribeCommand() throws Exception {
table.close();
assertTrue(response.indexOf(desc) >= 0);
}
+
+ @Test
+ public void testBackupDescribeCommandForContinuousBackup() throws Exception {
+LOG.info("test backup describe on a single table with data: command-line");
+BackupSystemTable table = new BackupSystemTable(TEST_UTIL.getConnection());
+
+Path root = TEST_UTIL.getDataTestDirOnTestFS();
+Path backupWalDir = new Path(root, "testBackupDescribeCommand");
+FileSystem fs = FileSystem.get(conf1);
+fs.mkdirs(backupWalDir);
+conf1.set(CONF_CONTINUOUS_BACKUP_WAL_DIR, backupWalDir.toString());
+
+String[] backupArgs = new String[] { "create", BackupType.FULL.name(),
BACKUP_ROOT_DIR, "-t",
+ table1.getNameAsString(), "-" + OPTION_ENABLE_CONTINUOUS_BACKUP };
+int ret = ToolRunner.run(conf1, new BackupDriver(), backupArgs);
+assertEquals("Backup should succeed", 0, ret);
+String backupId = table.getBackupHistory().get(0).getBackupId();
Review Comment:
ensure this is emprty before we start the test or maybe take the index + 1
if otherwise
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Re: [PR] HBASE-28994 Update backup describe command for continuous backup [hbase]
NihalJain commented on code in PR #7045:
URL: https://github.com/apache/hbase/pull/7045#discussion_r2140647823
##
hbase-backup/src/test/java/org/apache/hadoop/hbase/backup/TestBackupDescribe.java:
##
@@ -107,4 +112,50 @@ public void testBackupDescribeCommand() throws Exception {
table.close();
assertTrue(response.indexOf(desc) >= 0);
}
+
+ @Test
+ public void testBackupDescribeCommandForContinuousBackup() throws Exception {
+LOG.info("test backup describe on a single table with data: command-line");
+BackupSystemTable table = new BackupSystemTable(TEST_UTIL.getConnection());
+
+Path root = TEST_UTIL.getDataTestDirOnTestFS();
+Path backupWalDir = new Path(root, "testBackupDescribeCommand");
+FileSystem fs = FileSystem.get(conf1);
+fs.mkdirs(backupWalDir);
+conf1.set(CONF_CONTINUOUS_BACKUP_WAL_DIR, backupWalDir.toString());
+
+String[] backupArgs = new String[] { "create", BackupType.FULL.name(),
BACKUP_ROOT_DIR, "-t",
+ table1.getNameAsString(), "-" + OPTION_ENABLE_CONTINUOUS_BACKUP };
+int ret = ToolRunner.run(conf1, new BackupDriver(), backupArgs);
+assertEquals("Backup should succeed", 0, ret);
+String backupId = table.getBackupHistory().get(0).getBackupId();
Review Comment:
ensure this is empty before we start the test or maybe take the index + 1 if
otherwise
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Re: [PR] HBASE-28994 Update backup describe command for continuous backup [hbase]
NihalJain commented on code in PR #7045:
URL: https://github.com/apache/hbase/pull/7045#discussion_r2140634513
##
hbase-backup/src/test/java/org/apache/hadoop/hbase/backup/TestBackupDescribe.java:
##
@@ -107,4 +112,50 @@ public void testBackupDescribeCommand() throws Exception {
table.close();
assertTrue(response.indexOf(desc) >= 0);
}
+
+ @Test
+ public void testBackupDescribeCommandForContinuousBackup() throws Exception {
+LOG.info("test backup describe on a single table with data: command-line");
+BackupSystemTable table = new BackupSystemTable(TEST_UTIL.getConnection());
Review Comment:
nit: try-with-resources
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Re: [PR] HBASE-28994 Update backup describe command for continuous backup [hbase]
NihalJain commented on code in PR #7045:
URL: https://github.com/apache/hbase/pull/7045#discussion_r2140635244
##
hbase-backup/src/test/java/org/apache/hadoop/hbase/backup/TestBackupDescribe.java:
##
@@ -107,4 +112,50 @@ public void testBackupDescribeCommand() throws Exception {
table.close();
assertTrue(response.indexOf(desc) >= 0);
}
+
+ @Test
+ public void testBackupDescribeCommandForContinuousBackup() throws Exception {
+LOG.info("test backup describe on a single table with data: command-line");
+BackupSystemTable table = new BackupSystemTable(TEST_UTIL.getConnection());
+
+Path root = TEST_UTIL.getDataTestDirOnTestFS();
+Path backupWalDir = new Path(root, "testBackupDescribeCommand");
+FileSystem fs = FileSystem.get(conf1);
+fs.mkdirs(backupWalDir);
+conf1.set(CONF_CONTINUOUS_BACKUP_WAL_DIR, backupWalDir.toString());
+
+String[] backupArgs = new String[] { "create", BackupType.FULL.name(),
BACKUP_ROOT_DIR, "-t",
+ table1.getNameAsString(), "-" + OPTION_ENABLE_CONTINUOUS_BACKUP };
+int ret = ToolRunner.run(conf1, new BackupDriver(), backupArgs);
+assertEquals("Backup should succeed", 0, ret);
+String backupId = table.getBackupHistory().get(0).getBackupId();
+assertTrue(checkSucceeded(backupId));
+LOG.info("backup complete");
+
+BackupInfo info = getBackupAdmin().getBackupInfo(backupId);
+assertTrue(info.getState() == BackupState.COMPLETE);
+
+ByteArrayOutputStream baos = new ByteArrayOutputStream();
+System.setOut(new PrintStream(baos));
+
+String[] args = new String[] { "describe", backupId };
+// Run backup
+ret = ToolRunner.run(conf1, new BackupDriver(), args);
+assertTrue(ret == 0);
+String response = baos.toString();
+assertTrue(response.indexOf(backupId) > 0);
+assertTrue(response.indexOf("COMPLETE") > 0);
+assertTrue(response.indexOf("IsContinuous=true") > 0);
+
+BackupInfo status = table.readBackupInfo(backupId);
+String desc = status.getShortDescription();
+table.close();
+assertTrue(response.indexOf(desc) >= 0);
+
+// cleanup
+if (fs.exists(backupWalDir)) {
Review Comment:
put this in finally
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Re: [PR] HBASE-28994 Update backup describe command for continuous backup [hbase]
Apache-HBase commented on PR #7045:
URL: https://github.com/apache/hbase/pull/7045#issuecomment-2943797521
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexec | 0m 45s | | Docker mode activated. |
| -0 :warning: | 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 _ |
_ HBASE-28957 Compile Tests _ |
| +0 :ok: | mvndep | 0m 12s | | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 4m 37s | | HBASE-28957 passed |
| +1 :green_heart: | compile | 1m 10s | | HBASE-28957 passed |
| +1 :green_heart: | javadoc | 0m 29s | | HBASE-28957 passed |
| +1 :green_heart: | shadedjars | 7m 22s | | branch has no errors when
building our shaded downstream artifacts. |
_ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 18s | | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 4m 1s | | the patch passed |
| +1 :green_heart: | compile | 1m 9s | | the patch passed |
| +1 :green_heart: | javac | 1m 9s | | the patch passed |
| +1 :green_heart: | javadoc | 0m 26s | | the patch passed |
| +1 :green_heart: | shadedjars | 6m 43s | | patch has no errors when
building our shaded downstream artifacts. |
_ Other Tests _ |
| +1 :green_heart: | unit | 0m 37s | | hbase-protocol-shaded in the
patch passed. |
| -1 :x: | unit | 22m 12s |
[/patch-unit-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/2/artifact/yetus-jdk17-hadoop3-check/output/patch-unit-hbase-backup.txt)
| hbase-backup in the patch failed. |
| | | 51m 21s | | |
| Subsystem | Report/Notes |
|--:|:-|
| Docker | ClientAPI=1.43 ServerAPI=1.43 base:
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/2/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
|
| GITHUB PR | https://github.com/apache/hbase/pull/7045 |
| Optional Tests | javac javadoc unit compile shadedjars |
| uname | Linux c38d213e6458 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 tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | HBASE-28957 / bb6427835c54f84d2a973c139a2d4345ac987ed3 |
| Default Java | Eclipse Adoptium-17.0.11+9 |
| Test Results |
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/2/testReport/
|
| Max. process+thread count | 3746 (vs. ulimit of 3) |
| modules | C: hbase-protocol-shaded hbase-backup U: . |
| Console output |
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/2/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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Re: [PR] HBASE-28994 Update backup describe command for continuous backup [hbase]
Apache-HBase commented on PR #7045:
URL: https://github.com/apache/hbase/pull/7045#issuecomment-2943788285
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexec | 0m 47s | | Docker mode activated. |
_ Prechecks _ |
| +1 :green_heart: | dupname | 0m 0s | | No case conflicting files
found. |
| +0 :ok: | codespell | 0m 0s | | codespell was not available. |
| +0 :ok: | detsecrets | 0m 0s | | detect-secrets was not available.
|
| +0 :ok: | buf | 0m 0s | | buf was not available. |
| +0 :ok: | buf | 0m 0s | | buf was not available. |
| +1 :green_heart: | @author | 0m 0s | | The patch does not contain
any @author tags. |
| +1 :green_heart: | hbaseanti | 0m 0s | | Patch does not have any
anti-patterns. |
_ HBASE-28957 Compile Tests _ |
| +0 :ok: | mvndep | 0m 11s | | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 4m 39s | | HBASE-28957 passed |
| +1 :green_heart: | compile | 1m 26s | | HBASE-28957 passed |
| -0 :warning: | checkstyle | 0m 13s |
[/buildtool-branch-checkstyle-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/2/artifact/yetus-general-check/output/buildtool-branch-checkstyle-hbase-backup.txt)
| The patch fails to run checkstyle in hbase-backup |
| +1 :green_heart: | spotbugs | 3m 22s | | HBASE-28957 passed |
| +1 :green_heart: | spotless | 0m 52s | | branch has no errors when
running spotless:check. |
_ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 11s | | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 3m 56s | | the patch passed |
| +1 :green_heart: | compile | 1m 34s | | the patch passed |
| +1 :green_heart: | cc | 1m 34s | | the patch passed |
| -0 :warning: | javac | 0m 47s |
[/results-compile-javac-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/2/artifact/yetus-general-check/output/results-compile-javac-hbase-backup.txt)
| hbase-backup generated 1 new + 109 unchanged - 0 fixed = 110 total (was
109) |
| +1 :green_heart: | blanks | 0m 0s | | The patch has no blanks
issues. |
| -0 :warning: | checkstyle | 0m 12s |
[/buildtool-patch-checkstyle-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/2/artifact/yetus-general-check/output/buildtool-patch-checkstyle-hbase-backup.txt)
| The patch fails to run checkstyle in hbase-backup |
| +1 :green_heart: | spotbugs | 3m 48s | | the patch passed |
| +1 :green_heart: | hadoopcheck | 13m 33s | | Patch does not cause any
errors with Hadoop 3.3.6 3.4.0. |
| +1 :green_heart: | hbaseprotoc | 1m 26s | | the patch passed |
| +1 :green_heart: | spotless | 1m 8s | | patch has no errors when
running spotless:check. |
_ Other Tests _ |
| +1 :green_heart: | asflicense | 0m 25s | | The patch does not
generate ASF License warnings. |
| | | 47m 40s | | |
| Subsystem | Report/Notes |
|--:|:-|
| Docker | ClientAPI=1.43 ServerAPI=1.43 base:
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/2/artifact/yetus-general-check/output/Dockerfile
|
| GITHUB PR | https://github.com/apache/hbase/pull/7045 |
| Optional Tests | dupname asflicense javac spotbugs checkstyle codespell
detsecrets compile hadoopcheck hbaseanti spotless cc buflint bufcompat
hbaseprotoc |
| uname | Linux ed7d456559b7 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 tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | HBASE-28957 / bb6427835c54f84d2a973c139a2d4345ac987ed3 |
| Default Java | Eclipse Adoptium-17.0.11+9 |
| Max. process+thread count | 84 (vs. ulimit of 3) |
| modules | C: hbase-protocol-shaded hbase-backup U: . |
| Console output |
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/2/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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Re: [PR] HBASE-28994 Update backup describe command for continuous backup [hbase]
Apache-HBase commented on PR #7045:
URL: https://github.com/apache/hbase/pull/7045#issuecomment-2916031935
:broken_heart: **-1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexec | 8m 23s | | Docker mode activated. |
_ Prechecks _ |
| +1 :green_heart: | dupname | 0m 0s | | No case conflicting files
found. |
| +0 :ok: | codespell | 0m 1s | | codespell was not available. |
| +0 :ok: | detsecrets | 0m 1s | | detect-secrets was not available.
|
| +0 :ok: | buf | 0m 1s | | buf was not available. |
| +0 :ok: | buf | 0m 1s | | buf was not available. |
| +1 :green_heart: | @author | 0m 0s | | The patch does not contain
any @author tags. |
| +1 :green_heart: | hbaseanti | 0m 0s | | Patch does not have any
anti-patterns. |
_ HBASE-28957 Compile Tests _ |
| +0 :ok: | mvndep | 1m 47s | | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 4m 13s | | HBASE-28957 passed |
| +1 :green_heart: | compile | 1m 34s | | HBASE-28957 passed |
| +1 :green_heart: | checkstyle | 0m 21s | | HBASE-28957 passed |
| +1 :green_heart: | spotbugs | 14m 4s | | HBASE-28957 passed |
| -1 :x: | spotless | 0m 56s | | branch has 1 errors when running
spotless:check, run spotless:apply to fix. |
_ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 12s | | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 5m 41s | | the patch passed |
| +1 :green_heart: | compile | 1m 27s | | the patch passed |
| +1 :green_heart: | cc | 1m 27s | | the patch passed |
| -0 :warning: | javac | 0m 32s |
[/results-compile-javac-hbase-backup.txt](https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/1/artifact/yetus-general-check/output/results-compile-javac-hbase-backup.txt)
| hbase-backup generated 1 new + 108 unchanged - 0 fixed = 109 total (was
108) |
| +1 :green_heart: | blanks | 0m 0s | | The patch has no blanks
issues. |
| +1 :green_heart: | checkstyle | 0m 18s | | the patch passed |
| +1 :green_heart: | spotbugs | 13m 21s | | the patch passed |
| +1 :green_heart: | hadoopcheck | 23m 22s | | Patch does not cause any
errors with Hadoop 3.3.6 3.4.0. |
| +1 :green_heart: | hbaseprotoc | 11m 16s | | the patch passed |
| -1 :x: | spotless | 0m 57s | | patch has 1 errors when running
spotless:check, run spotless:apply to fix. |
_ Other Tests _ |
| +1 :green_heart: | asflicense | 0m 21s | | The patch does not
generate ASF License warnings. |
| | | 106m 33s | | |
| Subsystem | Report/Notes |
|--:|:-|
| Docker | ClientAPI=1.43 ServerAPI=1.43 base:
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/1/artifact/yetus-general-check/output/Dockerfile
|
| GITHUB PR | https://github.com/apache/hbase/pull/7045 |
| Optional Tests | dupname asflicense javac spotbugs checkstyle codespell
detsecrets compile hadoopcheck hbaseanti spotless cc buflint bufcompat
hbaseprotoc |
| uname | Linux 06a653b4cf78 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 tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | HBASE-28957 / bbeb82336515d7e9cf1b0889ec6d68b65885fdf2 |
| Default Java | Eclipse Adoptium-17.0.11+9 |
| spotless |
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/1/artifact/yetus-general-check/output/branch-spotless.txt
|
| spotless |
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/1/artifact/yetus-general-check/output/patch-spotless.txt
|
| Max. process+thread count | 85 (vs. ulimit of 3) |
| modules | C: hbase-protocol-shaded hbase-backup U: . |
| Console output |
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/1/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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Re: [PR] HBASE-28994 Update backup describe command for continuous backup [hbase]
Apache-HBase commented on PR #7045:
URL: https://github.com/apache/hbase/pull/7045#issuecomment-2915963668
:confetti_ball: **+1 overall**
| Vote | Subsystem | Runtime | Logfile | Comment |
|::|--:|:|::|:---:|
| +0 :ok: | reexec | 0m 42s | | Docker mode activated. |
| -0 :warning: | 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 _ |
_ HBASE-28957 Compile Tests _ |
| +0 :ok: | mvndep | 0m 13s | | Maven dependency ordering for branch |
| +1 :green_heart: | mvninstall | 4m 3s | | HBASE-28957 passed |
| +1 :green_heart: | compile | 1m 6s | | HBASE-28957 passed |
| +1 :green_heart: | javadoc | 0m 31s | | HBASE-28957 passed |
| +1 :green_heart: | shadedjars | 7m 5s | | branch has no errors when
building our shaded downstream artifacts. |
_ Patch Compile Tests _ |
| +0 :ok: | mvndep | 0m 15s | | Maven dependency ordering for patch |
| +1 :green_heart: | mvninstall | 3m 49s | | the patch passed |
| +1 :green_heart: | compile | 1m 3s | | the patch passed |
| +1 :green_heart: | javac | 1m 3s | | the patch passed |
| +1 :green_heart: | javadoc | 0m 26s | | the patch passed |
| +1 :green_heart: | shadedjars | 7m 5s | | patch has no errors when
building our shaded downstream artifacts. |
_ Other Tests _ |
| +1 :green_heart: | unit | 0m 44s | | hbase-protocol-shaded in the
patch passed. |
| +1 :green_heart: | unit | 22m 7s | | hbase-backup in the patch
passed. |
| | | 50m 34s | | |
| Subsystem | Report/Notes |
|--:|:-|
| Docker | ClientAPI=1.43 ServerAPI=1.43 base:
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/1/artifact/yetus-jdk17-hadoop3-check/output/Dockerfile
|
| GITHUB PR | https://github.com/apache/hbase/pull/7045 |
| Optional Tests | javac javadoc unit compile shadedjars |
| uname | Linux 58ecae442625 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 tool | maven |
| Personality | dev-support/hbase-personality.sh |
| git revision | HBASE-28957 / bbeb82336515d7e9cf1b0889ec6d68b65885fdf2 |
| Default Java | Eclipse Adoptium-17.0.11+9 |
| Test Results |
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/1/testReport/
|
| Max. process+thread count | 3677 (vs. ulimit of 3) |
| modules | C: hbase-protocol-shaded hbase-backup U: . |
| Console output |
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-7045/1/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.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
