Aitozi commented on issue #8280: [FLINK-12297]Harden ClosureCleaner to handle 
the wrapped function
URL: https://github.com/apache/flink/pull/8280#issuecomment-501073568
 
 
   Hi @aljoscha  The case you post can not past is due to the check below, 
because a closure can be hidden in any class as you say.
   
   ```
   (fo.getClass().isAnonymousClass()) ||
   (fo.getClass().isLocalClass()) ||
   (fo.getClass().isMemberClass())
   ```
   
   But I can not simply remove the check, because it will run into the 
StackOverFlowError, So i check the serializability before recursively clean the 
member field.  What do you think of the solution now? The travis failed seems 
unrelated. 

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