adutra commented on code in PR #4816:
URL: https://github.com/apache/polaris/pull/4816#discussion_r3493719734
##########
polaris-core/src/main/java/org/apache/polaris/core/rest/PolarisResourcePaths.java:
##########
@@ -48,6 +48,12 @@ public class PolarisResourcePaths {
"/polaris/v1/{prefix}/namespaces/{namespace}/policies/{policy-name}/mappings";
public static final String V1_APPLICABLE_POLICIES =
"/polaris/v1/{prefix}/applicable-policies";
+ // Semantic Model endpoints
+ public static final String V1_SEMANTIC_MODELS =
+ "/polaris/v1/{prefix}/namespaces/{namespace}/semantic-models";
+ public static final String V1_SEMANTIC_MODEL =
+
"/polaris/v1/{prefix}/namespaces/{namespace}/semantic-models/{semantic-model-name}";
Review Comment:
I can rework later this to introduce dynamic endpoint registration.
It is true though that this class specifically is becoming a bag of
loosely-coupled constants.
--
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]
