jelly-1203 commented on pull request #18017:
URL: https://github.com/apache/flink/pull/18017#issuecomment-989526212


   > Hi, @jelly-1203. You can rename one of the "name" for convenience. You 
needn't to mention your issue here because the GIT can record this change. If 
someone confuses this modifier, he can find your issue. So just modify it 
directly and code review will verify this correctness about this modifier. BTW, 
you may also need to modify the generated plan in UnionTest.xml.
   
   Hi, @xuyangzhong, Thanks for your comment, I will modify it and note the 
match to the logical plan in uniontest.xml


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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to