> On May 31, 2019, 8:16 p.m., Sarath Subramanian wrote: > > addons/models/1000-Hadoop/1090-impala_model.json > > Lines 193 (patched) > > <https://reviews.apache.org/r/70759/diff/3/?file=2147407#file2147407line193> > > > > we don't need to define a new impala_db_ddl and impala_table_ddl, since > > impala ddls are hive ddls. We can add an attribute in hive_db_ddl and > > hive_table_ddl to specify the application/service which issued the DDL > > (Hive or Impala). Please review.
removed - Na ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/70759/#review215623 ----------------------------------------------------------- On May 31, 2019, 4:02 a.m., Na Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/70759/ > ----------------------------------------------------------- > > (Updated May 31, 2019, 4:02 a.m.) > > > Review request for atlas, madhan and Sarath Subramanian. > > > Repository: atlas > > > Description > ------- > > ATLAS-3197 introduced a new feature on Hive integration. It introduced new > model called Hive_table_ddl and hive_db_ddl which are used to track commands > that are executed either on a table entity or bd entity. DDL entities will be > added to a table/db entity through relationshipAttribute call HIVE_DB_TO_DDL > and hive_tb_to_ddl. Every time when a new command is executed, a new ddl > entity will be created, it will append to the existing ddlQueries field if a > table/db has already been created. Once a table/db is deleted, all these > relationship attributes will be deleted as well. > > We need to support this new feature on Impala integration. > > > Diffs > ----- > > > addons/impala-bridge/src/main/java/org/apache/atlas/impala/hook/events/BaseImpalaEvent.java > d241b6a > > addons/impala-bridge/src/main/java/org/apache/atlas/impala/hook/events/CreateImpalaProcess.java > b7506a4 > > addons/impala-bridge/src/main/java/org/apache/atlas/impala/model/ImpalaDataType.java > 10ce448 > > addons/impala-bridge/src/test/java/org/apache/atlas/impala/ImpalaLineageITBase.java > b8cbf6b > > addons/impala-bridge/src/test/java/org/apache/atlas/impala/ImpalaLineageToolIT.java > 7f9a534 > > addons/impala-bridge/src/test/java/org/apache/atlas/impala/hook/ImpalaLineageHookIT.java > 6156208 > addons/models/0000-Area0/0010-base_model.json d14b227 > addons/models/1000-Hadoop/1030-hive_model.json 417c4c5 > addons/models/1000-Hadoop/1090-impala_model.json 24eb5db > > > Diff: https://reviews.apache.org/r/70759/diff/3/ > > > Testing > ------- > > add check for ddl entities in integration tests > > > Thanks, > > Na Li > >