Csaba Ringhofer has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/21783


Change subject: WIP IMPALA-13374: impala-shell: do not trait missing profile as 
error
......................................................................

WIP IMPALA-13374: impala-shell: do not trait missing profile as error

It is possible after closing a query that getting the profile fails
because it is already evicted from query log. This can lead to sporadic
failures if profile; command is used or show_profiles=True. This is
not an issue in interactive shell usage but can break the workload
with -q or -f.

After the change failing to get the profile leads to logging the error
message (to stdout where the profile would have been logged) instead
of throwing an error.

Also avoids getting the profile after queries if show_profiles is
False (default).

Change-Id: I9453d2c5364590bb4ac886f87bfc8a779eeee172
TODO: add tests after design is accepted
---
M shell/impala_shell.py
1 file changed, 48 insertions(+), 27 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/83/21783/1
--
To view, visit http://gerrit.cloudera.org:8080/21783
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I9453d2c5364590bb4ac886f87bfc8a779eeee172
Gerrit-Change-Number: 21783
Gerrit-PatchSet: 1
Gerrit-Owner: Csaba Ringhofer <[email protected]>

Reply via email to