Re: New committer to Apache Atlas: Apoorv Naik

2017-03-17 Thread Sarath Kumar Subramanian
Congrats Apoorv! Sarath > On Mar 16, 2017, at 10:56 PM, Kalyani Kashikar > wrote: > > Congratulations Apoorv > > On Fri, Mar 17, 2017 at 11:25 AM, Vimal Sharma > wrote: > >> Congrats Apoorv! >> >> -Vimal >> >> >>

Re: New PPMC member to Apache Atlas: Madhan Neethiraj

2017-03-17 Thread Sarath Kumar Subramanian
Congrats Madhan on your new role in the Atlas community! Thanks, Sarath > On Mar 16, 2017, at 10:55 PM, Vimal Sharma wrote: > > Congrats Madhan! > > > > >> On 3/17/17, 11:08 AM, "Sharmadha Sainath" wrote: >> >> >> Congrats Madhan !!

Re: [VOTE] Release Apache Atlas 0.8 (incubating) - release candidate 1 (dev group vote)

2017-03-08 Thread Sarath Kumar Subramanian
- Verified md5 checksum and signature. - No binaries in source - Build successful with embedded hbase-solr - Verified Quickstart, lineage, tags and search - Basic Sanity test +1 (non-binding) for the release. Thanks, Sarath Subramanian On Wed, Mar 8, 2017 at 9:10 PM, Ayub Khan

Re: [VOTE] Release Apache Atlas 0.7.1 (incubating) - release candidate 2 (dev group vote)

2017-01-18 Thread Sarath Kumar Subramanian
- Verified Checksum and Signature - Build Atlas using embedded hbase/solr - was successfull except some Integration test failures in webapp module - expected - Ran QuickStart and verified basic functionality - tags, entities, lineage and DSL search - No binaries found +1

Re: [VOTE] Release Apache Atlas 0.7.1 (incubating) - release candidate 1 (dev group vote)

2017-01-18 Thread Sarath Kumar Subramanian
- Verified Checksum and Signature - Build Atlas using embedded hbase/solr - was successfull except some Integration test failures in webapp module - expected - Ran QuickStart and verified basic functionality - tags, entities, lineage and DSL search - No binaries found +1

Re: Review Request 52533: patch file to add "position" attribute to hive_column type

2016-10-11 Thread Sarath Kumar Subramanian
----- On Oct. 11, 2016, 11:40 a.m., Sarath Kumar Subramanian wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/52533/ >

Re: Review Request 52533: patch file to add "position" attribute to hive_column type

2016-10-11 Thread Sarath Kumar Subramanian
atch since version is already in 2.0 Thanks, Sarath Kumar Subramanian

Re: Review Request 52585: Implementation of the new TypesREST API

2016-10-11 Thread Sarath Kumar Subramanian
che.org/r/52585/#comment220994> another NPE, initialize the list before adding items - Sarath Kumar Subramanian On Oct. 7, 2016, 10:28 p.m., Apoorv Naik wrote: > > --- > This is an automatically generated e-mail. To

Re: Review Request 52533: patch file to add "position" attribute to hive_column type

2016-10-06 Thread Sarath Kumar Subramanian
eviews.apache.org/r/52533/#review151632 --- On Oct. 6, 2016, 10:29 p.m., Sarath Kumar Subramanian wrote: > > --- > This is an automatically generated e-mail. To re

Re: Review Request 52533: patch file to add "position" attribute to hive_column type

2016-10-06 Thread Sarath Kumar Subramanian
nd updates version to 2.0 new builds - ignore the patch since version is already in 2.0 Thanks, Sarath Kumar Subramanian

Re: Review Request 52585: Implementation of the new TypesREST API

2016-10-06 Thread Sarath Kumar Subramanian
> > (Updated Oct. 5, 2016, 5:45 p.m.) > > > Review request for atlas, Madhan Neethiraj, Sarath Kumar Subramanian, Shwetha > GS, and Suma Shivaprasad. > > > Repository: atlas > > > Description > --- > > TypesREST implementation > > &g

