-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/72172/
-----------------------------------------------------------
(Updated March 3, 2020, 3:04 p.m.)
Review request for atlas, Ashutosh Mestry, Madhan Neethiraj, Nixon Rodrigues,
and Sarath Subramanian.
Changes
-------
Moved isValidLength() in AtlasNamespaceAttribute.
Added Max length check for Multi-valued String Namespace attributes.
Changes in AtlasNamespaceDefStoreV2Test to handle mandatory field maxStrLength
empty/null value
Bugs: ATLAS-3632
https://issues.apache.org/jira/browse/ATLAS-3632
Repository: atlas
Description
-------
Added check for length limit of String value
Diffs (updated)
-----
intg/src/main/java/org/apache/atlas/type/AtlasNamespaceType.java ede8443
repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasEntityStoreV2.java
30f5e5a
repository/src/test/java/org/apache/atlas/repository/store/graph/v2/AtlasNamespaceDefStoreV2Test.java
e2f5c16
Diff: https://reviews.apache.org/r/72172/diff/2/
Changes: https://reviews.apache.org/r/72172/diff/1-2/
Testing (updated)
-------
Pre-commit build:
https://builds.apache.org/job/PreCommit-ATLAS-Build-Test/1698/console
Manul testing done by adding values beyong maximum length for String namespace
attributes.
Done same for String multi-valued namespace attributes.
Thanks,
Mandar Ambawane
