impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bharath Vissapragada (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-3552: Make incremental stats max serialized size configurable
Date Tue, 08 Nov 2016 06:22:41 GMT
Bharath Vissapragada has posted comments on this change.

Change subject: IMPALA-3552: Make incremental stats max serialized size configurable

Patch Set 7:

File be/src/catalog/

Line 151: Status Catalog::GetCatalogConfigBytes(JNIEnv* jni_env, jbyteArray* cfg_bytes) const
> Can we refactor the code to only have one GetBackendConfigBytes() or are th
It was created this way since Fe and Catalog can have their own set of configs. For example
 --load_catalog_in_background is specific to Catalog. Maintaining a GetBackendConfigBytes()
sounds like a good idea but its unclear where to put that. May be a static method in a util
class somewhere where we DEFINE all the gflags?
File be/src/catalog/catalog.h:

Line 106:   Status GetCatalogConfigBytes(JNIEnv* jni_env, jbyteArray* cfg_bytes) const;
> GetBackendConfigBytes()?
Same comment as in frontend.h.
File common/thrift/Types.thrift:

Line 249:   6: optional bool compute_lineage
> Let's continue down this path for now and see how it goes.

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: I33684725a61eabc67237503e61178305d37d3cb5
Gerrit-PatchSet: 7
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Yonghyun Hwang <>
Gerrit-Reviewer: Alex Behm <>
Gerrit-Reviewer: Bharath Vissapragada <>
Gerrit-Reviewer: Dimitris Tsirogiannis <>
Gerrit-Reviewer: Huaisi Xu <>
Gerrit-Reviewer: Yonghyun Hwang
Gerrit-Reviewer: Yonghyun Hwang <>
Gerrit-HasComments: Yes

View raw message