Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

HBASE-26855 Delete unnecessary dependency on jaxb-runtime jar #4236

Merged
merged 1 commit into from
Mar 18, 2022
Merged

HBASE-26855 Delete unnecessary dependency on jaxb-runtime jar #4236

merged 1 commit into from
Mar 18, 2022

Conversation

ndimiduk
Copy link
Member

Similar to #4191.

@ndimiduk ndimiduk requested review from Apache9, apurtell and busbey March 16, 2022 16:41
@ndimiduk
Copy link
Member Author

Manual testing in the same way, this seems to work. This time, tested with both Accept: application/json and Accept: text/xml.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 31s Docker mode activated.
-0 ⚠️ yetus 0m 2s Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+1 💚 mvninstall 2m 47s master passed
+1 💚 compile 0m 19s master passed
+1 💚 shadedjars 3m 38s branch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 0m 17s master passed
_ Patch Compile Tests _
+1 💚 mvninstall 2m 35s the patch passed
+1 💚 compile 0m 19s the patch passed
+1 💚 javac 0m 19s the patch passed
+1 💚 shadedjars 4m 10s patch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 0m 16s the patch passed
_ Other Tests _
+1 💚 unit 4m 32s hbase-rest in the patch passed.
20m 23s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4236/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile
GITHUB PR #4236
Optional Tests javac javadoc unit shadedjars compile
uname Linux 5f813896d557 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision master / b1c161e
Default Java AdoptOpenJDK-11.0.10+9
Test Results https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4236/1/testReport/
Max. process+thread count 1385 (vs. ulimit of 30000)
modules C: hbase-rest U: hbase-rest
Console output https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4236/1/console
versions git=2.17.1 maven=3.6.3
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 0m 42s Docker mode activated.
-0 ⚠️ yetus 0m 2s Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck
_ Prechecks _
_ master Compile Tests _
+1 💚 mvninstall 2m 49s master passed
+1 💚 compile 0m 17s master passed
+1 💚 shadedjars 4m 14s branch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 0m 15s master passed
_ Patch Compile Tests _
+1 💚 mvninstall 2m 27s the patch passed
+1 💚 compile 0m 16s the patch passed
+1 💚 javac 0m 16s the patch passed
+1 💚 shadedjars 4m 13s patch has no errors when building our shaded downstream artifacts.
+1 💚 javadoc 0m 12s the patch passed
_ Other Tests _
+1 💚 unit 4m 11s hbase-rest in the patch passed.
20m 40s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4236/1/artifact/yetus-jdk8-hadoop3-check/output/Dockerfile
GITHUB PR #4236
Optional Tests javac javadoc unit shadedjars compile
uname Linux dbb68b85740f 5.4.0-1054-aws #57~18.04.1-Ubuntu SMP Thu Jul 15 03:21:36 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision master / b1c161e
Default Java AdoptOpenJDK-1.8.0_282-b08
Test Results https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4236/1/testReport/
Max. process+thread count 1640 (vs. ulimit of 30000)
modules C: hbase-rest U: hbase-rest
Console output https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4236/1/console
versions git=2.17.1 maven=3.6.3
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@Apache-HBase
Copy link

🎊 +1 overall

Vote Subsystem Runtime Comment
+0 🆗 reexec 1m 38s Docker mode activated.
_ Prechecks _
+1 💚 dupname 0m 0s No case conflicting files found.
+1 💚 @author 0m 0s The patch does not contain any @author tags.
_ master Compile Tests _
+1 💚 mvninstall 5m 37s master passed
+1 💚 compile 1m 21s master passed
_ Patch Compile Tests _
+1 💚 mvninstall 4m 2s the patch passed
+1 💚 compile 0m 41s the patch passed
+1 💚 javac 0m 41s the patch passed
+1 💚 whitespace 0m 0s The patch has no whitespace issues.
+1 💚 xml 0m 1s The patch has no ill-formed XML file.
+1 💚 hadoopcheck 17m 6s Patch does not cause any errors with Hadoop 3.1.2 3.2.2 3.3.1.
_ Other Tests _
+1 💚 asflicense 0m 11s The patch does not generate ASF License warnings.
37m 55s
Subsystem Report/Notes
Docker ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4236/1/artifact/yetus-general-check/output/Dockerfile
GITHUB PR #4236
Optional Tests dupname asflicense javac hadoopcheck xml compile
uname Linux 661395f42068 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality dev-support/hbase-personality.sh
git revision master / b1c161e
Default Java AdoptOpenJDK-1.8.0_282-b08
Max. process+thread count 60 (vs. ulimit of 30000)
modules C: hbase-rest U: hbase-rest
Console output https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4236/1/console
versions git=2.17.1 maven=3.6.3
Powered by Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

