Devaspati

Results 10 comments of Devaspati

Needs more work.There seems to be some compat issues with later release of netty and arrow https://github.com/apache/arrow/issues/43491

[mvn_dependency_tree.txt](https://github.com/user-attachments/files/16466175/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...

@zabetak I think hive-exec directly references json-path. https://github.com/apache/hive/blob/81c02a72e88dee8bf28a88b23401e9c3f1fad4a7/ql/pom.xml#L853 But I can try this upgrade in calcite as well.

Have tried removing json-path completely from hive. [DepTree](https://issues.apache.org/jira/secure/attachment/13076824/mvn_dependency_tree1.txt). Now this dependency comes only from Calcite(1.33) at a version of 2.7.0. If pre-commits pass and this is indeed not required then...

@zabetak Let me try that. I was under wrong impression.

[Dep Tree](https://issues.apache.org/jira/secure/attachment/13076915/mvn_dependency_tree5.txt). This now completely removes it from Hive , waiting for pre-commits.

@zabetak I do not think we can completely drop json-path , there are multiple test failures like this "missing com/jayway/jsonpath/spi/mapper/MappingProvider". We can remove all direct references from hive and keep...

@zabetak Any suggestions on which of the above two is the preferred approach ?

Got it . Will do the needful. [Dep-Tree](https://issues.apache.org/jira/secure/attachment/13077052/mvn_dependency_tree_final_4.txt) after dropping json-path from hive.

@zhangbutao Is this PR going to go in , does this affect the upgrade of guava in hive?