Re: Review Request 52533: patch file to add "position" attribute to hive_column type

2016-10-05 Thread Sarath Kumar Subramanian
generated e-mail. To reply, visit: https://reviews.apache.org/r/52533/#review151459 ----------- On Oct. 4, 2016, 4:17 p.m., Sarath Kumar Subramanian wrote: > > --- > This

Re: Review Request 52533: patch file to add "position" attribute to hive_column type

2016-10-04 Thread Sarath Kumar Subramanian
atches/001-hive_column_add_position.json PRE-CREATION Diff: https://reviews.apache.org/r/52533/diff/ Testing (updated) --- on older builds - updates the type by adding new attribute and updates version to 2.0 new builds - ignore the patch since version is already in 2.0 Thanks, Sarath Kumar Subramanian

Re: Review Request 51896: ATLAS-1171: structured, high-level APIs

2016-09-28 Thread Sarath Kumar Subramanian
ntinue supporting existing REST APIs, what will be the mapping from this granular attributes to current one? eg. Required Collection or Optional Set. Which one takes precedence? - Sarath Kumar Subramanian On Sept. 28, 2016, 9:08 p.m., Madhan Neethiraj

Re: Review Request 51939: Framework to apply updates to types in the type-system

2016-09-27 Thread Sarath Kumar Subramanian
inition.java f1ce1b7 typesystem/src/main/java/org/apache/atlas/typesystem/types/TraitType.java f23bf5b typesystem/src/main/java/org/apache/atlas/typesystem/types/TypeSystem.java 70ba89b typesystem/src/main/java/org/apache/atlas/typesystem/types/utils/TypesUtil.java ef8448d typesystem/src/main/scala/org/apache/atlas/typesystem/json/TypesSerialization.scala 5618938 Diff: https://reviews.apache.org/r/51939/diff/ Testing --- Thanks, Sarath Kumar Subramanian

Re: Review Request 51939: Framework to apply updates to types in the type-system

2016-09-26 Thread Sarath Kumar Subramanian
an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51939/#review149727 ------- On Sept. 26, 2016, 5:25 p.m., Sarath Kumar Subramanian wrote: > > --- >

Re: Review Request 51939: Framework to apply updates to types in the type-system

2016-09-26 Thread Sarath Kumar Subramanian
------- On Sept. 26, 2016, 5:25 p.m., Sarath Kumar Subramanian wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51939/ > --

Re: Review Request 51939: Framework to apply updates to types in the type-system

2016-09-26 Thread Sarath Kumar Subramanian
system/types/TraitType.java f23bf5b typesystem/src/main/java/org/apache/atlas/typesystem/types/TypeSystem.java 70ba89b typesystem/src/main/java/org/apache/atlas/typesystem/types/utils/TypesUtil.java ef8448d typesystem/src/main/scala/org/apache/atlas/typesystem/json/TypesSerialization.scala 5618938 Diff: https://reviews.apache.org/r/51939/diff/ Testing --- Thanks, Sarath Kumar Subramanian

Re: Review Request 51939: Framework to apply updates to types in the type-system

2016-09-26 Thread Sarath Kumar Subramanian
On Sept. 21, 2016, 7:46 a.m., Sarath Kumar Subramanian wrote: > > Add tests > > > > 1. Currently, the model files(like hive_model.json) are auto generated from > > model definitions defined in java(like HiveDataModelGenerator). The patch > > files in this c

Re: Review Request 51939: Framework to apply updates to types in the type-system

2016-09-26 Thread Sarath Kumar Subramanian
/typesystem/types/TraitType.java f23bf5b typesystem/src/main/java/org/apache/atlas/typesystem/types/TypeSystem.java 70ba89b typesystem/src/main/java/org/apache/atlas/typesystem/types/utils/TypesUtil.java ef8448d typesystem/src/main/scala/org/apache/atlas/typesystem/json/TypesSerialization.scala 5618938 Diff: https://reviews.apache.org/r/51939/diff/ Testing --- Thanks, Sarath Kumar Subramanian

Re: Review Request 51939: Framework to apply updates to types in the type-system

