You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "liuxiaocs7 (via GitHub)" <gi...@apache.org> on 2026/03/01 09:19:12 UTC

[PR] HBASE-29942 [hbase-thirdparty] Bump protobuf java to 4.34.0 [hbase-thirdparty]

liuxiaocs7 opened a new pull request, #154:
URL: https://github.com/apache/hbase-thirdparty/pull/154

   - see: HBASE-29942


-- 
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: issues-unsubscribe@hbase.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] HBASE-29942 [hbase-thirdparty] Bump protobuf java to 4.34.0 [hbase-thirdparty]

Posted by "NihalJain (via GitHub)" <gi...@apache.org>.
NihalJain commented on PR #154:
URL: https://github.com/apache/hbase-thirdparty/pull/154#issuecomment-3988754229

   Hey @liuxiaocs7 thanks for attempting this, upgrading to protobuf is a bit tricky. You may want to look at https://github.com/apache/hbase-thirdparty?tab=readme-ov-file#steps-to-generate-patches-for-protobuf-java to know the trick, in case you are not already aware.


-- 
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: issues-unsubscribe@hbase.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] HBASE-29942 [hbase-thirdparty] Bump protobuf java to 4.34.0 [hbase-thirdparty]

Posted by "liuxiaocs7 (via GitHub)" <gi...@apache.org>.
liuxiaocs7 closed pull request #154: HBASE-29942 [hbase-thirdparty] Bump protobuf java to 4.34.0
URL: https://github.com/apache/hbase-thirdparty/pull/154


-- 
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: issues-unsubscribe@hbase.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] HBASE-29942 [hbase-thirdparty] Bump protobuf java to 4.34.0 [hbase-thirdparty]

Posted by "liuxiaocs7 (via GitHub)" <gi...@apache.org>.
liuxiaocs7 commented on PR #154:
URL: https://github.com/apache/hbase-thirdparty/pull/154#issuecomment-3979694871

   need to dig more


-- 
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: issues-unsubscribe@hbase.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] HBASE-29942 [hbase-thirdparty] Bump protobuf java to 4.34.0 [hbase-thirdparty]

Posted by "Apache-HBase (via GitHub)" <gi...@apache.org>.
Apache-HBase commented on PR #154:
URL: https://github.com/apache/hbase-thirdparty/pull/154#issuecomment-3979577428

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |:----:|----------:|--------:|:--------:|:-------:|
   | +0 :ok: |  reexec  |   1m  6s |  |  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.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain any @author tags.  |
   | -0 :warning: |  test4tests  |   0m  0s |  |  The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch.  |
   |||| _ master Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   0m 56s |  |  master passed  |
   | +1 :green_heart: |  compile  |   0m 13s |  |  master passed  |
   | +1 :green_heart: |  javadoc  |   0m  7s |  |  master passed  |
   |||| _ Patch Compile Tests _ |
   | -1 :x: |  mvninstall  |   0m 29s | [/patch-mvninstall-root.txt](https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-154/1/artifact/yetus-precommit-check/output/patch-mvninstall-root.txt) |  root in the patch failed.  |
   | -1 :x: |  compile  |   0m  9s | [/patch-compile-root.txt](https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-154/1/artifact/yetus-precommit-check/output/patch-compile-root.txt) |  root in the patch failed.  |
   | -1 :x: |  javac  |   0m  9s | [/patch-compile-root.txt](https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-154/1/artifact/yetus-precommit-check/output/patch-compile-root.txt) |  root in the patch failed.  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks issues.  |
   | +1 :green_heart: |  xmllint  |   0m  0s |  |  No new issues.  |
   | +1 :green_heart: |  javadoc  |   0m  6s |  |  the patch passed  |
   |||| _ Other Tests _ |
   | -1 :x: |  unit  |   0m 39s | [/patch-unit-root.txt](https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-154/1/artifact/yetus-precommit-check/output/patch-unit-root.txt) |  root in the patch failed.  |
   | +1 :green_heart: |  asflicense  |   0m  6s |  |  The patch does not generate ASF License warnings.  |
   |  |   |   3m 59s |  |  |
   
   
   | Subsystem | Report/Notes |
   |----------:|:-------------|
   | Docker | ClientAPI=1.53 ServerAPI=1.53 base: https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-154/1/artifact/yetus-precommit-check/output/Dockerfile |
   | GITHUB PR | https://github.com/apache/hbase-thirdparty/pull/154 |
   | Optional Tests | dupname asflicense javac javadoc unit codespell detsecrets xmllint compile |
   | uname | Linux ee6ec1d66f77 6.14.0-1018-aws #18~24.04.1-Ubuntu SMP Mon Nov 24 19:46:27 UTC 2025 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | /home/jenkins/jenkins-home/workspace/Base-Thirdparty-PreCommit_PR-154/yetus-precommit-check/src/.yetus/personality.sh |
   | git revision | master / 22d0494e9fa87a7f706cfd60d4038935452844b8 |
   | Default Java | Eclipse Adoptium-17.0.11+9 |
   |  Test Results | https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-154/1/testReport/ |
   | Max. process+thread count | 413 (vs. ulimit of 2000) |
   | modules | C: . U: . |
   | Console output | https://ci-hbase.apache.org/job/HBase-Thirdparty-PreCommit/job/PR-154/1/console |
   | versions | git=2.34.1 maven=called xmllint=20913 |
   | 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: issues-unsubscribe@hbase.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org