0.8.1 release of Apache Atlas

2017-08-02 Thread Sarath Subramanian
ment in use of Solr index for attribute filtering. Please share your comments and thoughts. Thanks, Sarath Subramanian

Re: Review Request 61375: Fix Maven warnings ATLAS-2015

2017-08-02 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61375/#review182020 --- Ship it! Ship It! - Sarath Subramanian On Aug. 2, 2017, 3

Review Request 61681: [ATLAS-1475]: Create Entity via UI : Currently a simple entity can be associated with any number of complex entities, which seems inconsistent.

2017-08-16 Thread Sarath Subramanian
from UI, simple eneity restricted from getting associated to complex entity if it has an active ownedRef/inverseRef mvn clean install - in progress Thanks, Sarath Subramanian

Re: Review Request 61589: [ATLAS-2040]: Relationship with many-to-many cardinality gives incorrect relationship attribute value

2017-08-11 Thread Sarath Subramanian
UTs to test many-to-many relationships on same attribute name/same type * added new UTs to test one-to-one relationships on same attribute name/same type * mvn clean install -DskipITs=true suceeded without any failures Thanks, Sarath Subramanian

Re: Review Request 61708: ATLAS-2052: updated packaing to exclude junit jar

2017-08-16 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61708/#review183094 --- Ship it! Ship It! - Sarath Subramanian On Aug. 16, 2017, 6

Re: [ANNOUNCE] New Committer: Mandy Chessell

2017-08-16 Thread Sarath Subramanian
Congratulations Mandy. On Wed, Aug 16, 2017 at 4:36 AM, Keval Bhatt wrote: > Congratulations Mandy. > > On Wed, Aug 16, 2017 at 3:09 PM, David Radley > wrote: > > > Congratulations Mandy > > > > > > > > From: Mandy Chessell

Re: Review Request 60851: [ATLAS-1949]: Fix coverity scan issues and IT failure due to ATLAS-1907

2017-07-13 Thread Sarath Subramanian
org.apache.atlas.web.integration.EntityJerseyResourceIT.testPartialUpdate Thanks, Sarath Subramanian

Review Request 60783: [ATLAS-1943] : Fix IT failure due to incorrect inverse reference check using relationship

2017-07-11 Thread Sarath Subramanian
/EntityGraphMapper.java d5c1e862 Diff: https://reviews.apache.org/r/60783/diff/1/ Testing --- mvn clean package suceeded. Thanks, Sarath Subramanian

Re: Review Request 60578: [ATLA-1907]: Create RelationshipDefs for all base models and use this definitions when creating edges for entities

2017-07-10 Thread Sarath Subramanian
hive_table creation and lineage is rendering. UTs/ITs in progress. Thanks, Sarath Subramanian

Re: Review Request 61073: ATLAS-1982: Update references to "incubator" in website

2017-07-23 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61073/#review181189 --- Ship it! Ship It! - Sarath Subramanian On July 23, 2017, 9

Re: Relationship attributes

2017-07-24 Thread Sarath Subramanian
Hi David, I agree with using the term 'relationship attributes' for attributes of relationship, I suggest we use "relatedAttributes" for relationship attributes of entity. Thanks, Sarath Subramanian On Sun, Jul 23, 2017 at 2:22 AM, David Radley <david_rad...@uk.ibm.com> wro

Review Request 61088: [ATLAS-1988]: Implement REST API to search for related entities

2017-07-24 Thread Sarath Subramanian
/AtlasGremlin2QueryProvider.java 1bf03464 repository/src/main/java/org/apache/atlas/util/AtlasGremlinQueryProvider.java 8481a4f6 webapp/src/main/java/org/apache/atlas/web/rest/DiscoveryREST.java efab72a3 Diff: https://reviews.apache.org/r/61088/diff/1/ Testing --- Thanks, Sarath Subramanian

Review Request 61191: [ATLAS-1999]: Use AtlasRelatedObjectId to display relationshipAttribute values during entity retrieval

2017-07-27 Thread Sarath Subramanian
relationship attribute values in AtlasRelatedObjectId format is retrieved * mvn clean package - test in progress Thanks, Sarath Subramanian

Re: Review Request 60938: [ATLAS-1959]: Enhance relationship attributes to support different cardinality mappings

