Impala Public Jenkins has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/21094 )

Change subject: IMPALA-12859: Ensure query log estimated memory is initialized
......................................................................

IMPALA-12859: Ensure query log estimated memory is initialized

The query_log_est_total_size_ variable contains the total estimated
size in bytes of the query log list that holds the most recent queries
for use by the Impala http server in the web UI and web API. This
variable was not explicitly initialized and thus would sometimes be
initialized to very large values (on the order of petabytes). Since
this variable is read before being set, the large value prevented any
queries from being placed into the query log since those code saw the
query log as taking too much memory.

The internal-server-test calls the Impala web API to assert that
queries are run successfully. Since no queries were being placed into
the query log, no queries were being returned via the web API. The
internal-server-test ctests were then failing because they could not
assert their queries since the queries could not be found.

Change-Id: I431b50d35fb2db848cfd9cbc1ba32c4aa83931d2
Reviewed-on: http://gerrit.cloudera.org:8080/21094
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
M be/src/service/impala-server.h
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Impala Public Jenkins: Looks good to me, approved; Verified

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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I431b50d35fb2db848cfd9cbc1ba32c4aa83931d2
Gerrit-Change-Number: 21094
Gerrit-PatchSet: 4
Gerrit-Owner: Jason Fehr <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Jason Fehr <[email protected]>
Gerrit-Reviewer: Laszlo Gaal <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>

Reply via email to