Mailing-List: contact dev-help@atlas.incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@atlas.incubator.apache.org
Content-Type: multipart/alternative;
 boundary="===============3298031610485785251=="
MIME-Version: 1.0
Subject: Re: Review Request 48533: Business Catalog Update
From: Hemanth Yamijala <yhemanth@gmail.com>
To: Hemanth Yamijala <yhemanth@gmail.com>
Cc: John Speidel <jspeidel@hortonworks.com>,
 atlas <dev@atlas.incubator.apache.org>
Date: Sat, 11 Jun 2016 06:35:37 -0000
Message-ID: <20160611063537.1707.83044@reviews.apache.org>
Auto-Submitted: auto-generated
Sender: Hemanth Yamijala <noreply@reviews.apache.org>
References: <20160610054232.12907.27926@reviews.apache.org>
In-Reply-To: <20160610054232.12907.27926@reviews.apache.org>
Reply-To: Hemanth Yamijala <yhemanth@gmail.com>
archived-at: Sat, 11 Jun 2016 06:35:45 -0000

--===============3298031610485785251==
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Transfer-Encoding: 7bit


> On June 10, 2016, 5:42 a.m., Hemanth Yamijala wrote:
> > catalog/src/main/java/org/apache/atlas/catalog/TermResourceProvider.java, line 122
> > <https://reviews.apache.org/r/48533/diff/1/?file=1414341#file1414341line122>
> >
> >     If this condition does not hold, should we even execute the term query? IOW, can we move the check upwards before the term query as well.
> 
> John Speidel wrote:
>     This is actually the correct behavior. Because the properties of a term are 'administrative' type properties, they are only set on the instance associated with the Taxonomy (singleton instance). The exception to this is the 'description' property which is also set on the tags(trait instances associated with entities) as it makes sense in the tag context. So, a user can update any property of a term such as "available_as_tag" and because tags only have the "name" and "description" properties set, the property should only be set on the Taxonomy instance.

Thanks for the explanation. I understand now. However, in the UI for the tag instances associated to an entity we are showing the default values of the other administrative attributes like 'available_as_tag' etc. We show the default values according to their types. This is probably because of how the type system ends up showing default values for attributes whether they are defined or not. This can be taken up as a separate issue, I think.


- Hemanth


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/48533/#review136966
-----------------------------------------------------------


On June 10, 2016, 9:09 p.m., John Speidel wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/48533/
> -----------------------------------------------------------
> 
> (Updated June 10, 2016, 9:09 p.m.)
> 
> 
> Review request for atlas and Hemanth Yamijala.
> 
> 
> Bugs: ATLAS-794
>     https://issues.apache.org/jira/browse/ATLAS-794
> 
> 
> Repository: atlas
> 
> 
> Description
> -------
> 
> Allows for updating of taxonomy and term property values, excluding the name property.
> 
> 
> Diffs
> -----
> 
>   catalog/src/main/java/org/apache/atlas/catalog/AtlasTypeSystem.java 3a58488 
>   catalog/src/main/java/org/apache/atlas/catalog/BaseRequest.java 9ccb4e3 
>   catalog/src/main/java/org/apache/atlas/catalog/BaseResourceProvider.java ad2f7f4 
>   catalog/src/main/java/org/apache/atlas/catalog/CollectionRequest.java b1be1ae 
>   catalog/src/main/java/org/apache/atlas/catalog/DefaultTypeSystem.java a28a32b 
>   catalog/src/main/java/org/apache/atlas/catalog/EntityResourceProvider.java c8d6f68 
>   catalog/src/main/java/org/apache/atlas/catalog/EntityTagResourceProvider.java f73f80b 
>   catalog/src/main/java/org/apache/atlas/catalog/InstanceRequest.java 01583c4 
>   catalog/src/main/java/org/apache/atlas/catalog/Request.java 7dc781a 
>   catalog/src/main/java/org/apache/atlas/catalog/ResourceProvider.java 9c809c0 
>   catalog/src/main/java/org/apache/atlas/catalog/TaxonomyResourceProvider.java 0d63336 
>   catalog/src/main/java/org/apache/atlas/catalog/TermResourceProvider.java 0c72de6 
>   catalog/src/main/java/org/apache/atlas/catalog/VertexWrapper.java 6e5d28e 
>   catalog/src/main/java/org/apache/atlas/catalog/definition/BaseResourceDefinition.java 32d6f30 
>   catalog/src/main/java/org/apache/atlas/catalog/definition/EntityResourceDefinition.java cf55f1f 
>   catalog/src/main/java/org/apache/atlas/catalog/definition/ResourceDefinition.java f310c5a 
>   catalog/src/main/java/org/apache/atlas/catalog/definition/TaxonomyResourceDefinition.java a3fbdf1 
>   catalog/src/main/java/org/apache/atlas/catalog/definition/TermResourceDefinition.java 19dd049 
>   catalog/src/main/java/org/apache/atlas/catalog/query/AtlasEntityQuery.java c24b99a 
>   catalog/src/main/java/org/apache/atlas/catalog/query/AtlasEntityTagQuery.java df216c0 
>   catalog/src/main/java/org/apache/atlas/catalog/query/AtlasQuery.java af14697 
>   catalog/src/main/java/org/apache/atlas/catalog/query/AtlasTermQuery.java b761dcc 
>   catalog/src/main/java/org/apache/atlas/catalog/query/BaseQuery.java ba8e0e7 
>   catalog/src/main/java/org/apache/atlas/catalog/query/QueryFactory.java 39ce11a 
>   catalog/src/test/java/org/apache/atlas/catalog/CollectionRequestTest.java 0a2bace 
>   catalog/src/test/java/org/apache/atlas/catalog/EntityResourceProviderTest.java 2f29103 
>   catalog/src/test/java/org/apache/atlas/catalog/EntityTagResourceProviderTest.java 78204a6 
>   catalog/src/test/java/org/apache/atlas/catalog/InstanceRequestTest.java 5ccec02 
>   catalog/src/test/java/org/apache/atlas/catalog/TaxonomyResourceProviderTest.java a714a8c 
>   catalog/src/test/java/org/apache/atlas/catalog/TermResourceProviderTest.java 235bde4 
>   catalog/src/test/java/org/apache/atlas/catalog/VertexWrapperTest.java 64935a6 
>   catalog/src/test/java/org/apache/atlas/catalog/definition/EntityResourceDefinitionTest.java 303e2ba 
>   catalog/src/test/java/org/apache/atlas/catalog/definition/EntityTagResourceDefinitionTest.java 954262f 
>   catalog/src/test/java/org/apache/atlas/catalog/definition/TaxonomyResourceDefinitionTest.java bc6f74c 
>   catalog/src/test/java/org/apache/atlas/catalog/definition/TermResourceDefinitionTest.java 52deadf 
>   catalog/src/test/java/org/apache/atlas/catalog/query/AtlasEntityQueryTest.java 149134c 
>   webapp/src/main/java/org/apache/atlas/web/resources/BaseService.java 2a65538 
>   webapp/src/main/java/org/apache/atlas/web/resources/TaxonomyService.java f995198 
>   webapp/src/test/java/org/apache/atlas/web/resources/TaxonomyServiceTest.java a967805 
> 
> Diff: https://reviews.apache.org/r/48533/diff/
> 
> 
> Testing
> -------
> 
> Added new unit tests.
> All unit tests pass.
> Manual functional tests including create/update/delete/get of taxonomies/terms/tags.
> 
> 
> Thanks,
> 
> John Speidel
> 
>


--===============3298031610485785251==--