atlas-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ashutosh Mestry <ames...@hortonworks.com>
Subject Review Request 57352: Case where Tag with Date Attribute was not usable
Date Tue, 07 Mar 2017 01:27:13 GMT

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

Review request for atlas and Madhan Neethiraj.


Bugs: ATLAS-1646
    https://issues.apache.org/jira/browse/ATLAS-1646


Repository: atlas


Description
-------

Steps to duplicate:
- Create a Tag with date attribute.
- Select *hive_table* and attempt to assign this new attribute to it. You will receive an
error.
- Apply the patch and duplicate the steps above. Tag will get associated.

Implementation approach:
- Update *AtlasDateType* to include empty string as valid value.
- Added classification validation to *AtlasEntityStoreV1.addClassification* methods. Handled
invalid cases.


*CURL*
```
curl -X GET -u admin:admin -H "Content-Type: application/json" -H "Cache-Control: no-cache"
"http://localhost:21000/api/atlas/entities/77cc9adb-7608-4f13-b94a-eaee3b056884"
```


Diffs
-----

  intg/src/main/java/org/apache/atlas/type/AtlasBuiltInTypes.java 0b124b2 
  intg/src/test/java/org/apache/atlas/type/TestAtlasDateType.java 1b3bbc7 
  repository/src/main/java/org/apache/atlas/repository/store/graph/v1/AtlasEntityStoreV1.java
518b52b 


Diff: https://reviews.apache.org/r/57352/diff/1/


Testing
-------

Updated unit tests.
Executed unit tests.


Thanks,

Ashutosh Mestry


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message