Re: Review Request 52077: Column level lineage in Hive

2016-09-30 Thread Shwetha GS
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52077/#review150990 --- Ship it! Ship It! - Shwetha GS On Sept. 29, 2016, 8:15

Re: Review Request 52077: Column level lineage in Hive

2016-09-29 Thread Suma Shivaprasad
> On Sept. 21, 2016, 8:58 p.m., Suma Shivaprasad wrote: > > addons/hive-bridge/src/main/java/org/apache/atlas/hive/bridge/ColumnLineageUtils.java, > > line 97 > > > > > > why is column qualifiedName different from

Re: Review Request 52077: Column level lineage in Hive

2016-09-29 Thread Vimal Sharma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52077/ --- (Updated Sept. 29, 2016, 8:15 a.m.) Review request for atlas. Changes

Re: Review Request 52077: Column level lineage in Hive

2016-09-29 Thread Shwetha GS
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52077/#review150821 --- Fix it, then Ship it!

Re: Review Request 52077: Column level lineage in Hive

2016-09-26 Thread Vimal Sharma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52077/ --- (Updated Sept. 26, 2016, 1:06 p.m.) Review request for atlas. Changes

Re: Review Request 52077: Column level lineage in Hive

2016-09-26 Thread Shwetha GS
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52077/#review150383 --- 1. Can you add a test with lineage query on column? 2.

Re: Review Request 52077: Column level lineage in Hive

2016-09-26 Thread Vimal Sharma
> On Sept. 21, 2016, 8:58 p.m., Suma Shivaprasad wrote: > > addons/hive-bridge/src/main/java/org/apache/atlas/hive/bridge/ColumnLineageUtils.java, > > line 97 > > > > > > why is column qualifiedName different from

Re: Review Request 52077: Column level lineage in Hive

2016-09-23 Thread Suma Shivaprasad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52077/#review150215 ---

Re: Review Request 52077: Column level lineage in Hive

2016-09-23 Thread Suma Shivaprasad
> On Sept. 21, 2016, 8:58 p.m., Suma Shivaprasad wrote: > > addons/hive-bridge/src/main/java/org/apache/atlas/hive/bridge/ColumnLineageUtils.java, > > line 97 > > > > > > why is column qualifiedName different from

Re: Review Request 52077: Column level lineage in Hive

2016-09-22 Thread Vimal Sharma
> On Sept. 21, 2016, 8:58 p.m., Suma Shivaprasad wrote: > > addons/hive-bridge/src/main/java/org/apache/atlas/hive/bridge/ColumnLineageUtils.java, > > line 97 > > > > > > why is column qualifiedName different from

Re: Review Request 52077: Column level lineage in Hive

2016-09-22 Thread Vimal Sharma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52077/ --- (Updated Sept. 22, 2016, 11:53 a.m.) Review request for atlas. Changes

Re: Review Request 52077: Column level lineage in Hive

2016-09-22 Thread Vimal Sharma
> On Sept. 21, 2016, 9:01 p.m., Suma Shivaprasad wrote: > > addons/hive-bridge/src/main/java/org/apache/atlas/hive/hook/HiveHook.java, > > line 628 > > > > > > shouldnt the lineage process refer to the same set of

Re: Review Request 52077: Column level lineage in Hive

2016-09-21 Thread Suma Shivaprasad
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52077/#review149896 ---

Re: Review Request 52077: Column level lineage in Hive

2016-09-21 Thread Shwetha GS
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52077/#review149804 ---

Re: Review Request 52077: Column level lineage in Hive

2016-09-20 Thread Vimal Sharma
--- 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. Changes

Review Request 52077: Column level lineage in Hive

2016-09-20 Thread Vimal Sharma
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/52077/ --- Review request for atlas. Repository: atlas Description --- After a