You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@atlas.apache.org by Pinal Shah <pi...@freestoneinfotech.com> on 2020/12/14 09:56:31 UTC
Re: Review Request 73058: ATLAS-4057 : DSL Search : Support glossary
terms and relationship
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73058/
-----------------------------------------------------------
(Updated Dec. 14, 2020, 9:56 a.m.)
Review request for atlas, Ashutosh Mestry, Jayendra Parab, Madhan Neethiraj, Nixon Rodrigues, and Sarath Subramanian.
Summary (updated)
-----------------
ATLAS-4057 : DSL Search : Support glossary terms and relationship
Bugs: ATLAS-4057
https://issues.apache.org/jira/browse/ATLAS-4057
Repository: atlas
Description (updated)
-------
Supporting glossary term in advanced search, will allow user to search the entities which are assigned to particular glossary term.
**WorkAround:**
'hasTerm' keyword is added to support glossary term.
User needs to add fully qualifiedName of glossary term.
**Example queries to search with term:**
hive_table hasTerm "modernTrade@salesGlossary",
hive_table hasTerm "modernTrade@salesGlossary" and hive_table isA Dimension
**Example queries to search with relationship:**
hive_table where db.name = "sales"
Diffs (updated)
-----
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/AtlasDSLParserListener.java PRE-CREATION
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/6/
Changes: https://reviews.apache.org/r/73058/diff/5-6/
Testing
-------
Added testcases in DSLQueriesTest.java
Precommit : https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/245/
Thanks,
Pinal Shah
Re: Review Request 73058: ATLAS-4057 : DSL Search : Support glossary
terms and relationship
Posted by Pinal Shah <pi...@freestoneinfotech.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73058/
-----------------------------------------------------------
(Updated Jan. 14, 2021, 6:40 a.m.)
Review request for atlas, Ashutosh Mestry, Jayendra Parab, Madhan Neethiraj, Nixon Rodrigues, and Sarath Subramanian.
Changes
-------
rebase trailing whitespaces
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:**
'hasTerm' keyword is added to support glossary term.
User needs to add fully qualifiedName of glossary term.
**Example queries to search with term:**
hive_table hasTerm "modernTrade@salesGlossary",
hive_table hasTerm "modernTrade@salesGlossary" and hive_table isA Dimension
**Example queries to search with relationship:**
hive_table where db.name = "sales"
Diffs (updated)
-----
docs/src/documents/Search/SearchAdvanced.md 46be14278
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 36b514e84
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/AtlasDSLLexer.tokens PRE-CREATION
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/AtlasDSLParserListener.java PRE-CREATION
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/10/
Changes: https://reviews.apache.org/r/73058/diff/9-10/
Testing
-------
Added testcases in DSLQueriesTest.java
Precommit : https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/268/
Thanks,
Pinal Shah
Re: Review Request 73058: ATLAS-4057 : DSL Search : Support glossary
terms and relationship
Posted by Sarath Subramanian <sa...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73058/#review222468
-----------------------------------------------------------
Ship it!
Ship It!
- Sarath Subramanian
On Jan. 13, 2021, 4:31 a.m., Pinal Shah wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73058/
> -----------------------------------------------------------
>
> (Updated Jan. 13, 2021, 4:31 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:**
>
> 'hasTerm' keyword is added to support glossary term.
>
> User needs to add fully qualifiedName of glossary term.
>
>
>
> **Example queries to search with term:**
>
> hive_table hasTerm "modernTrade@salesGlossary",
>
> hive_table hasTerm "modernTrade@salesGlossary" and hive_table isA Dimension
>
> **Example queries to search with relationship:**
>
> hive_table where db.name = "sales"
>
>
> Diffs
> -----
>
> docs/src/documents/Search/SearchAdvanced.md 46be14278
> 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 36b514e84
> 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/AtlasDSLLexer.tokens PRE-CREATION
> 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/AtlasDSLParserListener.java PRE-CREATION
> 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/9/
>
>
> Testing
> -------
>
> Added testcases in DSLQueriesTest.java
> Precommit : https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/268/
>
>
> Thanks,
>
> Pinal Shah
>
>
Re: Review Request 73058: ATLAS-4057 : DSL Search : Support glossary
terms and relationship
Posted by Pinal Shah <pi...@freestoneinfotech.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73058/
-----------------------------------------------------------
(Updated Jan. 13, 2021, 12:31 p.m.)
Review request for atlas, Ashutosh Mestry, Jayendra Parab, Madhan Neethiraj, Nixon Rodrigues, and Sarath Subramanian.
Changes
-------
added documentation
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:**
'hasTerm' keyword is added to support glossary term.
User needs to add fully qualifiedName of glossary term.
**Example queries to search with term:**
hive_table hasTerm "modernTrade@salesGlossary",
hive_table hasTerm "modernTrade@salesGlossary" and hive_table isA Dimension
**Example queries to search with relationship:**
hive_table where db.name = "sales"
Diffs (updated)
-----
docs/src/documents/Search/SearchAdvanced.md 46be14278
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 36b514e84
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/AtlasDSLLexer.tokens PRE-CREATION
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/AtlasDSLParserListener.java PRE-CREATION
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/9/
Changes: https://reviews.apache.org/r/73058/diff/8-9/
Testing
-------
Added testcases in DSLQueriesTest.java
Precommit : https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/268/
Thanks,
Pinal Shah
Re: Review Request 73058: ATLAS-4057 : DSL Search : Support glossary
terms and relationship
Posted by Pinal Shah <pi...@freestoneinfotech.com>.
> On Jan. 12, 2021, 8:20 p.m., Sarath Subramanian wrote:
> > docs/src/documents/Search/SearchAdvanced.md
> > Lines 228 (patched)
> > <https://reviews.apache.org/r/73058/diff/8/?file=2244142#file2244142line228>
> >
> > can we add a new section and examples for relationship search?
okay, i will add new section specifically for relationship
- Pinal
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73058/#review222445
-----------------------------------------------------------
On Jan. 12, 2021, 11:54 a.m., Pinal Shah wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73058/
> -----------------------------------------------------------
>
> (Updated Jan. 12, 2021, 11:54 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:**
>
> 'hasTerm' keyword is added to support glossary term.
>
> User needs to add fully qualifiedName of glossary term.
>
>
>
> **Example queries to search with term:**
>
> hive_table hasTerm "modernTrade@salesGlossary",
>
> hive_table hasTerm "modernTrade@salesGlossary" and hive_table isA Dimension
>
> **Example queries to search with relationship:**
>
> hive_table where db.name = "sales"
>
>
> Diffs
> -----
>
> docs/src/documents/Search/SearchAdvanced.md 46be14278
> 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 36b514e84
> 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/AtlasDSLLexer.tokens PRE-CREATION
> 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/AtlasDSLParserListener.java PRE-CREATION
> 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/8/
>
>
> Testing
> -------
>
> Added testcases in DSLQueriesTest.java
> Precommit : https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/268/
>
>
> Thanks,
>
> Pinal Shah
>
>
Re: Review Request 73058: ATLAS-4057 : DSL Search : Support glossary
terms and relationship
Posted by Sarath Subramanian <sa...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73058/#review222445
-----------------------------------------------------------
docs/src/documents/Search/SearchAdvanced.md
Lines 228 (patched)
<https://reviews.apache.org/r/73058/#comment311500>
can we add a new section and examples for relationship search?
- Sarath Subramanian
On Jan. 12, 2021, 3:54 a.m., Pinal Shah wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73058/
> -----------------------------------------------------------
>
> (Updated Jan. 12, 2021, 3:54 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:**
>
> 'hasTerm' keyword is added to support glossary term.
>
> User needs to add fully qualifiedName of glossary term.
>
>
>
> **Example queries to search with term:**
>
> hive_table hasTerm "modernTrade@salesGlossary",
>
> hive_table hasTerm "modernTrade@salesGlossary" and hive_table isA Dimension
>
> **Example queries to search with relationship:**
>
> hive_table where db.name = "sales"
>
>
> Diffs
> -----
>
> docs/src/documents/Search/SearchAdvanced.md 46be14278
> 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 36b514e84
> 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/AtlasDSLLexer.tokens PRE-CREATION
> 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/AtlasDSLParserListener.java PRE-CREATION
> 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/8/
>
>
> Testing
> -------
>
> Added testcases in DSLQueriesTest.java
> Precommit : https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/268/
>
>
> Thanks,
>
> Pinal Shah
>
>
Re: Review Request 73058: ATLAS-4057 : DSL Search : Support glossary
terms and relationship
Posted by Ashutosh Mestry via Review Board <no...@reviews.apache.org>.
> On Jan. 12, 2021, 5:44 p.m., Ashutosh Mestry wrote:
> > Can you please update documentation with these new keywords?
Nevermind, i see it. Thanks!
- Ashutosh
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73058/#review222443
-----------------------------------------------------------
On Jan. 12, 2021, 11:54 a.m., Pinal Shah wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73058/
> -----------------------------------------------------------
>
> (Updated Jan. 12, 2021, 11:54 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:**
>
> 'hasTerm' keyword is added to support glossary term.
>
> User needs to add fully qualifiedName of glossary term.
>
>
>
> **Example queries to search with term:**
>
> hive_table hasTerm "modernTrade@salesGlossary",
>
> hive_table hasTerm "modernTrade@salesGlossary" and hive_table isA Dimension
>
> **Example queries to search with relationship:**
>
> hive_table where db.name = "sales"
>
>
> Diffs
> -----
>
> docs/src/documents/Search/SearchAdvanced.md 46be14278
> 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 36b514e84
> 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/AtlasDSLLexer.tokens PRE-CREATION
> 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/AtlasDSLParserListener.java PRE-CREATION
> 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/8/
>
>
> Testing
> -------
>
> Added testcases in DSLQueriesTest.java
> Precommit : https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/268/
>
>
> Thanks,
>
> Pinal Shah
>
>
Re: Review Request 73058: ATLAS-4057 : DSL Search : Support glossary
terms and relationship
Posted by Ashutosh Mestry via Review Board <no...@reviews.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73058/#review222443
-----------------------------------------------------------
Can you please update documentation with these new keywords?
- Ashutosh Mestry
On Jan. 12, 2021, 11:54 a.m., Pinal Shah wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73058/
> -----------------------------------------------------------
>
> (Updated Jan. 12, 2021, 11:54 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:**
>
> 'hasTerm' keyword is added to support glossary term.
>
> User needs to add fully qualifiedName of glossary term.
>
>
>
> **Example queries to search with term:**
>
> hive_table hasTerm "modernTrade@salesGlossary",
>
> hive_table hasTerm "modernTrade@salesGlossary" and hive_table isA Dimension
>
> **Example queries to search with relationship:**
>
> hive_table where db.name = "sales"
>
>
> Diffs
> -----
>
> docs/src/documents/Search/SearchAdvanced.md 46be14278
> 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 36b514e84
> 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/AtlasDSLLexer.tokens PRE-CREATION
> 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/AtlasDSLParserListener.java PRE-CREATION
> 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/8/
>
>
> Testing
> -------
>
> Added testcases in DSLQueriesTest.java
> Precommit : https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/268/
>
>
> Thanks,
>
> Pinal Shah
>
>
Re: Review Request 73058: ATLAS-4057 : DSL Search : Support glossary
terms and relationship
Posted by Pinal Shah <pi...@freestoneinfotech.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73058/
-----------------------------------------------------------
(Updated Jan. 12, 2021, 11:54 a.m.)
Review request for atlas, Ashutosh Mestry, Jayendra Parab, Madhan Neethiraj, Nixon Rodrigues, and Sarath Subramanian.
Changes
-------
added doc
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:**
'hasTerm' keyword is added to support glossary term.
User needs to add fully qualifiedName of glossary term.
**Example queries to search with term:**
hive_table hasTerm "modernTrade@salesGlossary",
hive_table hasTerm "modernTrade@salesGlossary" and hive_table isA Dimension
**Example queries to search with relationship:**
hive_table where db.name = "sales"
Diffs (updated)
-----
docs/src/documents/Search/SearchAdvanced.md 46be14278
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 36b514e84
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/AtlasDSLLexer.tokens PRE-CREATION
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/AtlasDSLParserListener.java PRE-CREATION
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/8/
Changes: https://reviews.apache.org/r/73058/diff/7-8/
Testing
-------
Added testcases in DSLQueriesTest.java
Precommit : https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/268/
Thanks,
Pinal Shah
Re: Review Request 73058: ATLAS-4057 : DSL Search : Support glossary
terms and relationship
Posted by Pinal Shah <pi...@freestoneinfotech.com>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73058/
-----------------------------------------------------------
(Updated Jan. 12, 2021, 10:53 a.m.)
Review request for atlas, Ashutosh Mestry, Jayendra Parab, Madhan Neethiraj, Nixon Rodrigues, and Sarath Subramanian.
Changes
-------
addressed - nit: imports
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:**
'hasTerm' keyword is added to support glossary term.
User needs to add fully qualifiedName of glossary term.
**Example queries to search with term:**
hive_table hasTerm "modernTrade@salesGlossary",
hive_table hasTerm "modernTrade@salesGlossary" and hive_table isA Dimension
**Example queries to search with relationship:**
hive_table where db.name = "sales"
Diffs (updated)
-----
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 36b514e84
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/AtlasDSLLexer.tokens PRE-CREATION
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/AtlasDSLParserListener.java PRE-CREATION
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/7/
Changes: https://reviews.apache.org/r/73058/diff/6-7/
Testing
-------
Added testcases in DSLQueriesTest.java
Precommit : https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/268/
Thanks,
Pinal Shah
Re: Review Request 73058: ATLAS-4057 : DSL Search : Support glossary
terms and relationship
Posted by Ashutosh Mestry via Review Board <no...@reviews.apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73058/#review222334
-----------------------------------------------------------
Can you please add a small section on docs?
- Ashutosh Mestry
On Dec. 14, 2020, 9:56 a.m., Pinal Shah wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73058/
> -----------------------------------------------------------
>
> (Updated Dec. 14, 2020, 9:56 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:**
>
> 'hasTerm' keyword is added to support glossary term.
>
> User needs to add fully qualifiedName of glossary term.
>
>
>
> **Example queries to search with term:**
>
> hive_table hasTerm "modernTrade@salesGlossary",
>
> hive_table hasTerm "modernTrade@salesGlossary" and hive_table isA Dimension
>
> **Example queries to search with relationship:**
>
> hive_table where db.name = "sales"
>
>
> 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/AtlasDSLParserListener.java PRE-CREATION
> 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/6/
>
>
> Testing
> -------
>
> Added testcases in DSLQueriesTest.java
> Precommit : https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/268/
>
>
> Thanks,
>
> Pinal Shah
>
>
Re: Review Request 73058: ATLAS-4057 : DSL Search : Support glossary
terms and relationship
Posted by Sarath Subramanian <sa...@apache.org>.
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/73058/#review222439
-----------------------------------------------------------
Fix it, then Ship it!
repository/src/main/java/org/apache/atlas/query/GremlinQueryComposer.java
Lines 28 (patched)
<https://reviews.apache.org/r/73058/#comment311496>
nit: review unused imports and avoid wilcard imports
repository/src/main/java/org/apache/atlas/query/antlr4/AtlasDSLLexer.java
Line 7 (original), 6 (patched)
<https://reviews.apache.org/r/73058/#comment311495>
avoid wildcard imports and review and remove unused imports
- Sarath Subramanian
On Dec. 14, 2020, 1:56 a.m., Pinal Shah wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/73058/
> -----------------------------------------------------------
>
> (Updated Dec. 14, 2020, 1:56 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:**
>
> 'hasTerm' keyword is added to support glossary term.
>
> User needs to add fully qualifiedName of glossary term.
>
>
>
> **Example queries to search with term:**
>
> hive_table hasTerm "modernTrade@salesGlossary",
>
> hive_table hasTerm "modernTrade@salesGlossary" and hive_table isA Dimension
>
> **Example queries to search with relationship:**
>
> hive_table where db.name = "sales"
>
>
> 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/AtlasDSLParserListener.java PRE-CREATION
> 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/6/
>
>
> Testing
> -------
>
> Added testcases in DSLQueriesTest.java
> Precommit : https://ci-builds.apache.org/job/Atlas/job/PreCommit-ATLAS-Build-Test/268/
>
>
> Thanks,
>
> Pinal Shah
>
>