zstan commented on code in PR #2101:
URL: https://github.com/apache/ignite-3/pull/2101#discussion_r1206897245
##########
modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/rel/AbstractIndexScan.java:
##########
@@ -111,33 +112,44 @@ public RelNode accept(RexShuttle shuttle) {
return super.accept(shuttle);
}
- /**
- * Get index name.
- * TODO Documentation https://issues.apache.org/jira/browse/IGNITE-15859
- */
+ /** Return index name. */
public String indexName() {
return idxName;
}
+ /** Index type. */
+ public Type indexType() {
+ return type;
+ }
+
/** {@inheritDoc} */
@Override
public RelOptCost computeSelfCost(RelOptPlanner planner, RelMetadataQuery
mq) {
double rows = table.getRowCount();
- double cost;
+ double cost = 0;
+
+ if (type == Type.HASH) {
+ boolean notExact = (searchBounds() == null)
+ || (searchBounds().stream().anyMatch(bound -> bound.type()
!= SearchBounds.Type.EXACT));
Review Comment:
as i can see from java doc : **Multiple values or multiple ranges**, thus
seems this is not applicable at common.
--
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]