2017-07-19 Thread Sarath Subramanian
; > will this come through to thie code. yes, to support backward compatibility entity creation with constraints is still allowed. - Sarath ------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6

Re: Review Request 60938: [ATLAS-1959]: Enhance relationship attributes to support different cardinality mappings

2017-07-20 Thread Sarath Subramanian
> On July 19, 2017, 8:11 a.m., David Radley wrote: > > addons/models/0080-storm_model.json > > Line 150 (original), 150 (patched) > > <https://reviews.apache.org/r/60938/diff/2/?file=1778156#file1778156line150> > > > > shouldn't this be aggr

Review Request 61007: [ATLAS-1979]: Update storm model relationship category and fix for UT and Coverity scan issues

2017-07-20 Thread Sarath Subramanian
a/org/apache/atlas/repository/store/graph/v1/AtlasRelationshipStoreV1Test.java 31efe868 Diff: https://reviews.apache.org/r/61007/diff/1/ Testing --- mvn clean package succeeded Thanks, Sarath Subramanian

Review Request 60578: [ATLA-1907]: Create RelationshipDefs for all base models and use this definitions when creating edges for entities

2017-06-30 Thread Sarath Subramanian
. UTs/ITs in progress. Thanks, Sarath Subramanian

Re: Review Request 60578: [ATLA-1907]: Create RelationshipDefs for all base models and use this definitions when creating edges for entities

2017-07-06 Thread Sarath Subramanian
://reviews.apache.org/r/60578/diff/2/ Changes: https://reviews.apache.org/r/60578/diff/1-2/ Testing --- Tested using Postman REST client. Validated all hive_table creation works fine and lineage is rendered right. UTs/ITs in progress. Thanks, Sarath Subramanian

Review Request 61247: [ATLAS-1984]: Use AtlasRelatedObjectId to refer to relationship attributes during entity create/update

2017-07-30 Thread Sarath Subramanian
.org/r/61247/diff/1/ Testing --- Tested using POSTMAN REST client mvn clean package - In Progress Thanks, Sarath Subramanian

Re: Review Request 61247: [ATLAS-1984]: Use AtlasRelatedObjectId to refer to relationship attributes during entity create/update

2017-08-08 Thread Sarath Subramanian
tributes takes precedence. I think this method would apply to both legacy and newer cases. - Sarath --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61247/#review182278 -

Re: Review Request 61037: ATLAS-1980: Add more relationship instance validation

2017-08-08 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61037/#review182444 --- Ship it! Ship It! - Sarath Subramanian On July 22, 2017, 1

Re: Review Request 61191: [ATLAS-1999]: Use AtlasRelatedObjectId to display relationshipAttribute values during entity retrieval

2017-08-08 Thread Sarath Subramanian
relationship attribute values in AtlasRelatedObjectId format is retrieved * mvn clean package - succeeded Thanks, Sarath Subramanian

Re: Review Request 61502: ATLAS-2025: Validation on type name and classification

2017-08-08 Thread Sarath Subramanian
<https://reviews.apache.org/r/61502/#comment258315> if (entityType == null) check is enough, since null and empty typeName, return null entityType. Same for line 66 - Sarath Subramanian On Aug. 8, 2017, 10:16 a.m., Apoor

Re: Review Request 61502: ATLAS-2025: Validation on type name and classification

2017-08-08 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61502/#review182431 --- Ship it! Ship It! - Sarath Subramanian On Aug. 8, 2017, 10

Review Request 61747: [ATLAS-2059]: Update test-patch.sh script for PreCommit-ATLAS-Build jenkins job

2017-08-18 Thread Sarath Subramanian
job - https://builds.apache.org/view/A/view/Atlas/job/PreCommit-ATLAS-Build-Test/ We can test patches using jenkins job (Build with parameters) using: * Review Board ID * JIRA ID * Local Atlas Patch Thanks, Sarath Subramanian

Re: Review Request 61721: [ATLAS-2054]: Add flag to exclude attribute mapping in FullTextMapper

2017-08-17 Thread Sarath Subramanian
/ Changes: https://reviews.apache.org/r/61721/diff/1-2/ Testing --- mvn clean package suceeded. Thanks, Sarath Subramanian

Review Request 61721: [ATLAS-2054]: Add flag to exclude attribute mapping in FullTextMapper

