XJDKC commented on code in PR #1624:
URL: https://github.com/apache/polaris/pull/1624#discussion_r2096982670
##########
service/common/src/test/java/org/apache/polaris/service/catalog/io/FileIOFactoryTest.java:
##########
@@ -198,7 +198,7 @@ public void testLoadFileIOForCleanupTask() {
// 1. BasePolarisCatalog:doCommit: for writing the table during the
creation
// 2. BasePolarisCatalog:doRefresh: for reading the table during the drop
// 3. TaskFileIOSupplier:apply: for clean up metadata files and merge files
- Mockito.verify(testServices.fileIOFactory(), Mockito.times(3))
Review Comment:
nit: We need to update the comment to reflect the change. The first method
call now comes from initializing the default catalog file IO.
--
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]
