Lars Volker has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8411 )

Change subject: IMPALA-6124: Fix alter table ddl updates and test
......................................................................


Patch Set 3:

> (1 comment)
 >
 > The patch looks good to me, but I'm still curious how it worked
 > before. Tried digging into the HMS code to see how the ddlTime is
 > updated, but no clues.

Thank you for having a look. It didn't work before. The test was broken and 
didn't catch this. I found it after fixing the test. :)


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I3126252e7709304d3e1fa4bb06a0b847180bd6cf
Gerrit-Change-Number: 8411
Gerrit-PatchSet: 3
Gerrit-Owner: Lars Volker <[email protected]>
Gerrit-Reviewer: Alex Behm <[email protected]>
Gerrit-Reviewer: Bharath Vissapragada <[email protected]>
Gerrit-Reviewer: Lars Volker <[email protected]>
Gerrit-Comment-Date: Mon, 30 Oct 2017 19:10:14 +0000
Gerrit-HasComments: No

Reply via email to