[ 
https://issues.apache.org/jira/browse/PHOENIX-6409?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17301430#comment-17301430
 ] 

Viraj Jasani commented on PHOENIX-6409:
---------------------------------------

[~larsh] I agree with you reg testing the explain plan as is, which shows up 
when we run EXPLAIN query. However, this new Explain API was created as part of 
PHOENIX-5265 to reduce hard coded String comparisons so that whenever even a 
small value changes, we won't have to make changes in 100+ tests as part of the 
same commit, which at times might be good to ship early but fixing tests would 
become a headache :)

But again, not all tests are converted to use new API, we have mixture of both 
ways of testing Explain plans.

> Include local index uncovered columns merge in explain plan.
> ------------------------------------------------------------
>
>                 Key: PHOENIX-6409
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-6409
>             Project: Phoenix
>          Issue Type: Improvement
>            Reporter: Lars Hofhansl
>            Assignee: Lars Hofhansl
>            Priority: Minor
>             Fix For: 5.1.1, 4.16.1, 4.17.0, 5.2.0
>
>
> This is a nice to have.
> It would be nice if EXPLAIN would show that uncovered columns have to be 
> merged during the index scan.
> (Not sure that is easily doable - filing it here for the record.)



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to