yannizhou05 opened a new pull request, #6342: URL: https://github.com/apache/nifi/pull/6342
The test of "testSendData" in TestPrometheusRecordSink.java is flaky because of HashMaps. The orders of elements in the HashMaps are not the same every time being called. In this case, LinkedHashMap is applied to replace HashMaps in order to ensure the orders are not changing. Also, the MapRecord cannot keep the same orders of row1 and row2 as well. To deal with this situation, two possible combinations of row1 and row2 are applied in the test cases to avoid possible failures due to orders. It matters to fix this flaky test as it may causes potential issues due to the uncertainty of orders from Maps rather than the program itself. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
