phoenix-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "James Taylor (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (PHOENIX-3822) Surface byte and row estimates in a machine readable way when doing EXPLAIN PLAN
Date Thu, 18 May 2017 18:12:04 GMT

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

James Taylor commented on PHOENIX-3822:
---------------------------------------

Looking good, [~samarthjain]. Here's some feedback:
* Add the new methods to StatementPlan instead of QueryPlan and get rid of this cast. It's
useful for DELETE and UPSERT SELECT queries to be able to do an explain and know how many
bytes are scanned. It'll be pretty straightforward to implement - you'll essentially delegate
to the QueryPlan built for these cases and for UPSERT VALUES and DELETE over a complete row
key, and other cases like TRACE, etc. you'll return 0. 
* How about we use slightly more descriptive names for the columns, like EST_BYTES_READ and
EST_ROWS_READ?

> Surface byte and row estimates in a machine readable way when doing EXPLAIN PLAN
> --------------------------------------------------------------------------------
>
>                 Key: PHOENIX-3822
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-3822
>             Project: Phoenix
>          Issue Type: Improvement
>            Reporter: Samarth Jain
>            Assignee: Samarth Jain
>         Attachments: PHOENIX-3822.patch, PHOENIX-3822_v2.patch
>
>




--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Mime
View raw message