2016-09-26 Thread Sarath Kumar Subramanian
typesystem/src/main/java/org/apache/atlas/typesystem/types/TraitType.java f23bf5b typesystem/src/main/java/org/apache/atlas/typesystem/types/TypeSystem.java 70ba89b typesystem/src/main/java/org/apache/atlas/typesystem/types/utils/TypesUtil.java ef8448d typesystem/src/main/scala/org/apache/atlas/typesystem/json/TypesSerialization.scala 5618938 Diff: https://reviews.apache.org/r/51939/diff/ Testing --- Thanks, Sarath Kumar Subramanian

Re: Review Request 51939: Framework to apply updates to types in the type-system

2016-09-22 Thread Sarath Kumar Subramanian
typesystem/src/main/java/org/apache/atlas/typesystem/types/TypeSystem.java 70ba89b typesystem/src/main/java/org/apache/atlas/typesystem/types/utils/TypesUtil.java ef8448d typesystem/src/main/scala/org/apache/atlas/typesystem/json/TypesSerialization.scala 5618938 Diff: https://reviews.apache.org/r/51939/diff/ Testing --- Thanks, Sarath Kumar Subramanian

Re: Review Request 51939: Framework to apply updates to types in the type-system

2016-09-22 Thread Sarath Kumar Subramanian
On Sept. 21, 2016, 7:46 a.m., Sarath Kumar Subramanian wrote: > > Add tests > > > > 1. Currently, the model files(like hive_model.json) are auto generated from > > model definitions defined in java(like HiveDataModelGenerator). The patch > > files in this c

Review Request 52166: Allow optional attributes to be removed when updating type

2016-09-22 Thread Sarath Kumar Subramanian
/TypeUtils.java f5c2ce9 Diff: https://reviews.apache.org/r/52166/diff/ Testing --- tested using REST calls to delete optional and required attributes and only optional attributes were removed. Thanks, Sarath Kumar Subramanian

Re: Review Request 51939: Framework to apply updates to types in the type-system

2016-09-22 Thread Sarath Kumar Subramanian
On Sept. 21, 2016, 7:46 a.m., Sarath Kumar Subramanian wrote: > > Add tests > > > > 1. Currently, the model files(like hive_model.json) are auto generated from > > model definitions defined in java(like HiveDataModelGenerator). The patch > > files in this c

Re: Review Request 51939: Framework to apply updates to types in the type-system

2016-09-22 Thread Sarath Kumar Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51939/#review149854 --- On Sept. 22, 2016, 12:34 a.m., Sarath Kumar Subramanian wrote: &

Re: Review Request 51939: Framework to apply updates to types in the type-system

2016-09-22 Thread Sarath Kumar Subramanian
typesystem/src/main/java/org/apache/atlas/typesystem/types/TypeSystem.java 70ba89b typesystem/src/main/java/org/apache/atlas/typesystem/types/utils/TypesUtil.java ef8448d typesystem/src/main/scala/org/apache/atlas/typesystem/json/TypesSerialization.scala 5618938 Diff: https://reviews.apache.org/r/51939/diff/ Testing --- Thanks, Sarath Kumar Subramanian

Re: Review Request 51939: Framework to apply updates to types in the type-system

2016-09-21 Thread Sarath Kumar Subramanian
typesystem/src/main/java/org/apache/atlas/typesystem/types/TypeSystem.java 70ba89b typesystem/src/main/java/org/apache/atlas/typesystem/types/utils/TypesUtil.java ef8448d typesystem/src/main/scala/org/apache/atlas/typesystem/json/TypesSerialization.scala 5618938 Diff: https://reviews.apache.org/r/51939/diff/ Testing --- Thanks, Sarath Kumar Subramanian

Re: Review Request 51939: Framework to apply updates to types in the type-system

