Bharath Vissapragada has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11230 )

Change subject: IMPALA-7140 fix: fix NDV calculation for partition columns in 
LocalFsTable
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/11230/2//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/11230/2//COMMIT_MSG@7
PS2, Line 7: fix
> hm, by this I meant that this is a fix to IMPALA-7140 (already committed).
Ah, I thought it was a typo. There is no standard way of doing this. People add 
tags like "addendum" to the commit message.  For ex, "IMPALA-7288: Fix Codegen 
Crash in FinalizeModule() (Addendum)", but there is no documented way. I'm ok 
with this.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I0a86750d52dcd744ace030a3c1ec70510d054acf
Gerrit-Change-Number: 11230
Gerrit-PatchSet: 2
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Bharath Vissapragada <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tianyi Wang <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Vuk Ercegovac <[email protected]>
Gerrit-Comment-Date: Thu, 16 Aug 2018 18:23:14 +0000
Gerrit-HasComments: Yes

Reply via email to