[GitHub] tillrohrmann commented on issue #7546: [FLINK-11390][tests] Port YARNSessionCapacitySchedulerITCase#testTaskManagerFailure to new code base

2019-01-25 Thread GitBox 
tillrohrmann commented on issue #7546: [FLINK-11390][tests] Port 
YARNSessionCapacitySchedulerITCase#testTaskManagerFailure to new code base
URL: https://github.com/apache/flink/pull/7546#issuecomment-457629435
 
 
   Hmm `YARNSessionCapacitySchedulerITCase` actually uses `assumeTrue`. Not 
sure what Travis thought when compiling this class.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] tillrohrmann commented on issue #7546: [FLINK-11390][tests] Port YARNSessionCapacitySchedulerITCase#testTaskManagerFailure to new code base

2019-01-25 Thread GitBox 
tillrohrmann commented on issue #7546: [FLINK-11390][tests] Port 
YARNSessionCapacitySchedulerITCase#testTaskManagerFailure to new code base
URL: https://github.com/apache/flink/pull/7546#issuecomment-457626068
 
 
   PR does not build because of 
   ```
   
src/test/java/org/apache/flink/yarn/YARNSessionCapacitySchedulerITCase.java:[94,15]
 (imports) UnusedImports: Unused import: org.junit.Assume.assumeTrue.
   ```


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

[GitHub] tillrohrmann commented on issue #7546: [FLINK-11390][tests] Port YARNSessionCapacitySchedulerITCase#testTaskManagerFailure to new code base

2019-01-23 Thread GitBox 
tillrohrmann commented on issue #7546: [FLINK-11390][tests] Port 
YARNSessionCapacitySchedulerITCase#testTaskManagerFailure to new code base
URL: https://github.com/apache/flink/pull/7546#issuecomment-456836215
 
 
   As @zentol pointed out, it could work to use the metrics to query the number 
of restarts.


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services