Fix flaky TestEventListener#testOutputStats

peakUserMemoryBytes is derived from peakUserMemoryReservation in QueryStats, not revocable memory reservation.
trinodb · Jun 8, 2020 · 6347897 · 6347897
1 parent 5fcfc93
commit 6347897
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/presto-tests/src/test/java/io/prestosql/execution/TestEventListener.java b/presto-tests/src/test/java/io/prestosql/execution/TestEventListener.java
@@ -376,7 +376,7 @@ public void testOutputStats()
         assertEquals(statistics.getResourceWaitingTime().get().toMillis(), queryStats.getResourceWaitingTime().toMillis());
         assertEquals(statistics.getAnalysisTime().get().toMillis(), queryStats.getAnalysisTime().toMillis());
         assertEquals(statistics.getExecutionTime().get().toMillis(), queryStats.getExecutionTime().toMillis());
-        assertEquals(statistics.getPeakUserMemoryBytes(), queryStats.getPeakRevocableMemoryReservation().toBytes());
+        assertEquals(statistics.getPeakUserMemoryBytes(), queryStats.getPeakUserMemoryReservation().toBytes());
         assertEquals(statistics.getPeakTotalNonRevocableMemoryBytes(), queryStats.getPeakTaskUserMemory().toBytes());
         assertEquals(statistics.getPeakTaskTotalMemory(), queryStats.getPeakTaskUserMemory().toBytes());
         assertEquals(statistics.getPhysicalInputBytes(), queryStats.getPhysicalInputDataSize().toBytes());