henrib commented on code in PR #4194:
URL: https://github.com/apache/hive/pull/4194#discussion_r1174272169
##########
standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/HMSHandler.java:
##########
@@ -7535,6 +7523,38 @@ 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 TException {
+ PropertyStore propertyStore = getMS().getPropertyStore();
+ PropertyManager mgr = PropertyManager.create(ns, propertyStore);
+ return mgr;
+ }
+ @Override
+ public PropertyGetResponse get_properties(PropertyGetRequest req) throws
TException {
+ PropertyManager mgr = getPropertyManager(req.getNameSpace());
+ Map<String, PropertyMap> selected =
mgr.selectProperties(req.getMapPrefix(), req.getMapPredicate(),
req.getMapSelection());
Review Comment:
I don't see how; the store, the dirty maps logic, the JEXL engine are
members of that instance. I must be missing the point.
--
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]
