kgyrtkirk commented on a change in pull request #1271:
URL: https://github.com/apache/hive/pull/1271#discussion_r458866179
##########
File path: standalone-metastore/metastore-server/pom.xml
##########
@@ -204,6 +204,11 @@
<artifactId>hive-storage-api</artifactId>
<version>${storage-api.version}</version>
</dependency>
+ <dependency>
+ <groupId>org.apache.hive</groupId>
+ <artifactId>hive-serde</artifactId>
Review comment:
`TypeInfo` and friends in the metastore...I don't think we want to do it
:)
I don't yet have a concreate idea how to avoid it - because in case we want
to evaluate partition values correctly we would need the same stuff we use on
the ql side...
technically this dependency means that if we want to move out the metastore
- we should also take the serde stuff with 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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org
