----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/74925/#review226403 -----------------------------------------------------------
intg/src/main/java/org/apache/atlas/type/AtlasStructType.java Line 1078 (original), 1078 (patched) <https://reviews.apache.org/r/74925/#comment314616> This change doesn't seem correct. Did you mean the following: if (StringUtils.isNotBlank(value)) { ... } intg/src/main/java/org/apache/atlas/type/AtlasStructType.java Lines 1083 (patched) <https://reviews.apache.org/r/74925/#comment314617> All CJK characters are treated as tokenizeChar here. While hastokenizeChar() only treats specific characters as tokenizeChar. Consider adding unit tests for hastokenizeChar(String) with several inputs. - Madhan Neethiraj On April 26, 2024, 9:54 a.m., chaitali wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/74925/ > ----------------------------------------------------------- > > (Updated April 26, 2024, 9:54 a.m.) > > > Review request for atlas, Ashutosh Mestry, Deep Singh, Jayendra Parab, Pinal > Shah, and Sheetal Shah. > > > Bugs: ATLAS-4225 > https://issues.apache.org/jira/browse/ATLAS-4225 > > > Repository: atlas > > > Description > ------- > > Updated latest patch with synchronized code > > Creating this review request to track and close > https://reviews.apache.org/r/73256/diff/3#1 as the patch here is not in sync > with latest code > > > Diffs > ----- > > intg/src/main/java/org/apache/atlas/type/AtlasStructType.java 591b31612 > > repository/src/main/java/org/apache/atlas/discovery/EntityDiscoveryService.java > 9be6517e9 > repository/src/main/java/org/apache/atlas/discovery/SearchContext.java > 9a9a5512a > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/EntityGraphMapper.java > 01b88eb2f > repository/src/test/java/org/apache/atlas/BasicTestSetup.java 593f8c19b > > repository/src/test/java/org/apache/atlas/discovery/AtlasDiscoveryServiceTest.java > 282a3fd4a > > repository/src/test/java/org/apache/atlas/discovery/ClassificationSearchProcessorTest.java > 121dca9f5 > > repository/src/test/java/org/apache/atlas/discovery/EntitySearchProcessorTest.java > 3f9e74d11 > > > Diff: https://reviews.apache.org/r/74925/diff/4/ > > > Testing > ------- > > Testcases are added in the patch > PC is successful : > https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/1561/consoleFull > > > Thanks, > > chaitali > >
