Fredy Wijaya has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13399 )

Change subject: IMPALA-8564: Add table/view create time in the lineage graph
......................................................................


Patch Set 17: Code-Review+2

> Patch Set 15: Verified-1
>
> Build failed: https://jenkins.impala.io/job/gerrit-verify-dryrun/4356/

Fixed a subtle bug in the lineage unit test where Vertex instances in the set 
of vertices and the Vertex instances in the set of Edges are different 
instances.

In the previous patch, I got lucky with my local testing due to the timestamps 
in lineage.test were exactly the same as the ones created in the metastore. I 
blew up the metastore and re-created the tables and was able to reproduce the 
issue and fixed it.


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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: If4f578d7b299a76c30323b10a883ba32f8713d82
Gerrit-Change-Number: 13399
Gerrit-PatchSet: 17
Gerrit-Owner: Fredy Wijaya <fwij...@cloudera.com>
Gerrit-Reviewer: Anonymous Coward (498)
Gerrit-Reviewer: Fredy Wijaya <fwij...@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-Comment-Date: Fri, 31 May 2019 21:20:47 +0000
Gerrit-HasComments: No

Reply via email to