TisonKun edited a comment on issue #7245: [FLINK-11069] [utils] Remove 
FutureUtil
URL: https://github.com/apache/flink/pull/7245#issuecomment-445107203
 
 
   Thanks @yanghua.
   I notice the inconsistent among `util.function`. With 
1e6a91a3bf7e734eccdb034ce505b3775b709265 @StephanEwen mark the functional 
interfaces in `util.function` as `@Public`, but the following interfaces were 
not marked. Such a `unchecked` static method is proposed by @GJL.
   I prefer the idea to make the interfaces final, and thus think it could be a 
follow up to refactor the static methods into `FunctionUtils`. Alternatively, 
for cohesion, add these static methods into the corresponding interfaces and 
regards the currently `@Public` ones are accidentally not final in the 
beginning.

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

Reply via email to