Dan Hecht has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8611 )

Change subject: IMPALA-6225: Query profile date-time strings should have ns 
precision.
......................................................................


Patch Set 3:

> (2 comments)
 >
 > > (1 comment)
 > >
 > > Please add a test as Phil suggested. If we decide to add 64-bit
 > > timestamps to the profile, please file a JIRA for it.
 >
 > Alas, I haven't yet found a quick way to add the test. How does one
 > get a query id from a query handle/result (returned by
 > execute_query_async/execute_query)? If I know the query id, that
 > can be passed to read_profile_page from where we can extract the
 > fields of interest.

./experiments/test_process_failures.py and ./webserver/test_web_pages.py looks 
like they have examples for beeswax and operation_id_to_query_id() looks like 
for HS2.


--
To view, visit http://gerrit.cloudera.org:8080/8611
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I2e124b9c7e0717b8dc2cdab46aea41d74c5f2fd0
Gerrit-Change-Number: 8611
Gerrit-PatchSet: 3
Gerrit-Owner: Zoram Thanga <zo...@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dhe...@cloudera.com>
Gerrit-Reviewer: Michael Ho <k...@cloudera.com>
Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com>
Gerrit-Reviewer: Zoram Thanga <zo...@cloudera.com>
Gerrit-Comment-Date: Tue, 21 Nov 2017 17:37:17 +0000
Gerrit-HasComments: No

Reply via email to