>From Ali Alsuliman <[email protected]>:

Attention is currently required from: Murtadha Hubail, Preetham Poluparthi, 
Shahrzad Shirazi.

Ali Alsuliman has posted comments on this change by Shahrzad Shirazi. ( 
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20962?usp=email )

Change subject: [ASTERIXDB-3719][COMP] Using index when redundant range 
predicates are present
......................................................................


Patch Set 13:

(3 comments)

Patchset:

PS13:
We need to handle cases like:
```
CREATE INDEX idx1 ON ds1(a);
SELECT * FROM ds1 WHERE a < 3 and a > 5;
SELECT * FROM ds1 WHERE a = 3 and a > 5;
SELECT * FROM ds1 WHERE a > 3 and a = 5;
SELECT * FROM ds1 WHERE a >= 3 and a = 3;
SELECT * FROM ds1 WHERE a >= 3 and a > 3;
SELECT * FROM ds1 WHERE a > 3 and a > 3;
```


File 
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/am/BTreeAccessMethod.java:

https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20962/comment/bd223336_76fb8918?usp=email
 :
PS13, Line 1214:             return Integer.compare(tag1.serialize(), 
tag2.serialize());
We can just bail out for this case for now.


https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20962/comment/d870f308_fa4af6b8?usp=email
 :
PS13, Line 1225:             ILogicalBinaryComparator.Result result = 
comparator.compare(ref1, ref2);
For now, let's use: compare(IAObject leftConstant, IAObject rightConstant). It 
is for numbers only.



--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/20962?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://asterix-gerrit.ics.uci.edu/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: Ifc39ad34b8bf551d21567963b8507c6961eae90d
Gerrit-Change-Number: 20962
Gerrit-PatchSet: 13
Gerrit-Owner: Shahrzad Shirazi <[email protected]>
Gerrit-Reviewer: Ali Alsuliman <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Preetham Poluparthi <[email protected]>
Gerrit-Reviewer: Shahrzad Shirazi <[email protected]>
Gerrit-CC: Murtadha Hubail <[email protected]>
Gerrit-Attention: Murtadha Hubail <[email protected]>
Gerrit-Attention: Shahrzad Shirazi <[email protected]>
Gerrit-Attention: Preetham Poluparthi <[email protected]>
Gerrit-Comment-Date: Wed, 22 Apr 2026 16:27:29 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No

Reply via email to