Riza Suminto has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22643 )

Change subject: IMPALA-13877: Fix missing stats time in TestLastDdlTimeUpdate
......................................................................


Patch Set 3: Code-Review+1

(1 comment)

http://gerrit.cloudera.org:8080/#/c/22643/3//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/22643/3//COMMIT_MSG@9
PS3, Line 9: This patch fixes broken tests after IMPALA-11597 that only show up 
in exhaustive exploration.
           :
           : Expanded TestHelper to able to check set/unset operations, in 
addition to changed/unchaged. Updated tests accordingly.
nit: paragraph it so that each line of commit message has 72 chars at most.

https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=65147115#ContributingtoImpala-Fix



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I99e37b9168c2c79a913ca0c717cad1ed9253100a
Gerrit-Change-Number: 22643
Gerrit-PatchSet: 3
Gerrit-Owner: Mihaly Szjatinya <msz...@pm.me>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Mihaly Szjatinya <msz...@pm.me>
Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com>
Gerrit-Comment-Date: Wed, 19 Mar 2025 20:35:25 +0000
Gerrit-HasComments: Yes

Reply via email to