zstan commented on code in PR #13389:
URL: https://github.com/apache/ignite/pull/13389#discussion_r3644721185
##########
modules/calcite/src/main/java/org/apache/ignite/internal/processors/query/calcite/metadata/cost/IgniteCost.java:
##########
@@ -41,7 +41,7 @@ public class IgniteCost implements RelOptCost {
public static final double AGG_CALL_MEM_COST = 5;
/** Cost of a lookup at the hash. */
- public static final double HASH_LOOKUP_COST = 10;
+ public static final double HASH_LOOKUP_COST = 4;
Review Comment:
'1' greater than ROW_COMPARISON_COST that correlates, in common, with LN
complexity search of hash map, all these empirics need to be fixed during real
statistics refactoring activity and currently changed only for store as more as
possible tests passed and not changed. I think we understand that tests like we
expect **affinity** distribution some kind of fiction for now and coveres just
a little scope of variants.
--
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]
