-
Notifications
You must be signed in to change notification settings - Fork 62
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
Update metrics.py, esquery.py and test_esquery.py to use only elasticsearch_dsl module #60
Comments
aswanipranjal
changed the title
Update metrics.py and test_esquery.py to use only elasticsearch_dsl module
Update metrics.py, esquery.py and test_esquery.py to use only elasticsearch_dsl module
May 27, 2018
aswanipranjal
added a commit
to aswanipranjal/grimoirelab-manuscripts
that referenced
this issue
May 27, 2018
aswanipranjal
added a commit
to aswanipranjal/grimoirelab-manuscripts
that referenced
this issue
Jun 3, 2018
aswanipranjal
added a commit
to aswanipranjal/grimoirelab-manuscripts
that referenced
this issue
Jun 3, 2018
aswanipranjal
added a commit
to aswanipranjal/grimoirelab-manuscripts
that referenced
this issue
Jun 4, 2018
aswanipranjal
added a commit
to aswanipranjal/grimoirelab-manuscripts
that referenced
this issue
Jun 7, 2018
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
After #57, this issue adds the use of elasticsearch_dsl to other files such as metrics.py.
esquery.py
in the functionget_aggs
, returns a json dict which is then used in metrics.py to get the results viarequests.get
method.This issue is about removing completely the direct querying to elasticsearch and using elasticsearch_dsl.py instead.
EDIT: The second PR is ready, Please review #58 so that the second PR can be added on top of it.
The text was updated successfully, but these errors were encountered: