poojanilangekar commented on code in PR #1925:
URL: https://github.com/apache/polaris/pull/1925#discussion_r2183931995
##########
runtime/service/src/test/java/org/apache/polaris/service/quarkus/admin/PolarisAdminServiceAuthzTest.java:
##########
@@ -151,7 +152,8 @@ public void testCreateCatalogSufficientPrivileges() {
@Test
public void testCreateCatalogInsufficientPrivileges() {
- final CatalogEntity newCatalog = new
CatalogEntity.Builder().setName("new_catalog").build();
+ final CatalogEntity newCatalog =
+ new
CatalogEntity.Builder().setName("new_catalog").setCatalogType("INTERNAL").build();
Review Comment:
Since I opened #1998, I reverted this and other similar changes. The build
still passes but I think we should try to get 1998 merged before this change.
--
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]
