[GitHub] [flink] GJL commented on a change in pull request #11783: [FLINK-17181][runtime] Drop generic Types in SchedulingTopology Interface

[GitBox]

GJL commented on a change in pull request #11783: [FLINK-17181][runtime] Drop 
generic Types in SchedulingTopology Interface
URL: https://github.com/apache/flink/pull/11783#discussion_r410077414
 
 

 ##########
 File path: 
flink-runtime/src/main/java/org/apache/flink/runtime/executiongraph/failover/flip1/RestartPipelinedRegionFailoverStrategy.java
 ##########
 @@ -98,7 +98,7 @@ public RestartPipelinedRegionFailoverStrategy(
        // 
------------------------------------------------------------------------
 
        private void buildFailoverRegions() {
-               final Set<? extends Set<? extends SchedulingExecutionVertex>> 
distinctRegions =
+               final Set<? extends Set<SchedulingExecutionVertex>> 
distinctRegions =
 
 Review comment:
   Fixed. The remaining wildcards look legitimate to me
   
   <img width="1296" alt="image" 
src="https://user-images.githubusercontent.com/1681921/79549620-864f9180-8097-11ea-97d8-1f3b7a45307e.png";>
   
   
   

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services