-
Notifications
You must be signed in to change notification settings - Fork 63
enable FT.TAGVALS command #94
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Codecov Report
@@ Coverage Diff @@
## master #94 +/- ##
==========================================
+ Coverage 91.35% 91.39% +0.04%
==========================================
Files 12 12
Lines 1584 1593 +9
==========================================
+ Hits 1447 1456 +9
Misses 137 137
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank for the contribution. It looks great!
Just minor comments.
@@ -617,6 +619,11 @@ def testTags(self): | |||
res = client.search(q) | |||
self.assertEqual(1, res.total) | |||
|
|||
q2 = client.tagvals('tags') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's add another document with additional words and see that we get the combined list.
Kudos, SonarCloud Quality Gate passed!
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great.
Thanks
add a new function to client which takes the name of a tagfield and runs the following command
Returning a list of tags.
Test is included in test/test.py