----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/65659/#review197597 -----------------------------------------------------------
Ship it! Ship It! - Madhan Neethiraj On Feb. 14, 2018, 10:12 p.m., Ashutosh Mestry wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/65659/ > ----------------------------------------------------------- > > (Updated Feb. 14, 2018, 10:12 p.m.) > > > Review request for atlas and Apoorv Naik. > > > Bugs: ATLAS-2447 > https://issues.apache.org/jira/browse/ATLAS-2447 > > > Repository: atlas > > > Description > ------- > > **Background** > Within _GremlinQueryComposer_ the gremlin query construction used a > combination of _limit_ and _limit-local_ for limiting results. This caused > some scenarios to fail. See bug for details. > > Also, if there was any exception within the Gremlin script engine, it would > not display the exact cause of the failure. This caused users not to know > the exact problem with the constructed query. > > **Approach** > * _GremlinClause_: Replaced _limit-local_ with _dedup_. > * Added exception handling to display cause of exception if available. > > > Diffs > ----- > > > repository/src/main/java/org/apache/atlas/discovery/EntityDiscoveryService.java > 4a18ed13 > repository/src/main/java/org/apache/atlas/query/GremlinClause.java c770f77e > repository/src/main/java/org/apache/atlas/query/GremlinQueryComposer.java > 92029f56 > repository/src/test/java/org/apache/atlas/query/DSLQueriesTest.java > 82b73812 > > repository/src/test/java/org/apache/atlas/query/GremlinQueryComposerTest.java > 85addc26 > > > Diff: https://reviews.apache.org/r/65659/diff/1/ > > > Testing > ------- > > **Unit tests** > - Additional tests added to verify error scenarios when comparing floats with > values larger than max float. > - Updated unit tests to reflect the new _dedup_ usage within the query > composition. > > **Functional tests** > - Executed scenarios to view new error messages. > > > Thanks, > > Ashutosh Mestry > >