guiyanakuang edited a comment on pull request #930: URL: https://github.com/apache/orc/pull/930#issuecomment-935312527
> Thank you for making a PR, @guiyanakuang . BTW, this looks like a revert of [ORC-807](https://issues.apache.org/jira/browse/ORC-807) (from @belugabehr ) > > So, is this the current status, @guiyanakuang ? > > > In fact, the bench module does not directly depend on jackson, only spark indirectly depends on the specified version of jackson. Yes, because the code that depends on jackson has been rewritten with gson in ORC-946. I'm sure this is the current state. > Comment before update: which reminds me that we don't even need to declare Spark's dependency on jackson, there is no conflict. > I immediately made a new commit to remove the explicit dependency of spark on jackson. update: I was wrong, actually spark and avro depend on different versions of jackson in orc-benchmarks-spark, explicit dependencies are better than letting maven choose based on the length of the dependency path, I think it's better to keep it. -- 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: dev-unsubscr...@orc.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org