gh-yzou commented on code in PR #1817:
URL: https://github.com/apache/polaris/pull/1817#discussion_r2131457168
##########
quarkus/service/src/main/java/org/apache/polaris/service/quarkus/task/QuarkusTaskExecutorImpl.java:
##########
@@ -49,9 +52,11 @@ public QuarkusTaskExecutorImpl(
MetaStoreManagerFactory metaStoreManagerFactory,
TaskFileIOSupplier fileIOSupplier,
Tracer tracer,
- PolarisEventListener polarisEventListener) {
+ PolarisEventListener polarisEventListener,
Review Comment:
```
This is covered by existing tests that produce tasks.
```
Unfortunately I don't think we have such case running in our CI today,
otherwise, it would have caught the problem when we do ConfigurationStore
injection. The only case i know which triggers background task is purge, which
was caught by an aws regression test, which can only run manually today.
I am trying to add a spark integration test here
https://github.com/apache/polaris/pull/1825/files, but runs into following
error when do drop with purge
```
2025-06-05 18:36:21,503 INFO [org.apa.pol.ser.exc.IcebergExceptionMapper]
[,POLARIS] [,,,] (executor-thread-1) Handling runtimeException Principal 'root'
with activated PrincipalRoles '[]' and activated grants via '[service_admin,
catalog_admin]' is not authorized for op DROP_TABLE_WITH_PURGE
```
i am wondering if you know how the catalogAPI privilege is setup with
quarkus test?
--
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
