Re: Review Request 49956: ATLAS-1010 Atlas allows recreation of tags with same name

2016-07-12 Thread David Kantor

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



PLease hold off on committing this, I need more time to review.  The fact that 
it breaks StoreBackedTypeCache is a concern.

- David Kantor


On July 12, 2016, 4:11 p.m., Shwetha GS wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49956/
> ---
> 
> (Updated July 12, 2016, 4:11 p.m.)
> 
> 
> Review request for atlas.
> 
> 
> Bugs: ATLAS-1010
> https://issues.apache.org/jira/browse/ATLAS-1010
> 
> 
> Repository: atlas
> 
> 
> Description
> ---
> 
> Transient type system which validates the new types being added, wasn't 
> checking against the singleton instance
> 
> 
> Diffs
> -
> 
>   
> repository/src/test/java/org/apache/atlas/repository/typestore/StoreBackedTypeCacheTest.java
>  ee4cc28 
>   
> repository/src/test/java/org/apache/atlas/service/StoreBackedTypeCacheMetadataServiceTest.java
>  4cb5c67 
>   typesystem/src/main/java/org/apache/atlas/typesystem/types/TypeSystem.java 
> c355ba9 
>   typesystem/src/test/java/org/apache/atlas/typesystem/types/EnumTest.java 
> adbeb19 
>   
> typesystem/src/test/java/org/apache/atlas/typesystem/types/TypeSystemTest.java
>  574e0f9 
>   
> webapp/src/test/java/org/apache/atlas/web/resources/TypesJerseyResourceIT.java
>  7675195 
> 
> Diff: https://reviews.apache.org/r/49956/diff/
> 
> 
> Testing
> ---
> 
> UT, IT added
> 
> 
> Thanks,
> 
> Shwetha GS
> 
>



Re: Review Request 49956: ATLAS-1010 Atlas allows recreation of tags with same name

2016-07-12 Thread Madhan Neethiraj

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


Ship it!




Ship It!

- Madhan Neethiraj


On July 12, 2016, 4:11 p.m., Shwetha GS wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/49956/
> ---
> 
> (Updated July 12, 2016, 4:11 p.m.)
> 
> 
> Review request for atlas.
> 
> 
> Bugs: ATLAS-1010
> https://issues.apache.org/jira/browse/ATLAS-1010
> 
> 
> Repository: atlas
> 
> 
> Description
> ---
> 
> Transient type system which validates the new types being added, wasn't 
> checking against the singleton instance
> 
> 
> Diffs
> -
> 
>   
> repository/src/test/java/org/apache/atlas/repository/typestore/StoreBackedTypeCacheTest.java
>  ee4cc28 
>   
> repository/src/test/java/org/apache/atlas/service/StoreBackedTypeCacheMetadataServiceTest.java
>  4cb5c67 
>   typesystem/src/main/java/org/apache/atlas/typesystem/types/TypeSystem.java 
> c355ba9 
>   typesystem/src/test/java/org/apache/atlas/typesystem/types/EnumTest.java 
> adbeb19 
>   
> typesystem/src/test/java/org/apache/atlas/typesystem/types/TypeSystemTest.java
>  574e0f9 
>   
> webapp/src/test/java/org/apache/atlas/web/resources/TypesJerseyResourceIT.java
>  7675195 
> 
> Diff: https://reviews.apache.org/r/49956/diff/
> 
> 
> Testing
> ---
> 
> UT, IT added
> 
> 
> Thanks,
> 
> Shwetha GS
> 
>