@ndimiduk ndimiduk merged commit eaf6c1c into apache:master Mar 18, 2022
@ndimiduk ndimiduk deleted the 26855-delete-dependency-on-jaxb-runtime branch March 18, 2022 15:26
ndimiduk added a commit to ndimiduk/hbase that referenced this pull request Mar 18, 2022
@ndimiduk
HBASE-26855 Delete unnecessary dependency on jaxb-runtime jar (apache…
00105af 
…#4236)

Signed-off-by: Duo Zhang <[email protected]>
ndimiduk added a commit to ndimiduk/hbase that referenced this pull request Mar 18, 2022
@ndimiduk
HBASE-26855 Delete unnecessary dependency on jaxb-runtime jar (apache…
28eaa9c 
…#4236)

Signed-off-by: Duo Zhang <[email protected]>
ndimiduk added a commit that referenced this pull request Mar 18, 2022
@ndimiduk
HBASE-26855 Delete unnecessary dependency on jaxb-runtime jar (#4236)
d09b04d 
Signed-off-by: Duo Zhang <[email protected]>
ndimiduk added a commit that referenced this pull request Mar 18, 2022
@ndimiduk
HBASE-26855 Delete unnecessary dependency on jaxb-runtime jar (#4236)
b11b18b 
Signed-off-by: Duo Zhang <[email protected]>
virajjasani pushed a commit to virajjasani/hbase that referenced this pull request May 17, 2022
@ndimiduk @virajjasani
HBASE-26855 Delete unnecessary dependency on jaxb-runtime jar (apache…
db90620 
…#4236)

Signed-off-by: Duo Zhang <[email protected]>
virajjasani added a commit that referenced this pull request May 18, 2022
@virajjasani
HBASE-26523 + HBASE-25465 + HBASE-26855 backport to branch-2.4 (#4439)
91a44f5 
* HBASE-26523 Upgrade hbase-thirdparty dependency to 4.0.1 (#3988)

Signed-off-by: GeorryHuang <[email protected]>

* HBASE-25465 Use javac --release option for supporting cross version compilation (#4164)

Signed-off-by: Andrew Purtell <[email protected]>

* HBASE-26855 Delete unnecessary dependency on jaxb-runtime jar (#4236)

Signed-off-by: Duo Zhang <[email protected]>

* spotless apply

Co-authored-by: Duo Zhang <[email protected]>
Co-authored-by: Nick Dimiduk <[email protected]>

Signed-off-by: Andrew Purtell <[email protected]>
vinayakphegde pushed a commit to vinayakphegde/hbase that referenced this pull request Apr 4, 2024
@virajjasani
HBASE-26523 + HBASE-25465 + HBASE-26855 backport to branch-2.4 (apach…
483fdd2 
…e#4439)

* HBASE-26523 Upgrade hbase-thirdparty dependency to 4.0.1 (apache#3988)

Signed-off-by: GeorryHuang <[email protected]>

* HBASE-25465 Use javac --release option for supporting cross version compilation (apache#4164)

Signed-off-by: Andrew Purtell <[email protected]>

* HBASE-26855 Delete unnecessary dependency on jaxb-runtime jar (apache#4236)

Signed-off-by: Duo Zhang <[email protected]>

* spotless apply

Co-authored-by: Duo Zhang <[email protected]>
Co-authored-by: Nick Dimiduk <[email protected]>

Signed-off-by: Andrew Purtell <[email protected]>
(cherry picked from commit 91a44f5)
Change-Id: I32d8a4fc4f7da7e24ce82d1d90cc9f22b4238599
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Apache9 Apache9 Apache9 approved these changes

@apurtell apurtell Awaiting requested review from apurtell

@busbey busbey Awaiting requested review from busbey

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ndimiduk @Apache-HBase @Apache9