----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/72474/#review220723 -----------------------------------------------------------
Fix it, then Ship it! repository/src/main/java/org/apache/atlas/query/RegistryBasedLookup.java Lines 127 (patched) <https://reviews.apache.org/r/72474/#comment309398> consider renaming the variable: et => entityType ret => attribute direction => ret (return value should be ret to avoid confusion) - Sarath Subramanian On May 12, 2020, 6:11 a.m., chaitali wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/72474/ > ----------------------------------------------------------- > > (Updated May 12, 2020, 6:11 a.m.) > > > Review request for atlas, Ashutosh Mestry, Nikhil Bonte, Nixon Rodrigues, and > Sarath Subramanian. > > > Bugs: ATLAS-3783 > https://issues.apache.org/jira/browse/ATLAS-3783 > > > Repository: atlas > > > Description > ------- > > Unable to get results for Dsl query eg;hive_column where > table.name="employee". > Due to regression only one edge direction was giving results. > This patch gives a fix for results for both relationship edge direction > queries. > > > Diffs > ----- > > repository/src/main/java/org/apache/atlas/query/GremlinQueryComposer.java > 04b177533 > repository/src/main/java/org/apache/atlas/query/IdentifierHelper.java > 6ab61e16b > repository/src/main/java/org/apache/atlas/query/Lookup.java 3c192d824 > repository/src/main/java/org/apache/atlas/query/RegistryBasedLookup.java > 2b34abff8 > repository/src/test/java/org/apache/atlas/query/DSLQueriesTest.java > 0bbff2f46 > > repository/src/test/java/org/apache/atlas/query/GremlinQueryComposerTest.java > 6f4df88f0 > > > Diff: https://reviews.apache.org/r/72474/diff/9/ > > > Testing > ------- > > TestCases added. > > > Thanks, > > chaitali > >