feat: expose /query endpoint to use InfluxQL (#365)#366
Merged
bednar merged 3 commits intoinfluxdata:masterfrom Aug 2, 2022
Merged
feat: expose /query endpoint to use InfluxQL (#365)#366bednar merged 3 commits intoinfluxdata:masterfrom
/query endpoint to use InfluxQL (#365)#366bednar merged 3 commits intoinfluxdata:masterfrom
Conversation
Andy2003
force-pushed
the
feature/gh-356-influxql-endpoint
branch
from
8483830 to
140d9d4
Compare
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #366 +/- ##
============================================
- Coverage 88.91% 88.31% -0.61%
Complexity 778 778
============================================
Files 169 172 +3
Lines 6864 6998 +134
Branches 364 375 +11
============================================
+ Hits 6103 6180 +77
- Misses 653 707 +54
- Partials 108 111 +3 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Contributor
Author
Yes it is |
Andy2003
force-pushed
the
feature/gh-356-influxql-endpoint
branch
from
c09c19c to
5facd17
Compare
Contributor
Author
|
@bednar Is there anything I can do to get this merged? |
Contributor
|
Hi @Andy2003, sorry for the late. We currently discuss how to line up the support of other languages in the our client libraries. I hope so that we will be able to do a review soon and then merge it into Regards |
Andy2003
force-pushed
the
feature/gh-356-influxql-endpoint
branch
from
5facd17 to
7cb475f
Compare
bednar
requested changes
Jul 29, 2022
Contributor
bednar
left a comment
bednar
left a comment
There was a problem hiding this comment.
Thanks again for your PR 👍 and sorry for the late.
There are few requirements which has to be satisfied before we will merge this PR into master:
- Please add an example how to use the
InfluxQLintoexamples/src/main/java/example,examples/README.md
and also satisfy following requirements:
bednar
requested changes
Jul 29, 2022
bednar
reviewed
Aug 2, 2022
bednar
requested changes
Aug 2, 2022
Andy2003
force-pushed
the
feature/gh-356-influxql-endpoint
branch
from
6c387ec to
58c64e0
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #365
Proposed Changes
Added an endpoint to query against the InfluxQL endpoint (v1)
Checklist
mvn testcompletes successfully