[GitHub] [hive] deniskuzZ commented on pull request #3784: HIVE-26484: Add proto3 hivemetastore.proto

["deniskuzZ (via GitHub)" <gi...@apache.org>]

deniskuzZ commented on PR #3784:
URL: https://github.com/apache/hive/pull/3784#issuecomment-1466295962

   > @cnauroth @deniskuzZ, one question: Any specific reason why guava version was not upgraded in standalone-metastore/pom.xml and storage-api/pom.xml? It is still 19.0!
   
   PR mentions that proto3/grpc won't compile with guava < 22, however, looks like standalane-metastore's root pom overrides it with v19. Not sure how it compiled then in the first place, maybe a version upgrade wasn't required.
   ````
   mvn dependency:tree | grep guava
   [INFO] +- com.google.guava:guava:jar:19.0:compile
   ````


-- 
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: gitbox-unsubscribe@hive.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscribe@hive.apache.org
For additional commands, e-mail: gitbox-help@hive.apache.org