----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73442/#review223234 -----------------------------------------------------------
Fix it, then Ship it! Fix & Ship repository/src/main/java/org/apache/atlas/services/MetricsService.java Lines 248 (patched) <https://reviews.apache.org/r/73442/#comment312326> CollectionUtils.isNotEmpty(entityDef.getSuperTypes()) I think this check is not required, please review & remove. - Nikhil Bonte On July 9, 2021, 1:11 p.m., Mandar Ambawane wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73442/ > ----------------------------------------------------------- > > (Updated July 9, 2021, 1:11 p.m.) > > > Review request for atlas, Ashutosh Mestry, Jayendra Parab, Nixon Rodrigues, > and Sarath Subramanian. > > > Bugs: ATLAS-4150 > https://issues.apache.org/jira/browse/ATLAS-4150 > > > Repository: atlas > > > Description > ------- > > This patch checks all the Super Types of an Entity Type. If any of the Super > Types is __internal > > Then it will not allow that Entity Type in Metrics api response. > > > Diffs > ----- > > repository/src/main/java/org/apache/atlas/services/MetricsService.java > e721108d3 > repository/src/test/java/org/apache/atlas/services/MetricsServiceTest.java > bea8eb8b5 > > > Diff: https://reviews.apache.org/r/73442/diff/1/ > > > Testing > ------- > > PreCommit: > https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/734/consoleFull > > > Thanks, > > Mandar Ambawane > >