You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@phoenix.apache.org by "James Taylor (JIRA)" <ji...@apache.org> on 2015/04/05 23:16:34 UTC
[jira] [Resolved] (PHOENIX-1809) Improve explain plan
[ https://issues.apache.org/jira/browse/PHOENIX-1809?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
James Taylor resolved PHOENIX-1809.
-----------------------------------
Resolution: Fixed
Fix Version/s: 4.4.0
5.0.0
Assignee: James Taylor
Thanks for the review, [~maryannxue]. Pushed to 4.x and master.
> Improve explain plan
> --------------------
>
> Key: PHOENIX-1809
> URL: https://issues.apache.org/jira/browse/PHOENIX-1809
> Project: Phoenix
> Issue Type: Bug
> Reporter: James Taylor
> Assignee: James Taylor
> Fix For: 5.0.0, 4.4.0
>
> Attachments: PHOENIX-1809.patch, PHOENIX-1809_v2.patch
>
>
> Two changes:
> 1) We already output the limit information, so it's redundant to display the PageFilter filter. Instead we should skip that filter in ExplainTable when we generate the explain plan - there are about 5 places in the code that need to be changed that expect this in the explain plan.
> 2) Instead of the hacky boolean we pass through MergeSortTopNResultIterator to control the output of the server-side order by/limit, we should output that information in ExplainTable as well. We'd need to pass through the LIMIT information there too. Then the MergeSortTopNResultIterator would only output the CLIENT MERGE SORT step.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)