[GitHub] [phoenix-connectors] chrajeshbabu commented on pull request #73: PHOENIX-6683 Surround the OR filters with parentheses while convertin…

[GitBox <gi...@apache.org>]

chrajeshbabu commented on PR #73:
URL: https://github.com/apache/phoenix-connectors/pull/73#issuecomment-1090085221

   > I haven't looked into the logic closely. Do we process the parentheses correctly ?
   > 
   > i.e. Make sure that we do not turn
   > 
   > (a and b or c) into (a and (b or c))
   
   Yes this is not happening and AND is getting precedence. Just added another test case to ensure that..
   
   45336 [main] DEBUG org.apache.phoenix.execute.BaseQueryPlan  - Iterator for table N000001 ready: resultIterators=ResultIterators [name=PARALLEL,id=5337088f-cf25-3568-ad24-630b5fd8c07f,scans=[[{"loadColumnFamiliesOnDemand":true,"filter":"**((\"ENTITY_INSTANCE_ID\" = 40 AND \"CASE_MATCH_TYPE\" = 'M') OR NOT (\"CANCELLATION_FLAG\" = 'Y'))"**,"startRow":"","stopRow":"","batch":-1,"cacheBlocks":true,"totalColumns":3,"maxResultSize":-1,"families":{"0":["\\x00\\x00\\x00\\x00","\\x80\\x0B","\\x80\\x0C"]},"caching":2147483647,"maxVersions":1,"timeRange":[0,9223372036854775807]}]]], index=0]
   


-- 
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: issues-unsubscribe@phoenix.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org