2017-08-17 Thread Sarath Subramanian
atlas.search.fulltext.type.hive_column.attributes.exclude=profileData Diffs - repository/src/main/java/org/apache/atlas/repository/graph/FullTextMapperV2.java 1ed12bc0 Diff: https://reviews.apache.org/r/61721/diff/1/ Testing --- mvn clean package suceeded. Thanks, Sarath Subramanian

Re: Review Request 61085: ATLAS-1983: Add relationship cardinality validation

2017-08-16 Thread Sarath Subramanian
getEndDef2().getName() : null; attribute = end2Type.getAttribute(attributeName); } return (attribute != null) ? attribute.getRelationshipEdgeLabel() : null; } - Sarath Subramanian On July 24, 2017, 10:18 a.m.

Re: [ANNOUNCE] New Committer: Ashutosh Mestry

2017-08-16 Thread Sarath Subramanian
Congratulations Ashutosh. On Wed, Aug 16, 2017 at 4:38 AM, Keval Bhatt wrote: > Congratulations Ashutosh! > > On Wed, Aug 16, 2017 at 12:40 AM, Apoorv Naik > wrote: > > > Congratulations Ashutosh. > > > > > > > > > > > > On 8/15/17, 10:21 AM,

Re: [ANNOUNCE] New Committer: David Radley

2017-08-16 Thread Sarath Subramanian
Congratulations David. On Wed, Aug 16, 2017 at 4:38 AM, Keval Bhatt wrote: > Congratulations David. > > On Wed, Aug 16, 2017 at 12:40 AM, Apoorv Naik > wrote: > > > Congratulations David. > > > > > > > > > > > > On 8/15/17, 10:20 AM, "Ayub Khan

Re: Review Request 61762: [ATLAS-2062]: Add flag to disable v1 gremlin search API using atlas-application property

2017-08-18 Thread Sarath Subramanian
that the gremlin saerch is enabled/disabled by toggling the flag atlas.search.gremlin.enable in atlas-application.properties Thanks, Sarath Subramanian

Review Request 61825: [ATLAS-2079]: Fix IT failures introduced by ATLAS-2062

2017-08-22 Thread Sarath Subramanian
org/r/61825/diff/1/ Testing --- The ITs works fine for webapp module. Thanks, Sarath Subramanian

Review Request 61824: [ATLAS-2078]: Updating types using v1 API doesn't route to createUpdateTypesDef in v2 API

2017-08-22 Thread Sarath Subramanian
://reviews.apache.org/r/61824/diff/1/ Testing --- validated using POSTMAN REST client. Thanks, Sarath Subramanian

Review Request 61719: [ATLAS-2050]: Add helper method in relationship store to check if entity has relationship associated

2017-08-17 Thread Sarath Subramanian
/AtlasRelationshipStoreV1.java 4e8c148a Diff: https://reviews.apache.org/r/61719/diff/1/ Testing --- Thanks, Sarath Subramanian

Re: Review Request 60272: ATLAS-1891 Additional Validation checks for RelationshipDefs

2017-06-26 Thread Sarath Subramanian
> On June 25, 2017, 11:24 p.m., Sarath Subramanian wrote: > > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasRelationshipDefStoreV1.java > > Lines 413 (patched) > > <https://reviews.apache.org/r/60272/diff/1/?file=1762491#file1762491line421>

Re: Tag propagation and classification entityTypes

2017-09-18 Thread Sarath Subramanian
This offers more flexibility to add constraints on tag propagation than including in graph query. Once we move to Janus graph we can tweak the query using TP3 syntax. Thanks, Sarath Subramanian On Fri, Sep 15, 2017 at 1:25 AM, David Radley <david_rad...@uk.ibm.com> wrote: > Hi Madhan a

Re: Review Request 62401: ATLAS-2138: Admin import calls in Atlas client

2017-09-19 Thread Sarath Subramanian
/AtlasBaseClient.java Lines 596 (patched) <https://reviews.apache.org/r/62401/#comment261969> considering using static string: this(path, method, status, JSON_MEDIA_TYPE, MediaType.APPLICATION_JSON); - Sarath Subramanian On Sept. 18, 2017, 10:54 p.m., Apoorv Naik

Re: Review Request 62457: ATLAS-2157: HiveHook error while processing temporary tables

2017-09-21 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62457/#review185921 --- Ship it! Ship It! - Sarath Subramanian On Sept. 21, 2017

Re: Review Request 62455: ATLAS-2092: Upfront edgelabel creation to avoid concurrency issues

2017-09-20 Thread Sarath Subramanian
;& validated2); - Sarath Subramanian On Sept. 20, 2017, 2:55 p.m., Apoorv Naik wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https

Review Request 62342: [ATLAS-2136]: Change audit and notification entityChangeListener to start using v2 API structure

2017-09-14 Thread Sarath Subramanian
operations: 1. Entity Create 2. Entity Update 3. Entity Delete 4. Add Classification 5. Delete Classification 6. Update Classification Listeners were notified using v2 API structure Thanks, Sarath Subramanian

Re: Review Request 62457: ATLAS-2157: HiveHook error while processing temporary tables

2017-09-21 Thread Sarath Subramanian
/apache/atlas/hive/hook/HiveHook.java Line 552 (original), 550 (patched) <https://reviews.apache.org/r/62457/#comment262246> do we need this additional get on db? doesn't db have a valid value assigned from switch statement above? - Sarath Subramanian On Sept. 21, 2017, 10:39 a.m.,

Re: Review Request 62342: [ATLAS-2136]: Change audit and notification entityChangeListener to start using v2 API structure

2017-09-18 Thread Sarath Subramanian
Classification 6. Update Classification Listeners were notified using v2 API structure Thanks, Sarath Subramanian

Re: Review Request 63037: ATLAS-2213: Remove unused attributes from Hive hook

2017-10-16 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63037/#review188172 --- Ship it! Ship It! - Sarath Subramanian On Oct. 16, 2017, 10

Re: Review Request 62875: ATLAS-2201: Test consistency and dependency upgrade

2017-10-13 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62875/#review188017 --- Ship it! Ship It! - Sarath Subramanian On Oct. 11, 2017, 11

Review Request 62109: [ATLAS-2116] : Implement tag propagation using relationships

2017-09-06 Thread Sarath Subramanian
6eadc967 Diff: https://reviews.apache.org/r/62109/diff/1/ Testing --- Tested using UI and POSTMAN REST client. Unit Tests/Integration tests - In progress Thanks, Sarath Subramanian

Re: [ANNOUNCE] New Committer: Sharmadha Sainath

2017-09-08 Thread Sarath Subramanian
Congratulations Sharmadha! Regards, Sarath Subramanian On Thu, Sep 7, 2017 at 4:49 PM, Madhan Neethiraj <mad...@apache.org> wrote: > Atlas community, > > > > Apache Atlas PMC has asked Sharmadha Sainath to become a committer and we > are pleased to announ

Review Request 61997: [ATLAS-2099]: Change branch-0.8 pom version to 0.8.2-SNAPSHOT

2017-08-30 Thread Sarath Subramanian
/pom.xml 38a7d10b webapp/pom.xml 59d5f085 Diff: https://reviews.apache.org/r/61997/diff/1/ Testing --- Thanks, Sarath Subramanian

Re: Review Request 61200: ATLAS-1996: Log4J Default Configuration Updated

2017-09-07 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61200/#review184848 --- Ship it! Ship It! - Sarath Subramanian On Sept. 7, 2017, 10

Review Request 62126: [ATLAS-2114]: Regression : Type Update using V1 APIs

2017-09-06 Thread Sarath Subramanian
/apache/atlas/repository/store/graph/AtlasTypeDefGraphStore.java 8d68986d Diff: https://reviews.apache.org/r/62126/diff/1/ Testing --- validated using test script to create types and update types using same version number. suceeded. Thanks, Sarath Subramanian

Re: Review Request 62244: ATLAS2058 Add description to attributedefs and relationship enddefs

2017-09-12 Thread Sarath Subramanian
/AtlasStructDef.java Lines 275 (patched) <https://reviews.apache.org/r/62244/#comment261513> can we include descriptions for existing attributes and relationshipEndDefs for out of the box models as part of this JIRA? - Sarath Subramanian On Sept. 12, 2017, 10:13 a.m., David Radley

Re: Review Request 62342: [ATLAS-2136]: Change audit and notification entityChangeListener to start using v2 API structure

2017-09-26 Thread Sarath Subramanian
using v2 API structure Thanks, Sarath Subramanian

Re: [GitHub] atlas pull request #1: fixed Export-API documentation curl example json synt...

2017-09-27 Thread Sarath Subramanian
Daniel-Lee-Narvar, Atlas team uses Review Board to review patches. Could you open a JIRA for the issue and upload your patch in Review Board for the team to review. https://reviews.apache.org/groups/atlas/ Thanks, Sarath Subramanian On Tue, Sep 26, 2017 at 3:44 PM, Daniel-Lee-Narvar &l

Re: Review Request 62691: ATLAS-2089: Upgrade Jetty to Newer Version

2017-09-29 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62691/#review186747 --- Ship it! Ship It! - Sarath Subramanian On Sept. 29, 2017, 9

Re: Review Request 61670: ATLAS-1880: Integration tests for the search functionality

2017-09-29 Thread Sarath Subramanian
/BasicSearchIT.java Lines 106 (patched) <https://reviews.apache.org/r/61670/#comment263524> consider consolidating individual search-parameters json files into a small group of related files - array of json objects. - Sarath Subramanian On Sept. 29, 2017, 11:46 a.m., Apoorv Naik

Re: Review Request 62679: ATLAS-2184: IS_NULL and NOT_NULL operator support

2017-09-29 Thread Sarath Subramanian
/model/discovery/SearchParameters.java Line 349 (original), 349 (patched) <https://reviews.apache.org/r/62679/#comment263548> BEGINS_WITH is specified in both line 348 and 349 -> should it be ENDS_WITH? - Sarath Subramanian On Sept. 29, 2017, 9:49 a.m., Apoorv N

Review Request 62557: [ATLAS-2170]: Change master branch version from 0.9-SNAPSHOT to 1.0.0-SNAPSHOT

2017-09-25 Thread Sarath Subramanian
fe596802 shaded/hbase-server-shaded/pom.xml fd61ac96 typesystem/pom.xml 5b27ed75 webapp/pom.xml 58c2e0aa Diff: https://reviews.apache.org/r/62557/diff/1/ Testing --- mvn clean install -DskipTests validated all the build artifacts have new version 1.0.0-SNAPSHOT Thanks, Sarath

Re: Review Request 61854: ATLAS-2082: basic-search fails when in-memory filters all results from index-query

2017-08-23 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61854/#review183630 --- Ship it! Ship It! - Sarath Subramanian On Aug. 23, 2017, 10

Re: Review Request 61589: [ATLAS-2040]: Relationship with many-to-many cardinality gives incorrect relationship attribute value

2017-08-23 Thread Sarath Subramanian
many-to-many relationships on same attribute name/same type * added new UTs to test one-to-one relationships on same attribute name/same type * mvn clean install -DskipITs=true suceeded without any failures Thanks, Sarath Subramanian

Re: Review Request 61247: [ATLAS-1984]: Use AtlasRelatedObjectId to refer to relationship attributes during entity create/update

2017-08-23 Thread Sarath Subramanian
N REST client mvn clean package - In Progress Thanks, Sarath Subramanian

Re: Emtity Resource v1 test failures

2017-08-24 Thread Sarath Subramanian
Hi Nigel, Few Integration Test fixes have gone into master in the last couple of days, could you rebase master and try the tests again. Thanks, Sarath Subramanian On Thu, Aug 24, 2017 at 7:05 AM, Nigel Jones <nigel.l.jo...@gmail.com> wrote: > Whilst working with ATLAS-2012 (effectiv

Re: [VOTE] Release Apache Atlas 0.8.1 - release candidate 1

2017-08-24 Thread Sarath Subramanian
(and reason why) +1 from my side for the release. Thanks, Sarath Subramanian On Thu, Aug 24, 2017 at 4:47 PM, Madhan Neethiraj <mad...@apache.org> wrote: > +1 for release candidate 1. > > Impressive list of improvements and fixes – 142 JIRAs. Here are few that > standout:

Re: Review Request 61247: [ATLAS-1984]: Use AtlasRelatedObjectId to refer to relationship attributes during entity create/update

2017-08-24 Thread Sarath Subramanian
esting --- Tested using POSTMAN REST client mvn clean package - In Progress Thanks, Sarath Subramanian

Re: Review Request 61794: ATLAS-2076: Add RELATIONSHIP type to search filters

2017-08-21 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61794/#review183409 --- Ship it! Ship It! - Sarath Subramanian On Aug. 21, 2017, 1

[RESULT] [VOTE] Release Apache Atlas 0.8.1 - RC 1

2017-08-28 Thread Sarath Subramanian
following members (binding): +1 Sarath Subramanian (Committer, PMC) +1 Nixon Rodrigues (Committer, PMC) +1 Keval Bhatt (Committer, PMC) +1 Apoorv Naik (Committer, PMC) +1 Madhan Neethiraj (Committer, PMC) +1 Ashutosh Mestry (Committer) +1 Ayub Khan Pathan (Committer, PMC) Five +1 vote from following

Re: Review Request 61589: [ATLAS-2040]: Relationship with many-to-many cardinality gives incorrect relationship attribute value

2017-08-28 Thread Sarath Subramanian
new UTs to test many-to-many relationships on same attribute name/same type * added new UTs to test one-to-one relationships on same attribute name/same type * mvn clean install -DskipITs=true suceeded without any failures Thanks, Sarath Subramanian

[VOTE] Release Apache Atlas 0.8.1 - release candidate 1

2017-08-23 Thread Sarath Subramanian
from my side for the release. Thanks, Sarath Subramanian

Re: Emtity Resource v1 test failures

2017-08-25 Thread Sarath Subramanian
which also modified/read the same property, since Berkley DB is ACID compliant subsequent transactions times out since the previous transaction is not closed properly. We don't see this in HBase since its not an ACID compliant. Thanks, Sarath Subramanian On Fri, Aug 25, 2017 at 7:10 AM, Apoorv

branch 0.8-incubating renamed to branch-0.8

2017-08-21 Thread Sarath Subramanian
Atlas team, In view of Atlas graduation, we have renamed branch *0.8-incubating* to *branch-0.8* Thanks, Sarath Subramanian

Re: Review Request 61526: ATLAS-2029: Restrict entities, classifications can be applied to

2017-09-01 Thread Sarath Subramanian
) to populate this.entityTypes intg/src/main/java/org/apache/atlas/type/AtlasClassificationType.java Lines 408 (patched) <https://reviews.apache.org/r/61526/#comment260555> doesn't entityTypes contain all superType entityTypes and current entityTypes? why is the need to recompute it again in allEntitySu

Re: Review Request 61980: ATLAS-2087: Allow Atlas server to bind on a specific address

2017-08-29 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61980/#review184111 --- Ship it! Ship It! - Sarath Subramanian On Aug. 29, 2017, 4

Re: Review Request 62009: ATLAS-2101: Update Implementation to Eliminate Use of Stopwatch

2017-08-31 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62009/#review184302 --- Ship it! Ship It! - Sarath Subramanian On Aug. 30, 2017, 10

Re: Review Request 62009: ATLAS-2101: Update Implementation to Eliminate Use of Stopwatch

2017-08-31 Thread Sarath Subramanian
/database/idassigner/StandardIDPool.java Lines 1 (patched) <https://reviews.apache.org/r/62009/#comment260436> Need to add apache license header. getting rat failure. - Sarath Subramanian On Aug. 30, 2017, 10:02 p.m., Ashutosh Mestry

[ANNOUNCE] Apache Atlas 0.8.1 released

2017-08-29 Thread Sarath Subramanian
Hi All, The Apache Atlas team is happy to announce the release of Apache Atlas - version 0.8.1. Atlas is a scalable and extensible set of core foundational governance services – enabling enterprises to effectively and efficiently meet their compliance requirements within Hadoop and allows

Review Request 62914: [ATLAS-2206]: Remove shading of hbase-server and hbase-client libraries

2017-10-11 Thread Sarath Subramanian
39ae6e72 repository/pom.xml b7eedde0 Diff: https://reviews.apache.org/r/62914/diff/1/ Testing --- mvn clean install and validated atlas functionality Thanks, Sarath Subramanian

Review Request 62909: [ATLAS-2191]: Update atlas to use the latest component release versions

2017-10-11 Thread Sarath Subramanian
/KafkaNotificationMockTest.java 09e2e437 pom.xml 39ae6e72 repository/pom.xml b7eedde0 shaded/hbase-client-shaded/pom.xml 180ff03b shaded/hbase-server-shaded/pom.xml 3cf500a8 webapp/pom.xml bfa79e81 Diff: https://reviews.apache.org/r/62909/diff/1/ Testing --- Thanks, Sarath

Review Request 62909: [ATLAS-2191]: Update atlas to use the latest component release versions

2017-10-11 Thread Sarath Subramanian
/KafkaNotificationMockTest.java 09e2e437 pom.xml 39ae6e72 repository/pom.xml b7eedde0 shaded/hbase-client-shaded/pom.xml 180ff03b shaded/hbase-server-shaded/pom.xml 3cf500a8 webapp/pom.xml bfa79e81 Diff: https://reviews.apache.org/r/62909/diff/1/ Testing --- Thanks, Sarath

Review Request 62910: [ATLAS-2191]: Update atlas to use the latest component release versions

2017-10-11 Thread Sarath Subramanian
atlas server with embedded HBase and Solr. Tested basic sanity - lineage, quick start, basic search and hive hook message processing. Thanks, Sarath Subramanian

Re: Review Request 62874: Hive hook OOM fix

2017-10-10 Thread Sarath Subramanian
/apache/atlas/hive/hook/HiveHook.java Line 592 (original), 579 (patched) <https://reviews.apache.org/r/62874/#comment264630> we need to add split messages here => populateEventMessagesPerEntity(event, entities); - Sarath Subramanian On Oct. 10, 2017, 3:32 p.m., Apoorv N

Review Request 63120: [ATLAS-2216]: Remove Catalog/Taxonomy feature from Atlas

2017-10-18 Thread Sarath Subramanian
-DskipTests build was sucessfull validation with embedded-hbase-solr profile - inprogress Thanks, Sarath Subramanian

Re: Review Request 63136: ATLAS-2218: Separate UT and IT in webapp module

2017-10-23 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63136/#review189009 --- could you rebase the patch - Sarath Subramanian On Oct. 18

Re: Review Request 63152: ATLAS-2218: TestMetadata.py fix

2017-10-23 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63152/#review188991 --- Ship it! Ship It! - Sarath Subramanian On Oct. 19, 2017, 9

Re: Review Request 63136: ATLAS-2218: Separate UT and IT in webapp module

2017-10-23 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63136/#review189017 --- Ship it! Ship It! - Sarath Subramanian On Oct. 18, 2017, 9

Re: Review Request 63792: [ATLAS-2258]: Fix new Integration test failures introduced due to ATLAS-2251

2017-11-14 Thread Sarath Subramanian
ResourceIT.java f25fc5f8 webapp/src/test/java/org/apache/atlas/web/integration/TypesJerseyResourceIT.java cd876f4f Diff: https://reviews.apache.org/r/63792/diff/2/ Changes: https://reviews.apache.org/r/63792/diff/1-2/ Testing --- mvn clean package Validated the failing ITs succeeds Thanks, Sarath Subramanian

Re: Unable to start atlas on master

2017-11-14 Thread Sarath Subramanian
to licensing requirements. Thanks, Sarath Subramanian On 11/14/17, 2:55 AM, "David Radley" <david_rad...@uk.ibm.com> wrote: >clean package >-Pdist,berkeley-elasticsearch -DskipTests

Re: Instructions to build & run Atlas in dev environment

2017-11-28 Thread Sarath Subramanian
, Sarath Subramanian On 11/28/17, 12:19 AM, "Madhan Neethiraj" <mad...@apache.org> wrote: atlas.graphdb.backend=org.apache.atlas.repository.graphdb.titan0.Titan0GraphDatabase

Re: Instructions to build & run Atlas in dev environment

2017-11-28 Thread Sarath Subramanian
Hi Graham, Could you share your modified webapp/pom.xml I’m trying to follow the instructions and build for Janus 0.2.0 but I’m facing a class not found exception during atlas startup - java.lang.ClassNotFoundException: org.apache.lucene.analysis.TokenStream Thanks, Sarath Subramanian

Review Request 64141: [ATLAS-2287]: Include lucene libraries when building atlas distribution with Janus profile

2017-11-28 Thread Sarath Subramanian
/ Testing --- validated building atlas distribution using both janus and titan0 profile. Atlas starts fine and basic functionalities working. mvn clean install -DskipTests -Pdist,embedded-hbase-solr mvn clean install -DskipTests -Pdist,embedded-hbase-solr -DGRAPH-PROVIDER=titan0 Thanks, Sarath

Re: Review Request 64141: [ATLAS-2287]: Include lucene libraries when building atlas distribution with Janus profile

2017-11-30 Thread Sarath Subramanian
ll. - Sarath ------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64141/#review192267 --- On Nov. 28, 2017, 6:32 p.m., Sarath Subramanian wro

Re: Review Request 64141: [ATLAS-2287]: Include lucene libraries when building atlas distribution with Janus profile

2017-11-30 Thread Sarath Subramanian
Thanks, Sarath Subramanian

Re: Review Request 64141: [ATLAS-2287]: Include lucene libraries when building atlas distribution with Janus profile

2017-11-30 Thread Sarath Subramanian
_HOME is set in your environment, echo $JAVA_HOME to confirm 3. Export the following (or add it to your bashrc and source it) export MANAGE_LOCAL_SOLR=true export MANAGE_LOCAL_HBASE=true # Clone and Build Atlas (for Janus profile) git clone https://github.com/apache/atlas.git -b

Re: Review Request 64235: ATLAS-2289: separate embedded kafka/zookeeper start/stop from KafkaNotification

2017-11-30 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64235/#review192427 --- Ship it! Ship It! - Sarath Subramanian On Nov. 30, 2017, 4

Re: Review Request 64253: ATLAS-2291: Update scope of hbase-server libraries to compile & test

2017-12-01 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64253/#review192550 --- Ship it! Ship It! - Sarath Subramanian On Dec. 1, 2017, 11

Re: Review Request 63905: ATLAS-2265: upgrade Jackson library version from 1.9.13 to 2.9.2

2017-11-17 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/63905/#review191378 --- Ship it! Ship It! - Sarath Subramanian On Nov. 17, 2017, 2

Re: Review Request 63658: ATLAS-2251: Remove/replace use of TypeSystem with AtlasTypeRegistry

2017-11-12 Thread Sarath Subramanian
> On Nov. 12, 2017, 2:09 a.m., Sarath Subramanian wrote: > > "mvn clean package" failed with this patch: Failed tests: AdaptiveWaiterTest.basicTest:43 expected [95] but found [100] - Sarath --- This is an automa

Re: Review Request 63658: ATLAS-2251: Remove/replace use of TypeSystem with AtlasTypeRegistry

2017-11-12 Thread Sarath Subramanian
4> for migration cases, existing vertices might still have __type => typeSystem webapp/src/main/java/org/apache/atlas/web/resources/LineageResource.java Line 193 (original), 180 (patched) <https://reviews.apache.org/r/63658/#comment268395> remove commented lines.

Review Request 63792: [ATLAS-2258]: Fix new Integration test failures introduced due to ATLAS-2251

2017-11-14 Thread Sarath Subramanian
an package Validated the failing ITs succeeds Thanks, Sarath Subramanian

Review Request 63892: [ATLAS-2264]: Update JanusGraph version to 0.2.0

2017-11-16 Thread Sarath Subramanian
an0 => uses titan0 profile and use elasticsearch packaged with titan jar Thanks, Sarath Subramanian

Review Request 63706: [ATLAS-2256]: Make JanusGraph as the default graph-provider in atlas

2017-11-09 Thread Sarath Subramanian
/AtlasRepositoryConfiguration.java 09f366e2 Diff: https://reviews.apache.org/r/63706/diff/1/ Testing --- mvn clean package succeeded. Ran quickstart and hive hook messages. Thanks, Sarath Subramanian

Re: Review Request 64350: ATLAS-2296: basic search enhancement to optionally exclude sub-types and sub-classification-types

2017-12-05 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64350/#review192940 --- Ship it! Ship It! - Sarath Subramanian On Dec. 5, 2017, 11

Re: Review Request 64435: ATLAS-2301: fix for concurrent modification error while creating audit record

2017-12-07 Thread Sarath Subramanian
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/64435/#review193227 --- Ship it! Ship It! - Sarath Subramanian On Dec. 7, 2017, 7

  1   2   3   4   5   6   7   8   9   10   >