You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@flink.apache.org by tr...@apache.org on 2017/09/28 08:59:49 UTC
[3/4] flink git commit: [FLINK-7514][tests] fix
BackPressureStatsTrackerITCase releasing buffers twice
[FLINK-7514][tests] fix BackPressureStatsTrackerITCase releasing buffers twice
This closes #4591.
Project: http://git-wip-us.apache.org/repos/asf/flink/repo
Commit: http://git-wip-us.apache.org/repos/asf/flink/commit/37156043
Tree: http://git-wip-us.apache.org/repos/asf/flink/tree/37156043
Diff: http://git-wip-us.apache.org/repos/asf/flink/diff/37156043
Branch: refs/heads/master
Commit: 37156043687a05519f42ca6f02265803d3b03761
Parents: dcc35f2
Author: Nico Kruber <ni...@data-artisans.com>
Authored: Fri Aug 25 09:49:44 2017 +0200
Committer: Till Rohrmann <tr...@apache.org>
Committed: Thu Sep 28 09:31:17 2017 +0200
----------------------------------------------------------------------
.../legacy/backpressure/BackPressureStatsTrackerITCase.java | 5 +----
1 file changed, 1 insertion(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/flink/blob/37156043/flink-runtime/src/test/java/org/apache/flink/runtime/rest/handler/legacy/backpressure/BackPressureStatsTrackerITCase.java
----------------------------------------------------------------------
diff --git a/flink-runtime/src/test/java/org/apache/flink/runtime/rest/handler/legacy/backpressure/BackPressureStatsTrackerITCase.java b/flink-runtime/src/test/java/org/apache/flink/runtime/rest/handler/legacy/backpressure/BackPressureStatsTrackerITCase.java
index e2289f0..1f28d3d 100644
--- a/flink-runtime/src/test/java/org/apache/flink/runtime/rest/handler/legacy/backpressure/BackPressureStatsTrackerITCase.java
+++ b/flink-runtime/src/test/java/org/apache/flink/runtime/rest/handler/legacy/backpressure/BackPressureStatsTrackerITCase.java
@@ -219,6 +219,7 @@ public class BackPressureStatsTrackerITCase extends TestLogger {
//
for (Buffer buf : buffers) {
buf.recycle();
+ assertTrue(buf.isRecycled());
}
// Wait for all buffers to be available. The tasks
@@ -277,10 +278,6 @@ public class BackPressureStatsTrackerITCase extends TestLogger {
highAvailabilityServices.closeAndCleanupAllData();
- for (Buffer buf : buffers) {
- buf.recycle();
- }
-
testBufferPool.lazyDestroy();
}
}};