2016-09-21 Thread Sarath Kumar Subramanian
system/types/TypeSystem.java 70ba89b typesystem/src/main/java/org/apache/atlas/typesystem/types/TypeUtils.java f5c2ce9 typesystem/src/main/java/org/apache/atlas/typesystem/types/utils/TypesUtil.java ef8448d typesystem/src/main/scala/org/apache/atlas/typesystem/json/TypesSerialization.scala 5618938 Diff: https://reviews.apache.org/r/51939/diff/ Testing --- Thanks, Sarath Kumar Subramanian

Re: Review Request 51939: Framework to apply updates to types in the type-system

2016-09-20 Thread Sarath Kumar Subramanian
are not laid in order. - Sarath Kumar --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/51939/#review149608 --- On Sept. 19, 2016, 6:

Re: Review Request 51939: Framework to apply updates to types in the type-system

2016-09-20 Thread Sarath Kumar Subramanian
e.org/r/51939/#review149652 --- On Sept. 19, 2016, 6:10 p.m., Sarath Kumar Subramanian wrote: > > --- > This is an automatically generated e-mail. To reply, visit:

Re: Review Request 51939: Framework to apply updates to types in the type-system

2016-09-19 Thread Sarath Kumar Subramanian
java f23bf5b typesystem/src/main/java/org/apache/atlas/typesystem/types/TypeSystem.java 70ba89b typesystem/src/main/java/org/apache/atlas/typesystem/types/utils/TypesUtil.java ef8448d typesystem/src/main/scala/org/apache/atlas/typesystem/json/TypesSerialization.scala 5618938 Diff: https://reviews.apache.org/r/51939/diff/ Testing --- Thanks, Sarath Kumar Subramanian

Re: Review Request 51939: Framework to apply updates to types in the type-system

2016-09-19 Thread Sarath Kumar Subramanian
c/main/java/org/apache/atlas/typesystem/types/TraitType.java f23bf5b typesystem/src/main/java/org/apache/atlas/typesystem/types/TypeSystem.java 70ba89b typesystem/src/main/java/org/apache/atlas/typesystem/types/utils/TypesUtil.java ef8448d typesystem/src/main/scala/org/apache/atlas/typesystem/json/TypesSerialization.scala 5618938 Diff: https://reviews.apache.org/r/51939/diff/ Testing --- Thanks, Sarath Kumar Subramanian

Re: Review Request 51939: Framework to apply updates to types in the type-system

2016-09-19 Thread Sarath Kumar Subramanian
------ On Sept. 15, 2016, 11:03 p.m., Sarath Kumar Subramanian wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/51939/ > ---

Review Request 51939: Framework to apply updates to types in the type-system

2016-09-15 Thread Sarath Kumar Subramanian
ypesystem/src/main/java/org/apache/atlas/typesystem/types/StructTypeDefinition.java f1ce1b7 typesystem/src/main/java/org/apache/atlas/typesystem/types/TraitType.java f23bf5b typesystem/src/main/java/org/apache/atlas/typesystem/types/TypeSystem.java 70ba89b typesystem/src/main/java/org/apache/

Review Request 51936: Framework to apply updates to types in the type-system

2016-09-15 Thread Sarath Kumar Subramanian
ition.json 002-hive_column_add_anotherattribute.json Diffs - repository/src/main/java/org/apache/atlas/services/AtlasTypeUpdate.java PRE-CREATION Diff: https://reviews.apache.org/r/51936/diff/ Testing --- Thanks, Sarath Kumar Subramanian

Re: Review Request 51723: ATLAS-1149 : Changes to UI to sort the hive table schema based on "position" attribute of hive_column

2016-09-09 Thread Sarath Kumar Subramanian
schema in UI looks good. - Sarath Kumar Subramanian On Sept. 8, 2016, 12:22 p.m., Kalyani Kashikar wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Review Request 51515: ATLAS-772 : New attribute has been added to the hive_column type to capture the position of hive columns for sorting

2016-08-30 Thread Sarath Kumar Subramanian
atlas/hive/model/HiveDataModelGenerator.java b308cc9 Diff: https://reviews.apache.org/r/51515/diff/ Testing --- Tested the changes with hive column operations – Add new column, Drop column, Rename column, Move column to first or at a position. Thanks, Sarath Kumar Subramanian