> On May 11, 2020, 6:25 a.m., Sarath Subramanian wrote:
> > repository/src/main/java/org/apache/atlas/query/IdentifierHelper.java
> > Line 160 (original), 163 (patched)
> > <https://reviews.apache.org/r/72474/diff/6/?file=2231084#file2231084line163>
> >
> > why change ex.getMessage()? revert if not intended.
>
> chaitali wrote:
> While running testcases no valid error was thrown when testcase caught
> null which made it complicated to trace the error
> .So i have added this error message to notify
org.apache.atlas.exception.AtlasBaseException: Invalid DSL query: hive_db as d
select d | Reason: . Please refer to Atlas DSL grammar for more information
at
org.apache.atlas.query.AtlasDSL$Translator.processErrorList(AtlasDSL.java:143)
at
org.apache.atlas.query.AtlasDSL$Translator.translate(AtlasDSL.java:130)
at
org.apache.atlas.discovery.EntityDiscoveryService.toGremlinQuery(EntityDiscoveryService.java:736)
org.apache.atlas.exception.AtlasBaseException: Invalid DSL query: hive_db as d
select d | Reason: NullPointerException. Please refer to Atlas DSL grammar for
more information
at
org.apache.atlas.query.AtlasDSL$Translator.processErrorList(AtlasDSL.java:143)
at
org.apache.atlas.query.AtlasDSL$Translator.translate(AtlasDSL.java:130)
at
org.apache.atlas.discovery.EntityDiscoveryService.toGremlinQuery(EntityDiscoveryService.java:736)
- chaitali
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/72474/#review220702
-----------------------------------------------------------
On May 11, 2020, 9:24 a.m., chaitali wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/72474/
> -----------------------------------------------------------
>
> (Updated May 11, 2020, 9:24 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/7/
>
>
> Testing
> -------
>
> TestCases added.
>
>
> Thanks,
>
> chaitali
>
>