zentol commented on issue #9758: [FLINK-14185][tests]Move unstable assertion 
statement out of close method of QS test server.
URL: https://github.com/apache/flink/pull/9758#issuecomment-539548505
 
 
   I think we should to both; use different state as I see nothing in the test 
that requires the stats to be shared, and having separate stats also seems 
close to how things would work in production (if this thing would be used at 
all), and the assertion in close is super odd due to the applied cast.
   

----------------------------------------------------------------
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