You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2019/08/21 07:56:50 UTC

[GitHub] [flink] Fokko commented on issue #9492: FLINK-13796: Remove unused variable

Fokko commented on issue #9492: FLINK-13796: Remove unused variable
URL: https://github.com/apache/flink/pull/9492#issuecomment-523342930
 
 
   So, this got caught by my code quality checker while debugging some code. I don't think we want to have lingering variables in the code, which clutter up the codebase, please correct me if I'm wrong.
   
   I've updated the template. I didn't do it in the first place since the change is rather trivial. It is a private variable, so it can't be used anywhere else besides the class.
   
   Sorry for not following the processes so strickly. If you don't agree with the above stated, please feel free to close the PR. However, I can imagine this will be noticed by others as well.

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


With regards,
Apache Git Services