impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bikramjeet Vig (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-4276: Profile displays non-default query options set by planner
Date Fri, 14 Jul 2017 18:32:34 GMT
Bikramjeet Vig has posted comments on this change.

Change subject: IMPALA-4276: Profile displays non-default query options set by planner

Patch Set 1:

Commit Message:

PS1, Line 10: expilictly
> explicitly
Removing this, according to the comment in, this is not a bug

PS1, Line 10: expilictly set
            : default query options were also populated in the runtime profile.
> can you clarify what this means - what's an explicitly set default query op
same as above
File be/src/service/

PS1, Line 151: exec_request
> nit: use exec_request_
File be/src/service/

PS1, Line 84: 
> can you quickly explain why this is right? It looks to me like we would wan
Thanks for pointing that out. I was under the impression that all values had a default value
set during initialization when I looked at ImpalaInternalService_types.h
I now realize that some values dont have a default value set in the thrift file and this is
indicated by __isset.NAME.
Hence reverting this.
File tests/query_test/

PS1, Line 112: query = "set mem_limit = 0"
> can you change this to some other  query option, then test that MEM_LIMIT=8

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I08e9dc2bebb83101976bbbd903ee48c5068dbaab
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Bikramjeet Vig <>
Gerrit-Reviewer: Bikramjeet Vig <>
Gerrit-Reviewer: Henry Robinson <>
Gerrit-HasComments: Yes

View raw message