impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Impala Public Jenkins (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-4710: There is an error in control audit log file size number
Date Wed, 28 Dec 2016 17:58:05 GMT
Impala Public Jenkins has submitted this change and it was merged.

Change subject: IMPALA-4710: There is an error in control audit log file size number
......................................................................


IMPALA-4710: There is an error in control audit log file size number

The comparison logic of log file size and maximum config is wrong,
leading to log file size is greater than the maximum config.

This patch corrects the error, makes both are consistent.

Change-Id: I1808d69359102d63b732469b440c9c7675a371b7
Reviewed-on: http://gerrit.cloudera.org:8080/5582
Reviewed-by: Tim Armstrong <tarmstrong@cloudera.com>
Tested-by: Impala Public Jenkins
---
M be/src/util/simple-logger.cc
1 file changed, 1 insertion(+), 1 deletion(-)

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



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I1808d69359102d63b732469b440c9c7675a371b7
Gerrit-PatchSet: 3
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Donghui Xu <xu.donghui@zte.com.cn>
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: Tim Armstrong <tarmstrong@cloudera.com>

Mime
View raw message