dengzhhu653 commented on code in PR #4194:
URL: https://github.com/apache/hive/pull/4194#discussion_r1187276410
##########
standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HMSHandler.java:
##########
@@ -7536,9 +7524,41 @@ public List<Partition> get_partitions_by_names(final
String dbName, final String
return ret;
}
+ /**
+ * Creates an instance of property manager based on the (declared) namespace.
+ * @param ns the namespace
+ * @return the manager instance
+ * @throws TException
+ */
+ private PropertyManager getPropertyManager(String ns) throws MetaException,
NoSuchObjectException {
+ PropertyStore propertyStore = getMS().getPropertyStore();
+ PropertyManager mgr = PropertyManager.create(ns, propertyStore);
+ return mgr;
+ }
+ @Override
+ public PropertyGetResponse get_properties(PropertyGetRequest req) throws
MetaException, NoSuchObjectException, TException {
+ PropertyManager mgr = getPropertyManager(req.getNameSpace());
+ Map<String, PropertyMap> selected =
mgr.selectProperties(req.getMapPrefix(), req.getMapPredicate(),
req.getMapSelection());
Review Comment:
Should we convert the `PropertyException`, `JexlException` or others to
`MetaException` here so that the client can tell the real cause?
--
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]
