----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52077/#review149804 -----------------------------------------------------------
addons/hive-bridge/src/main/java/org/apache/atlas/hive/bridge/ColumnLineageUtils.java (line 73) <https://reviews.apache.org/r/52077/#comment217525> Don't add toString() to arguments addons/hive-bridge/src/main/java/org/apache/atlas/hive/hook/HiveHook.java (line 634) <https://reviews.apache.org/r/52077/#comment217532> Why have you changed Entity update request to entity create request? It should be update addons/hive-bridge/src/main/java/org/apache/atlas/hive/hook/HiveHook.java (line 819) <https://reviews.apache.org/r/52077/#comment217533> e.key() is the output column name? e.key doesn't contain cluster name, so might get de-duped across other lineage processes. Qualified name for lineage process should be 'qualified name of hive command process : column name'. addons/hive-bridge/src/main/java/org/apache/atlas/hive/hook/HiveHook.java (line 1044) <https://reviews.apache.org/r/52077/#comment217522> Use syntax: LOG.debug("Column Lineage Map - {}", this.lineageInfo.entrySet()); so that toString() is not evaluated if debug log level is not enabled addons/hive-bridge/src/test/java/org/apache/atlas/hive/hook/HiveHookIT.java (line 1121) <https://reviews.apache.org/r/52077/#comment217523> rename to testColumnLevelLineage addons/hive-bridge/src/test/java/org/apache/atlas/hive/hook/HiveHookIT.java (line 1143) <https://reviews.apache.org/r/52077/#comment217524> Also assert on lineage API response on columns - Shwetha GS On Sept. 20, 2016, 9:07 a.m., Vimal Sharma wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52077/ > ----------------------------------------------------------- > > (Updated Sept. 20, 2016, 9:07 a.m.) > > > Review request for atlas. > > > Bugs: ATLAS-247 > https://issues.apache.org/jira/browse/ATLAS-247 > > > Repository: atlas > > > Description > ------- > > After a CTAS query, lineage relationship between source columns and > destination column can be captured. This information can be used to create a > column lineage process. > > > Diffs > ----- > > > addons/hive-bridge/src/main/java/org/apache/atlas/hive/bridge/ColumnLineageUtils.java > PRE-CREATION > addons/hive-bridge/src/main/java/org/apache/atlas/hive/hook/HiveHook.java > a3464a0 > > addons/hive-bridge/src/main/java/org/apache/atlas/hive/model/HiveDataModelGenerator.java > 45f0bc9 > > addons/hive-bridge/src/main/java/org/apache/atlas/hive/model/HiveDataTypes.java > e094cb6 > addons/hive-bridge/src/test/java/org/apache/atlas/hive/hook/HiveHookIT.java > a5838b4 > > Diff: https://reviews.apache.org/r/52077/diff/ > > > Testing > ------- > > > Thanks, > > Vimal Sharma > >
