illiabarbashov-sketch commented on code in PR #6748:
URL: https://github.com/apache/hive/pull/6748#discussion_r3949166400
##########
beeline/pom.xml:
##########
@@ -70,6 +65,7 @@
<groupId>org.apache.hive</groupId>
<artifactId>hive-standalone-metastore-server</artifactId>
<version>${standalone-metastore.version}</version>
+ <scope>provided</scope>
Review Comment:
Not exactly. It's needed for SchemaTool. And that's is the part of open
questions. Do we want native beeline installer to contain the SchemaTool in it?
It can't be used anyways but it's the part of beeline module as of now. So I
suggested to remove that dependency from compile time if the SchemaTool
distribution remains the same and native beeline can go on its own.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
