luoyuxia commented on code in PR #2265:
URL: https://github.com/apache/fluss/pull/2265#discussion_r2678287570


##########
fluss-lake/fluss-lake-iceberg/src/test/java/org/apache/fluss/lake/iceberg/maintenance/IcebergRewriteITCase.java:
##########
@@ -186,6 +186,9 @@ void testLogTableCompaction() throws Exception {
                             t1, t1Bucket, ++i, true, 
Collections.singletonList(row(1, "v1"))));
             checkFileStatusInIcebergTable(t1, 3, false);
 
+            // Ensure tiering job has fully processed the previous writes
+            assertReplicaStatus(t1Bucket, i);

Review Comment:
   >  it was possible to request a table from the queue that was no longer 
actually in the "Pending" state. 
   
   @rionmonster 
   Could you please explain a bit about in which case the table in the queue 
that was no longer actually in the "Pending" state.? Although it's no in 
"Pending" state, the tiering service still can requst the table and do the 
tiering. Also, from the log, I can't see the log like "Fail to change state for 
table ".
    
   To me, it looks like that the pendingTieringTables is empty, but I still 
can't figure out why



-- 
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]

Reply via email to