ShreelekhyaG opened a new pull request #4107:
URL: https://github.com/apache/carbondata/pull/4107


    ### Why is this PR needed?
    Query with SI after add partition based on location on partition table 
gives incorrect results. 
   While pruning, if it's an external segment, it should use 
`ExternalSegmentResolver `, and no need to use `ImplicitIncludeFilterExecutor 
`as an external segment is not added in the SI table.
   
    ### What changes were proposed in this PR?
   Based on `isRelative `path, set `isExternalSegment `value for partition 
segment.
   In `getBlockId `method, when segment id is not present, setting from block 
name.
       
    ### Does this PR introduce any user interface change?
    - No
   
    ### Is any new testcase added?
    - Yes
   
       
   


----------------------------------------------------------------
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.

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


Reply via email to