eric-maynard commented on code in PR #1665:
URL: https://github.com/apache/polaris/pull/1665#discussion_r2109828459
##########
integration-tests/src/main/java/org/apache/polaris/service/it/test/PolarisPolicyServiceIntegrationTest.java:
##########
@@ -100,6 +96,11 @@ public class PolarisPolicyServiceIntegrationTest {
private static final String CATALOG_ROLE_1 = "catalogrole1";
private static final String CATALOG_ROLE_2 = "catalogrole2";
private static final String EXAMPLE_TABLE_MAINTENANCE_POLICY_CONTENT =
"{\"enable\":true}";
+ private static final String INVALID_NAMESPACE = "INVALID_NAMESPACE";
+ private static final String INVALID_TABLE = "INVALID_TABLE";
+ private static final String INVALID_POLICY = "INVALID_POLICY";
+ private static final String INVALID_NAMESPACE_MSG =
+ String.format("Namespace does not exist: %s", INVALID_NAMESPACE);
Review Comment:
It might be good to shift these message constants below rather than being in
the middle of the entity names
--
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: issues-unsubscr...@polaris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
