zstan commented on code in PR #13389:
URL: https://github.com/apache/ignite/pull/13389#discussion_r3644683337
##########
modules/calcite/src/main/java/org/apache/ignite/internal/processors/query/calcite/rel/set/IgniteSetOp.java:
##########
@@ -67,7 +67,7 @@ public default RelOptCost computeSetOpCost(RelOptPlanner
planner, RelMetadataQue
for (RelNode input : getInputs())
inputRows += mq.getRowCount(input);
- double mem = 0.5 * inputRows * aggregateFieldsCount() *
IgniteCost.AVERAGE_FIELD_SIZE;
+ double mem = 0.7 * inputRows * aggregateFieldsCount() *
IgniteCost.AVERAGE_FIELD_SIZE;
Review Comment:
empiric constants need to be fixed cause row count estimation is changes for
now and some tests (2-3) are fails, all empirics need to be reevaluated during
statistics activity changes.
--
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]
