----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/73058/#review222281 -----------------------------------------------------------
repository/src/main/java/org/apache/atlas/query/DSLVisitor.java Lines 195 (patched) <https://reviews.apache.org/r/73058/#comment311344> Remove this line. repository/src/main/java/org/apache/atlas/query/GremlinQueryComposer.java Lines 208 (patched) <https://reviews.apache.org/r/73058/#comment311345> This doesn't make senese. repository/src/test/java/org/apache/atlas/query/DSLQueriesTest.java Lines 202 (patched) <https://reviews.apache.org/r/73058/#comment311340> Good! repository/src/test/java/org/apache/atlas/query/GremlinQueryComposerTest.java Line 102 (original), 102 (patched) <https://reviews.apache.org/r/73058/#comment311342> Can you please include a test that verifies the genreated gremlin query? repository/src/test/java/org/apache/atlas/query/GremlinQueryComposerTest.java Line 190 (original), 190 (patched) <https://reviews.apache.org/r/73058/#comment311343> What is the reason for these queries changing? - Ashutosh Mestry On Dec. 3, 2020, 6:05 a.m., Pinal Shah wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/73058/ > ----------------------------------------------------------- > > (Updated Dec. 3, 2020, 6:05 a.m.) > > > Review request for atlas, Ashutosh Mestry, Jayendra Parab, Madhan Neethiraj, > Nixon Rodrigues, and Sarath Subramanian. > > > Bugs: ATLAS-4057 > https://issues.apache.org/jira/browse/ATLAS-4057 > > > Repository: atlas > > > Description > ------- > > Supporting glossary term in advanced search, will allow user to search the > entities which are assigned to particular glossary term. > > > **WorkAround:** > > 'with' keyword is added to support glossary term. > > User needs to add fully qualifiedName of glossary term. > > > > **Example queries to search:** > > hive_table with "modernTrade@salesGlossary", > > hive_table with "modernTrade@salesGlossary" and hive_table isA Dimension > > > Diffs > ----- > > > repository/src/main/java/org/apache/atlas/discovery/TermSearchProcessor.java > 6c48a82f4 > repository/src/main/java/org/apache/atlas/query/DSLVisitor.java 700da955f > repository/src/main/java/org/apache/atlas/query/GremlinQueryComposer.java > 249381059 > repository/src/main/java/org/apache/atlas/query/antlr4/AtlasDSLLexer.g4 > 15d1bb4cb > repository/src/main/java/org/apache/atlas/query/antlr4/AtlasDSLLexer.java > 142b9cac9 > repository/src/main/java/org/apache/atlas/query/antlr4/AtlasDSLParser.g4 > f1c10600c > repository/src/main/java/org/apache/atlas/query/antlr4/AtlasDSLParser.java > 6b33edb53 > > repository/src/main/java/org/apache/atlas/query/antlr4/AtlasDSLParserBaseVisitor.java > 3139d4374 > > repository/src/main/java/org/apache/atlas/query/antlr4/AtlasDSLParserVisitor.java > a1a727c82 > repository/src/test/java/org/apache/atlas/BasicTestSetup.java d733c1642 > > repository/src/test/java/org/apache/atlas/discovery/AtlasDiscoveryServiceTest.java > 1839e76c7 > repository/src/test/java/org/apache/atlas/query/DSLQueriesTest.java > 3bb3b07bf > > repository/src/test/java/org/apache/atlas/query/GremlinQueryComposerTest.java > 487cf2778 > > > Diff: https://reviews.apache.org/r/73058/diff/4/ > > > Testing > ------- > > Added testcases in DSLQueriesTest.java > Precommit : > https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/245/ > > > Thanks, > > Pinal Shah > >