virajjasani commented on code in PR #2247:
URL: https://github.com/apache/phoenix/pull/2247#discussion_r2241006726


##########
phoenix-core/src/it/java/org/apache/phoenix/end2end/TableTTLIT.java:
##########
@@ -782,6 +784,130 @@ private void majorCompact(TableName table) throws 
Exception {
     TestUtil.majorCompact(getUtility(), table);
   }
 
+  /**
+   * Test CDC batch mutations for TTL expired rows. This test creates a table 
with TTL and CDC
+   * index, inserts 82 rows (to test batching: 25+25+25+7), lets them expire 
via TTL, and verifies
+   * that all 82 rows have CDC TTL_DELETE events recorded with correct 
pre-image data.
+   */
+  @Test
+  public void testCDCBatchMutationsForTTLExpiredRows() throws Exception {

Review Comment:
   @palashc i have added bunch of logs, do you think we still need metrics?



-- 
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...@phoenix.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to