TisonKun commented on issue #9492: FLINK-13796: Remove unused variable URL: https://github.com/apache/flink/pull/9492#issuecomment-523339316 Hi @xintongsong I agree with your concerns on the contribution workflow. We can move the discussion back to JIRA and leave the pull request as is since our current strategy of unassigned pull request, as said by @flinkbot above, is > This pull request references an unassigned Jira ticket. According to the code contribution guide, tickets need to be assigned before starting with the implementation work. Also I encourage @Fokko follow our workflow in #9493 . Besides, IMO it is a valid contribution is the variable is effectively unused. Let's move this discussion back to JIRA FLINK-13796.
---------------------------------------------------------------- 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
