Skip to content
This repository was archived by the owner on May 20, 2021. It is now read-only.

Add detailed stage-level query progress info to frontend.#168

Open
qwwqwwq wants to merge 15 commits intoairbnb:masterfrom
qwwqwwq:jq_airpal_query_monitoring
Open

Add detailed stage-level query progress info to frontend.#168
qwwqwwq wants to merge 15 commits intoairbnb:masterfrom
qwwqwwq:jq_airpal_query_monitoring

Conversation

@qwwqwwq
Copy link

@qwwqwwq qwwqwwq commented Mar 7, 2016

This PR wires through the detailed prestodb stage-level query progress info into the frontend. This is the same info that is shown typographically when you use presto-cli to run a query, i.e.:

Query 20131105_005539_00082_ee7y3, RUNNING, 13 nodes, 187 splits                
0:04 [97.7M rows, 14.3GB] [27.9M rows/s,  4.1GB/s] [==============>>>>>>>>] 61% 

     STAGES   ROWS  ROWS/s  BYTES  BYTES/s  QUEUED    RUN   DONE                
0.........R  2.43K   1.03K  49.9K    21.3K       0      1      0                
  1.......R  83.9M   35.8M  12.2G    5.22G       0     88     97                
    2.....F     25       0  2.19K       0B       0      0      1 

Now, clicking on the progress bar of a running query from the RunsTable will bring up a modal that looks like this:

screen shot 2016-03-06 at 9 26 07 pm

I wasn't really sure where this could fit in the page so I figured a modal was the safest bet for now.

I didn't see any javascript tests in the project currently, but if the maintainers have opinions about which testing frameworks to use I'd be happy to add tests for this feature.

This also required some backend changes but those were more minimal, they basically amount to adding a new field to the Job object.

qwwqwwq added 15 commits March 6, 2016 21:14
Signed-off-by: qwwqwwq <jeffreyquinn@Jeffreys-MacBook-Air.local>
Signed-off-by: qwwqwwq <jeffreyquinn@Jeffreys-MacBook-Air.local>
Signed-off-by: qwwqwwq <jeffreyquinn@Jeffreys-MacBook-Air.local>
inp
Signed-off-by: qwwqwwq <jeffreyquinn@Jeffreys-MacBook-Air.local>
Signed-off-by: qwwqwwq <jeffreyquinn@Jeffreys-MacBook-Air.local>
Signed-off-by: qwwqwwq <jeffreyquinn@Jeffreys-MacBook-Air.local>
Signed-off-by: qwwqwwq <jeffreyquinn@Jeffreys-MacBook-Air.local>
Signed-off-by: qwwqwwq <jeffreyquinn@Jeffreys-MacBook-Air.local>
Signed-off-by: qwwqwwq <jeffreyquinn@Jeffreys-MacBook-Air.local>
Signed-off-by: qwwqwwq <jeffreyquinn@Jeffreys-MacBook-Air.local>
Signed-off-by: qwwqwwq <jeffreyquinn@Jeffreys-MacBook-Air.local>
Signed-off-by: qwwqwwq <jeffreyquinn@Jeffreys-MacBook-Air.local>
Signed-off-by: qwwqwwq <jeffreyquinn@Jeffreys-MacBook-Air.local>
Signed-off-by: qwwqwwq <jeffreyquinn@Jeffreys-MacBook-Air.local>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant