---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56634/#review165453
---
Ship it!
Ship It!
- Suma Shivaprasad
On Feb. 14, 2017, 2:20
AtlasDeleteHandlerV1Test.testUpdateEntity_MultiplicityOneNonCompositeReference
Thanks,
Suma Shivaprasad
AtlasDeleteHandlerV1Test.testUpdateEntity_MultiplicityOneNonCompositeReference
Thanks,
Suma Shivaprasad
AtlasDeleteHandlerV1Test.testUpdateEntity_MultiplicityOneNonCompositeReference
Thanks,
Suma Shivaprasad
ready in the last diff. Hence dropping the issue
- Suma
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56535/#review165581
-------
On Feb. 14, 2017, 10:34 p.m., Suma Shivaprasad wrote:
>
> --
onfusing from the naming. Will take this refactoring up as part of another
jira
- Suma
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56535/#review165581
-
failure in
AtlasDeleteHandlerV1Test.testUpdateEntity_MultiplicityOneNonCompositeReference
Thanks,
Suma Shivaprasad
/
Testing
---
Need to fix a test failure in
AtlasDeleteHandlerV1Test.testUpdateEntity_MultiplicityOneNonCompositeReference
Thanks,
Suma Shivaprasad
/java/org/apache/atlas/repository/store/graph/v1/SoftDeleteHandlerV1Test.java
987951e
server-api/src/main/java/org/apache/atlas/RequestContextV1.java bf73174
Diff: https://reviews.apache.org/r/56535/diff/
Testing (updated)
---
Added tests for hard delete and soft delete
Thanks,
Suma
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56900/#review166256
---
Ship it!
Ship It!
- Suma Shivaprasad
On Feb. 21, 2017, 10
repository/src/test/java/org/apache/atlas/repository/store/graph/v1/SoftDeleteHandlerV1Test.java
eba9c77
Diff: https://reviews.apache.org/r/56910/diff/
Testing
---
Thanks,
Suma Shivaprasad
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56963/#review166467
---
Ship it!
Ship It!
- Suma Shivaprasad
On Feb. 23, 2017, 1:51
/v1/EntityGraphMapper.java
(line 330)
<https://reviews.apache.org/r/56860/#comment238510>
handle map case as well?
- Suma Shivaprasad
On Feb. 22, 2017, 11:48 p.m., David Kantor wrote:
>
> ---
> This is an automatica
/repository/store/graph/v1/HardDeleteHandlerV1Test.java
fd685c6
repository/src/test/java/org/apache/atlas/repository/store/graph/v1/SoftDeleteHandlerV1Test.java
eba9c77
Diff: https://reviews.apache.org/r/56910/diff/
Testing
---
Thanks,
Suma Shivaprasad
> On Feb. 23, 2017, 7:23 p.m., Suma Shivaprasad wrote:
> > repository/src/main/java/org/apache/atlas/repository/store/graph/v1/EntityGraphMapper.java,
> > line 330
> > <https://reviews.apache.org/r/56860/diff/1/?file=1644999#file1644999line330>
> >
> >
> On Feb. 4, 2017, 6:39 p.m., Suma Shivaprasad wrote:
> > Ship It!
>
> Jeff Hagelberg wrote:
> How will this work for composite references? At least in V0 (I guess we
> can call it that), objects referenced via a composite reference need to be
> inlined.
T
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57020/#review166787
---
Ship it!
Ship It!
- Suma Shivaprasad
On Feb. 24, 2017, 7:36
through TestEntityREST
Thanks,
Suma Shivaprasad
Thanks,
Suma Shivaprasad
Thanks,
Suma Shivaprasad
/EntityMutationContext.java
23e825e
webapp/src/main/java/org/apache/atlas/web/resources/EntityResource.java
d7adb3a
webapp/src/main/java/org/apache/atlas/web/rest/EntityREST.java d1bef78
Diff: https://reviews.apache.org/r/57103/diff/
Testing
---
Tested through TestEntityREST
Thanks,
Suma
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57227/#review167638
---
Ship it!
Ship It!
- Suma Shivaprasad
On March 2, 2017, 2:18
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/57230/#review167645
---
Ship it!
Ship It!
- Suma Shivaprasad
On March 2, 2017, 3:37
Line 369 (original), 371 (patched)
<https://reviews.apache.org/r/57230/#comment239555>
should we change this to BigDecimal?
- Suma Shivaprasad
On March 2, 2017, 3:37 a.m., Madhan Neethiraj wrote:
>
> ---
> This is a
/resources/BaseService.java 2e999fc
Diff: https://reviews.apache.org/r/57356/diff/1/
Testing
---
Thanks,
Suma Shivaprasad
://reviews.apache.org/r/57408/diff/1/
Testing
---
All UTS pass
Thanks,
Suma Shivaprasad
Thanks,
Suma Shivaprasad
+1(binding)
Verified
1. signatures
2. Build sanity
3. Quickstart, taxonomy, lineage, schema and search
Thanks
Suma
On Wed, Mar 8, 2017 at 12:29 PM, Apoorv Naik wrote:
> +1 (non-binding) for the release
>
> 1. Tag exists
> 2. Build goes through fine
> 3. No rat failures
> 4. Was able to run q
Congrats, Apoorv!
On Wed, Mar 15, 2017 at 11:37 PM, Keval Bhatt
wrote:
> Congratulation, Apoorv!
>
> On Wed, Mar 15, 2017 at 11:23 AM, Apoorv Naik
> wrote:
>
> > Thanks Hemanth. I’m very happy and proud to be a part of the Apache Atlas
> > community. Looking forward to grow and help the communi
There is a build failure due to respository.codehause.org getting
deprecated on a clean maven repo. Thanks to Shwetha for pointing this out.
[ERROR] Failed to execute goal
org.apache.maven.plugins:maven-site-plugin:3.3:site (default) on project
atlas-docs: SiteToolException: The site descriptor ca
/r/35645/diff/
Testing
---
Added 2 tests abd verified that mixed index is getting picked up
Thanks,
Suma Shivaprasad
typesystem/src/main/java/org/apache/atlas/typesystem/types/TypeSystem.java
0d0c783
Diff: https://reviews.apache.org/r/35645/diff/
Testing
---
Added 2 tests abd verified that mixed index is getting picked up
Thanks,
Suma Shivaprasad
/TypeSystem.java
0d0c783
Diff: https://reviews.apache.org/r/35645/diff/
Testing
---
Added 2 tests abd verified that mixed index is getting picked up
Thanks,
Suma Shivaprasad
+1 (non binding, forwarded from podling vote)
On Fri, Jun 26, 2015 at 4:17 AM, Seetharam Venkatesh <
venkat...@innerzeal.com> wrote:
> Sorry Chris, one of the repo, codehaus is shutdown - we have already
> addressed this and will spin a new patch release post this. Thanks!
>
> On Thu, Jun 25, 201
tps://reviews.apache.org/r/36359/#comment144634>
Could remove protected to make it package private. Dont get what a
protected static method means.
- Suma Shivaprasad
On July 9, 2015, 6:06 p.m., Jonathan Maron wrote:
>
> ---
Hi Nigel,
Yes the Falcon plugin was removed since it was not yet mature. Only HIve
plugin is available as of now.There is work in progress to make these
plugins reliable - https://issues.apache.org/jira/browse/ATLAS-58
The plan is to work on Storm and Kafka plugins next. Falcon is not yet
planned
> On July 10, 2015, 5:05 p.m., Suma Shivaprasad wrote:
> > webapp/src/main/java/org/apache/atlas/Main.java, line 103
> > <https://reviews.apache.org/r/36359/diff/1/?file=1003745#file1003745line103>
> >
> > Could remove protected to make it package privat
Its, all other tests succeed.
Thanks,
Suma Shivaprasad
://reviews.apache.org/r/36550/diff/
Testing
---
Ran all UTs and ITS. Except the hive hook security Its, all other tests succeed.
Thanks,
Suma Shivaprasad
enerated e-mail. To reply, visit:
https://reviews.apache.org/r/36550/#review92230
---
On July 17, 2015, 1:12 p.m., Suma Shivaprasad wrote:
>
> ---
> This is an automatically generated e-mai
reviews.apache.org/r/36550/#review92075
---
On July 17, 2015, 1:12 p.m., Suma Shivaprasad wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
&
> On July 17, 2015, 1:18 p.m., Jonathan Maron wrote:
> > I'm curious about the switch from mortbay to eclipse for the jetty plugin?
> > Was it a configuraiton convenience issue? Any differences between the two
> > that we should test to ensure no loss of functional
://reviews.apache.org/r/36550/diff/
Testing
---
Ran all UTs and ITS. Except the hive hook security Its, all other tests succeed.
Thanks,
Suma Shivaprasad
/GraphBackedSearchIndexer.java
(line 302)
<https://reviews.apache.org/r/36615/#comment146314>
didnt get where we are applying the unique constraint on mixed indexes?
- Suma Shivaprasad
On July 20, 2015, 10:38 a.m., Shwetha GS
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36550/#review92335
---
On July 20, 2015, 11:19 a.m., Suma Shivaprasad wrote:
>
> ---
> On July 17, 2015, 1:18 p.m., Jonathan Maron wrote:
> > I'm curious about the switch from mortbay to eclipse for the jetty plugin?
> > Was it a configuraiton convenience issue? Any differences between the two
> > that we should test to ensure no loss of functional
(line 99)
<https://reviews.apache.org/r/36724/#comment147345>
+1. LGTM. Minor nit. Please correct indentation in the substituteVars
function
- Suma Shivaprasad
On July 24, 2015, 5:52 a.m., Shwetha GS wrote:
>
> ---
tps://reviews.apache.org/r/36935/#comment147957>
{ is missing
- Suma Shivaprasad
On July 30, 2015, 9:33 a.m., Shwetha GS wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36935/#review93574
---
Can we run tests in distributed mode once?
- Suma Shivaprasad
On
<https://reviews.apache.org/r/36935/#comment147956>
why is this removed? Not needed now?
- Suma Shivaprasad
On July 30, 2015, 9:33 a.m., Shwetha GS wrote:
>
> ---
> This is an automatically generated e-mail. To reply,
/GraphBackedSearchIndexer.java
(line 123)
<https://reviews.apache.org/r/36615/#comment148287>
Can we make force=false by default so that composite indexes are not
created by mistake
- Suma Shivaprasad
On Aug. 3, 2015, 6:48 a.m., Shwetha GS
/GraphBackedSearchIndexer.java
(line 297)
<https://reviews.apache.org/r/36615/#comment148286>
can we remove this comment since we are no longer ignoring cardinality
- Suma Shivaprasad
On Aug. 3, 2015, 6:48 a.m., Shwetha GS
/
Testing
---
NA
Thanks,
Suma Shivaprasad
ent148992>
Nice +1
- Suma Shivaprasad
On Aug. 6, 2015, 11:59 a.m., Shwetha GS wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://re
src/conf/solr/zoo.cfg PRE-CREATION
Diff: https://reviews.apache.org/r/37285/diff/
Testing
---
All tests pass . No new tests added
Thanks,
Suma Shivaprasad
/atlasConfig/conf/stopwords.txt PRE-CREATION
src/conf/solr/atlasConfig/conf/synonyms.txt PRE-CREATION
Diff: https://reviews.apache.org/r/37285/diff/
Testing
---
All tests pass . No new tests added
Thanks,
Suma Shivaprasad
added
Thanks,
Suma Shivaprasad
/solr/atlasConfig/conf/stopwords.txt PRE-CREATION
src/conf/solr/atlasConfig/conf/synonyms.txt PRE-CREATION
Diff: https://reviews.apache.org/r/37285/diff/
Testing
---
All tests pass . No new tests added
Thanks,
Suma Shivaprasad
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/37349/#review94908
---
Ship it!
+1. Thanks for taking this up Shwetha
- Suma Shivaprasad
/TitanGraphProvider.java
(line 63)
<https://reviews.apache.org/r/37349/#comment149592>
Pls add some comments to understand why the change was done. Took a while
to understand :).
- Suma Shivaprasad
On Aug. 11, 2015, 9:32 a.m., Shwetha GS
Hi Nigel,
You should be able to point to the graph created in ATLAS through Rexster
by following the steps at
http://s3.thinkaurelius.com/docs/titan/current/server.html#_downloading_rexster_and_titan_separately.
I was able to get the Rexster instance up this way but got some issue while
loading th
conf pointed out in the above link and point to the directory where the
data for them resides.
Connecting through the gremlin shell also requires a .properties file with
all these properties set to load the graph stored by ATLAS.
Thanks
Suma
On Fri, Sep 4, 2015 at 5:47 PM, Suma Shivaprasad
Thanks,
Suma Shivaprasad
can also be removed to use existing interfaces
- Suma Shivaprasad
On Sept. 14, 2015, 4:08 a.m., Tom Beerbower wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://re
/HiveMetaStoreBridge.java
(line 277)
<https://reviews.apache.org/r/38393/#comment156363>
is sdQualifiedName supposed to be same as tableQualifiedName ?
- Suma Shivaprasad
On Sept. 15, 2015, 6:56 a.m., Shwetha GS wrote:
>
> -
g/r/38393/#comment156374>
we should make rge readTimeout configurable and also add connectTimeout
common/src/main/java/org/apache/atlas/service/Services.java (line 38)
<https://reviews.apache.org/r/38393/#comment156375>
should we add a register and unregister service met
tps://reviews.apache.org/r/38393/#comment156385>
Pls add docs for these configs
- Suma Shivaprasad
On Sept. 15, 2015, 6:56 a.m., Shwetha GS wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://re
/KafkaNotification.java (line
131)
<https://reviews.apache.org/r/38393/#comment156387>
should we throw and exceptiopn instead?
- Suma Shivaprasad
On Sept. 15, 2015, 6:56 a.m., Shwetha GS wrote:
>
> ---
> This is an automatically gener
attrname is same as i.name in this case. So shoudnt be an issue
Thanks
Suma
On Wed, Sep 16, 2015 at 1:20 PM, ltf wrote:
> Hello,
> In the following code, for ENUM, the key for fieldPos is i.name, but
> others are attrName.
> Is it a issue??? Thanks.
>
>
>
>
> file : org/apache/atlas/typesystem/
ent156738>
we should include the typedefinition as well in the exception?
repository/src/main/java/org/apache/atlas/services/MetadataService.java (line
83)
<https://reviews.apache.org/r/38393/#comment156739>
pls add java doc for th args to make it clear
- Suma Shivaprasad
tps://reviews.apache.org/r/38393/#comment156745>
pls add docs for modelling unique entities by qualifiedName
typesystem/src/main/resources/application.properties (line 59)
<https://reviews.apache.org/r/38393/#comment156744>
pls add comments for the configs
- Suma Shivaprasad
On Sept. 21
you pls fix the error message in other places where
ValueCnversionException is being thrown curently :) Right now, we dont ge to
know which attribute is causing the issue
- Suma Shivaprasad
On Sept. 21, 2015, 8:13 a.m., Shwe
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38393/#review99954
---
Ship it!
Ship It!
- Suma Shivaprasad
On Sept. 22, 2015, 9:09
Hi Herman,
You can fire a DSL query in the UI as "hive_table is "trait_name"
Thanks
Suma
On Tue, Sep 22, 2015 at 8:24 AM, Herman Yu wrote:
> Hi,
>
> With version v0.5, is search based on combination of a Trait and a type
> supported? I have both hive_tables and hive_columns associated with a
/atlas/query/QueryTestsUtils.scala
3c66da0
typesystem/src/main/java/org/apache/atlas/typesystem/types/DataTypes.java
e35453e
Diff: https://reviews.apache.org/r/38179/diff/
Testing
---
Tests added in GremlinTest
Thanks,
Suma Shivaprasad
Could be an issue with 0.5. I fixed an issue while querying for entities
with traits as part of https://reviews.apache.org/r/38179/. . Should be
committed soon.
Thanks
Suma
On Wed, Sep 23, 2015 at 8:40 PM, Shwetha Shivalingamurthy <
sshivalingamur...@hortonworks.com> wrote:
> 'hive_table is ¹
ache/atlas/query/QueryTestsUtils.scala
3c66da0
typesystem/src/main/java/org/apache/atlas/typesystem/types/DataTypes.java
e35453e
Diff: https://reviews.apache.org/r/38179/diff/
Testing (updated)
---
Tests added in Parser, GremlinTest, GraphBackedDiscoveryTest
Thanks,
Suma Shivaprasad
Parser, GremlinTest, GraphBackedDiscoveryTest
Thanks,
Suma Shivaprasad
> >>> [ERROR]^
> >>> [ERROR]
> >>>
> >>>/teeup/source_repo/atlas/typesystem/src/main/java/org/apache/atlas/types
> >>>ys
> >>> tem/types/TypeSystem.java:564: illegal start of
itory/src/test/scala/org/apache/atlas/query/QueryTestsUtils.scala
3c66da0
typesystem/src/main/java/org/apache/atlas/typesystem/types/DataTypes.java
e35453e
Diff: https://reviews.apache.org/r/38179/diff/
Testing
---
Tests added in Parser, GremlinTest, GraphBackedDiscoveryTest
Thanks,
Suma Shivaprasad
ated e-mail. To reply, visit:
https://reviews.apache.org/r/38179/#review101105
-------
On Oct. 1, 2015, 6:36 a.m., Suma Shivaprasad wrote:
>
> ---
> This is a
reviews.apache.org/r/38179/diff/
Testing
---
Tests added in Parser, GremlinTest, GraphBackedDiscoveryTest
Thanks,
Suma Shivaprasad
8179/diff/
Testing
---
Tests added in Parser, GremlinTest, GraphBackedDiscoveryTest
Thanks,
Suma Shivaprasad
/KafkaNotification.java (line
223)
<https://reviews.apache.org/r/39157/#comment159590>
consumerConnector need to be shutdown
- Suma Shivaprasad
On Oct. 9, 2015, 1:40 a.m., Tom Beerbower wrote:
>
> ---
> This is an automatically gener
> On Oct. 9, 2015, 4 p.m., Suma Shivaprasad wrote:
> > notification/src/main/java/org/apache/atlas/kafka/KafkaNotification.java,
> > line 226
> > <https://reviews.apache.org/r/39157/diff/1/?file=1093468#file1093468line226>
> >
> > consumerConnecto
care of?
- Suma Shivaprasad
On Oct. 9, 2015, 1:40 a.m., Tom Beerbower wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39157/#review102184
---
Ship it!
Ship It!
- Suma Shivaprasad
On Oct. 9, 2015, 1:40 a.m
HI Rishabh,
Can you pls do a clean checkout and try to build. There are some runtime
files generated in dahsboard which was part of the older build process but
the rat ignore is removed for them
Thanks
Suma
On Mon, Oct 12, 2015 at 10:33 AM, Rishabh Bhardwaj
wrote:
> Hi,
> I am new to atlas a
Hi Folks,
I have created a build for Apache Atlas 0.6-incubating, release candidate
0. Thanks to everyone who have contributed to this release and also tested
this.
The source tarball (*.tar.gz), signature (*.asc), checksum (*.md5, *.sha):
>
https://dist.apache.org/repos/dist/dev/incubator/atlas
/0.6.0-incubating-rc0/apache-atlas-0.6-incubating-sources.tar.gz
>
>
> On Tue, Dec 8, 2015 at 9:26 AM, Suma Shivaprasad
> wrote:
>
> > Hi Folks,
> >
> > I have created a build for Apache Atlas 0.6-incubating, release candidate
> > 0. Thanks to everyone who
Canceling release candidate-0 of Apache Atlas 0.6-incubating as there were
issues with the source tar ball.
Thanks
Suma
Hi Folks,
I have created a build for Apache Atlas 0.6-incubating, release candidate
1. Thanks to everyone who have contributed to this release and also tested
this.
The source tarball (*.tar.gz), signature (*.asc), checksum (*.md5, *.sha):
> *https://dist.apache.org/repos/dist/dev/incubator/atla
_start and verified results.
> >
> > I could not run the hive bridge against a local, kerberized Hadoop
> > cluster, but that may be a configuration issue on my side. It failed with
> > kerberos authentication errors. I don¹t think it¹s a blocker, but I am
> > continuing
ory: 173M/726M
[INFO]
Would request someone else to try the same and let me know. If its working
fine, request you to reconsider the vote.
Thanks
Suma
On Thu, Dec 10, 2015 at 1:58 PM, Suma Shivaprasad <
sumasai.shivapra...@gmail.com> wrote:
> Hi all,
>
> Thanks
at 4:31 PM, Sandeep Samudrala <
> sandys...@gmail.com>
> > > wrote:
> > >
> > > > +1.(non-binding)
> > > > Verified build and signatures.
> > > >
> > > > On Thu, Dec 10, 2015 at 3:09 PM, Shwetha G S
> > > wrote
An issue was identified with this release -
https://issues.apache.org/jira/browse/ATLAS-383 and hence canceling the
vote for this release candidate.
Thanks
Suma
Hi Folks,
I have created a build for Apache Atlas 0.6-incubating, release candidate 2.
Thanks to everyone who have contributed to this release and also tested
this.
The source tarball (*.tar.gz), signature (*.asc), checksum (*.md5, *.sha):
> *https://dist.apache.org/repos/dist/dev/incubator/atla
Hi folks,
Thanks for taking time to review and vote on this release. The vote passes
with five binding +1's and one 0, no -1s.
Binding:
IPMC - Arun Murthy, Jitendra Pandey
PPMC - Harish Butani, Shwetha GS, Suma Shivaprasad
I'll start voting on Incubator general.
Thanks!
Regards,
Su
Hello folks,
This is a call for a vote on the Apache Atlas 0.6 incubating release.
A vote was held on developer mailing list and it passed with five +1's.
Vote thread: http://s.apache.org/nxl
Results thread: http://s.apache.org/HyG
The source tarball (*.tar.gz), signature (*.asc), checksum (*.m
o me.
> > On Dec 24, 2015 22:07, "Suma Shivaprasad" wrote:
> >
> > > Hello folks,
> > >
> > > This is a call for a vote on the Apache Atlas 0.6 incubating release.
> > >
> > > A vote was held on developer mailing list and it
401 - 500 of 1928 matches
Mail list logo