Github user arina-ielchiieva commented on a diff in the pull request:
https://github.com/apache/drill/pull/1051#discussion_r153456718
--- Diff:
exec/java-exec/src/main/java/org/apache/drill/exec/work/foreman/Foreman.java ---
@@ -256,12 +239,14 @@ public void run() {
// track how long the query takes
queryManager.markStartTime();
- enqueuedQueries.dec();
runningQueries.inc();
+ queryText = queryRequest.getPlan();
+ recordNewState(QueryState.PLANNING);
--- End diff --
`getPlan` is simple getter and cannot throw en exception. It was moved up
to display query string on Web UI when we have entered PLANNING and other
states rather then having empty string, please see screenshot in Jira.
---
