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

Vladimir Ozerov commented on IGNITE-10347:
------------------------------------------

[~jooger], my comments:
# This view expose only local running queries. There will be more such views 
with local-only info, along with views, which collect info from the whole 
cluster. Hence, we need a prefix to distinguish local from non-local stuff
# From what I understand, this view is concernted only about SQL queries, so 
this should be reflected in the name as well. Taking this and previous points 
in count, I would name the view {{LOCAL_SQL_RUNNING_QUERIES}} or 
{LOCAL_SQL_QUERIES}}
# Do we really need {{NODE_ID}} for local view? IMO it is not necessary
# Query ID should not be manipulated anyhow inside the view. Instead, correct 
string representation of query ID should already be prepared inside indexing.
# Not sure if we need any special mechanics for duration ranges. H2 already 
will do all necessary stuff for us, so why bother? Note that 
{{GridQueryProcessor#runningQueries}} method with {{long}} parameter was 
created long ago and for completely different purpose - to report long-running 
queries. No need to rely on it, we are free to create our own overload which 
will return all running queries
# Why do we need to perform any HEX manipulations on query ID parts? It doesn't 
add any clarity to user, neither seem to add any other value.
# Please confirm that non-SQL queries are filtered out in the view. I cannot 
find relevant code.

> Expose system SQL view for running queries
> ------------------------------------------
>
>                 Key: IGNITE-10347
>                 URL: https://issues.apache.org/jira/browse/IGNITE-10347
>             Project: Ignite
>          Issue Type: Task
>            Reporter: Yury Gerzhedovich
>            Assignee: Yury Gerzhedovich
>            Priority: Major
>              Labels: iep-29, sql
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> Need to  expose system SQL view to provide list of running queries. Proposed 
> name is *running_queries* with following columns: query_id, node_id, sql, 
> schema_name, duration.
> Where,
> query_id - cluster unique id of query with format \{node_order_id}
> {X}
> {node_qry_cntr}, both node_order_id and node_qry_cntr encoded to HEX. X - 'X' 
> letter used as separator.
> node_id - id of node where query was started
> sql - sql command
> schema_name - SQL schema name
> duration - time in ms from start of execution of query
>  
> The view should contains all kind of running queries from RunningQueryManager 
> on local node



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to