Devaspati
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?