[pinot] branch master updated: more context info to debug a flaky test (#8853)

[ja...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

jackie pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pinot.git


The following commit(s) were added to refs/heads/master by this push:
     new 8082838ff2 more context info to debug a flaky test (#8853)
8082838ff2 is described below

commit 8082838ff2720b6fa87a9e57f569c3a2b199ad76
Author: Xiaobing <61...@users.noreply.github.com>
AuthorDate: Tue Jun 7 12:23:28 2022 -0700

    more context info to debug a flaky test (#8853)
---
 .../pinot/integration/tests/OfflineClusterIntegrationTest.java    | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/OfflineClusterIntegrationTest.java b/pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/OfflineClusterIntegrationTest.java
index a07db020e7..bbfa359c15 100644
--- a/pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/OfflineClusterIntegrationTest.java
+++ b/pinot-integration-tests/src/test/java/org/apache/pinot/integration/tests/OfflineClusterIntegrationTest.java
@@ -493,8 +493,8 @@ public class OfflineClusterIntegrationTest extends BaseClusterIntegrationTestSet
     // with only one segment being reloaded with force download and dropping the inverted index.
     long tableSizeAfterReloadSegment = getTableSize(getTableName());
     assertTrue(tableSizeAfterReloadSegment > DISK_SIZE_IN_BYTES && tableSizeAfterReloadSegment < tableSizeWithNewIndex,
-        String.format("Table size: %d should be between %s and %s after dropping inverted index from one segment",
-            tableSizeAfterReloadSegment, DISK_SIZE_IN_BYTES, tableSizeWithNewIndex));
+        String.format("Table size: %d should be between %d and %d after dropping inverted index from segment: %s",
+            tableSizeAfterReloadSegment, DISK_SIZE_IN_BYTES, tableSizeWithNewIndex, segmentName));
 
     // Add inverted index back to check if reloading whole table with force download works.
     // Note that because we have force downloaded a segment above, it's important to reset the table state by adding
@@ -510,8 +510,8 @@ public class OfflineClusterIntegrationTest extends BaseClusterIntegrationTestSet
     // order of entries in the index_map file can change when the raw segment adds/deletes indices back and forth.
     long tableSizeAfterAddIndex = getTableSize(getTableName());
     assertTrue(tableSizeAfterAddIndex > tableSizeAfterReloadSegment, String
-        .format("Table size: %d should increase after adding inverted index, as compared with %s",
-            tableSizeAfterAddIndex, tableSizeAfterReloadSegment));
+        .format("Table size: %d should increase after adding inverted index on segment: %s, as compared with %d",
+            tableSizeAfterAddIndex, segmentName, tableSizeAfterReloadSegment));
 
     // Force to download the whole table and use the original table config, so the disk usage should get back to
     // initial value.


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org