mgaido91 commented on a change in pull request #3607: NIFI-6490 MergeRecord 
supports Variable Registry for MIN_RECORDS and MAX_RECORDS
URL: https://github.com/apache/nifi/pull/3607#discussion_r307804574
 
 

 ##########
 File path: 
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestMergeRecord.java
 ##########
 @@ -486,4 +486,52 @@ public void testDefragmentExpiredBinFailsOnTimeout() 
throws InterruptedException
         runner.assertTransferCount(MergeRecord.REL_ORIGINAL, 0);
         runner.assertTransferCount(MergeRecord.REL_FAILURE, 1);
     }
+
+    @Test
+    public void testMergeWithMinRecordsFromVariableRegistry() {
 
 Review comment:
   one is better, it also reduces the number of tests and hence the running 
time of the CI which is already close to travis limit

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to