hive icon indicating copy to clipboard operation
hive copied to clipboard

HIVE-28360: Upgrade jersey to 1.19.4

Open lvyankui opened this issue 1 year ago • 5 comments

What changes were proposed in this pull request?

Upgrade jersey to 1.19.4

Why are the changes needed?

HIVE-28360 After upgrading to Hadoop 3.3.5, the Hive WebHCat server fails to start because of inconsistent versions of the Jersey JAR package. Hive HCat lacks the jersey-server-1.19 jar.

Does this PR introduce any user-facing change?

None, that I am aware of.

How was this patch tested?

CI

lvyankui avatar Jul 03 '24 12:07 lvyankui

mvn_dependency_tree.txt Hi @lvyankui This is the dependency tree with this change and it would be a bit cleaner practice to exclude 1.19 versions which are shown in dependency tree as a part of org.apache.tez:tez-api:jar:0.10.3 and org.apache.atlas:atlas-client-common:jar:2.3.0.Otherwise this change looks good.

devaspatikrishnatri avatar Aug 02 '24 06:08 devaspatikrishnatri

mvn_dependency_tree.txt Hi @lvyankui This is the dependency tree with this change and it would be a bit cleaner practice to exclude 1.19 versions which are shown in dependency tree as a part of org.apache.tez:tez-api:jar:0.10.3 and org.apache.atlas:atlas-client-common:jar:2.3.0.Otherwise this change looks good.

@devaspatikrishnatri Thank you for your feedback. I have already taken care of it

lvyankui avatar Aug 05 '24 03:08 lvyankui

@lvyankui have you verified the hcatalog ok with the latest change? seems like the https://github.com/apache/hive/blob/master/common/pom.xml#L214 still has direct reference to the old jersey

dengzhhu653 avatar Aug 12 '24 08:08 dengzhhu653

@lvyankui have you verified the hcatalog ok with the latest change? seems like the https://github.com/apache/hive/blob/master/common/pom.xml#L214 still has direct reference to the old jersey

@dengzhhu653 I have processed all the references. Please help review again, thanks!

lvyankui avatar Aug 14 '24 08:08 lvyankui