> On April 1, 2021, 11:12 p.m., Sarath Subramanian wrote: > > > > Sarath Subramanian wrote: > Add Unit test coverage with example of circular relationship import.
Added test coverage for this - Sidharth ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73218/#review222766 ----------------------------------------------------------- On April 2, 2021, 6:49 p.m., Sidharth Mishra wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73218/ > ----------------------------------------------------------- > > (Updated April 2, 2021, 6:49 p.m.) > > > Review request for atlas, Ashutosh Mestry, Deep Singh, Madhan Neethiraj, > Radhika Kundam, and Sarath Subramanian. > > > Bugs: ATLAS-4191 > https://issues.apache.org/jira/browse/ATLAS-4191 > > > Repository: atlas > > > Description > ------- > > Creation of relations does not work for glossary import > > > Diffs > ----- > > intg/src/main/java/org/apache/atlas/AtlasErrorCode.java 884f81fd0 > repository/src/main/java/org/apache/atlas/glossary/GlossaryService.java > d1567009c > repository/src/main/java/org/apache/atlas/glossary/GlossaryTermUtils.java > 647dd3bef > > repository/src/main/java/org/apache/atlas/repository/store/graph/v2/AtlasEntityStoreV2.java > 9f143d494 > repository/src/main/java/org/apache/atlas/util/FileUtils.java 57df9abeb > repository/src/test/java/org/apache/atlas/glossary/GlossaryServiceTest.java > 371b9420c > > repository/src/test/resources/csvFiles/template_with_circular_relationship.csv > PRE-CREATION > repository/src/test/resources/excelFiles/invalid_xls.xls PRE-CREATION > > > Diff: https://reviews.apache.org/r/73218/diff/3/ > > > Testing > ------- > > Manually tested importing the glossary with circular reference > > > Thanks, > > Sidharth Mishra > >