afedulov commented on a change in pull request #9694: [hotfix] [docs] Fixes in 
glossar
URL: https://github.com/apache/flink/pull/9694#discussion_r326088658
 
 

 ##########
 File path: docs/concepts/glossary.md
 ##########
 @@ -79,7 +79,7 @@ whole [Flink Master](#flink-master) was called JobManager.
 #### Logical Graph
 
 A logical graph is a directed graph describing the high-level logic of a 
stream processing program.
-The nodes are [Operators](#operator) and the edges indicate 
input/output-relationships or
 
 Review comment:
   I am pretty sure it is a copy-paste typo from the Physical Graph where the 
wrong preposition got removed along with "partitions".
   Physical graph:
   "The nodes are Tasks and the edges indicate input/output-relationships 
**or** partitions **of** data streams or data sets." 
   
   Also the construct "this OR that OR that" sounds awkward and people usually 
write "this, that OR that" when listing more than two items.

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