Github user tison1 commented on the issue:
https://github.com/apache/flink/pull/6347
> vertices is the correct plural, but this is another one of those cases
where fixing it might cause more harm than good since it could cause merge
conflicts, yet provides no functional benefit.
Github user tison1 commented on the issue:
https://github.com/apache/flink/pull/6347
> vertices is the correct plural, but this is another one of those cases
where fixing it might cause more harm than good since it could cause merge
conflicts, yet provides no functional benefit.
Github user zentol commented on the issue:
https://github.com/apache/flink/pull/6347
yes
---
Github user tison1 commented on the issue:
https://github.com/apache/flink/pull/6347
> Additionally this PR makes a lot of whitespace changes that should be
reverted in any case.
did you mean the whitespace in comment `* ` is significant?
---
Github user zentol commented on the issue:
https://github.com/apache/flink/pull/6347
`vertices` is the correct plural, but this is another one of those cases
where fixing it might cause more harm than good since it could cause merge
conflicts, yet provides no functional benefit.
Github user yanghua commented on the issue:
https://github.com/apache/flink/pull/6347
+1 from myside, it depends on @zentol or @tillrohrmann 's opinion.
---
Github user yanghua commented on the issue:
https://github.com/apache/flink/pull/6347
it seems Flink also uses "vertices", I agree your opinion.
---
