rubenada commented on a change in pull request #2245:
URL: https://github.com/apache/calcite/pull/2245#discussion_r518615633
##########
File path: site/_docs/algebra.md
##########
@@ -37,7 +37,7 @@ planner engine generates an alternative expression that has
the same
semantics as the original but a lower cost.
The planning process is extensible. You can add your own relational
-operators, planner rules, cost model, and statistics.
+operators, planner rules, cost models, and statistics.
Review comment:
minor remark: I think the original one (in singular) was arguably more
accurate, since the cost model is a single one in the whole process (as
described in the previous paragraph). So, I'd lean towards leaving this
sentence as it was.
----------------------------------------------------------------
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]
