fix the conflicts of PR https://github.com/influxdata/influxdb-java/pull/321 #1
Conversation
This file contains 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
QueryResult to Object mapper added
InfluxDBResultMapper was not prepared to handle the 'null' value instad of an empty collection returned by the JSON parser used by InfluxDB client. The null value will be returned whenever a query returns with no result as explained in influxdata/influxdb#7596
…ceived-from-influxdb Fixed NPE when no series is present in QueryResult object
propagate IOExceptions to consumer
…-queryresult-groupby
…ryresult-groupby Added support map queryresult groupby
…ryresult-groupby Added support map queryresult groupby
Fixed checkstyle issues and formatted according to project standard (hopefully)
Added a test which executes a BoundParameterQuery against an actual database.
The checks in BoundParameterQuery have been removed, because InfluxDB does that already and returns appropriate error messages in case of an invalid request.
Added short paragraph in README, which describes the parameter binding for queries. Also added an entry for the parameter binding pull request to the changelog.
Parameter binding in InfluxQL (influxdata#274)
…date from 0.8.0 -> 0.8.1 because only this works with java10
Fixed setAccessible modif. on POJOs and changes are not reversed anymore
… 10 and therefor gain even more test speed
since the build mehtod contains validation for fields emptiness, there should be also way, how to figure out, whether the Builder contains any fields (to prevent the build method from throwing an exception)
allow to figure out, whether the Point.Builder has any fields
Speedup travis build
Update junit from 5.1.0 -> 5.1.0
influxdata#371 Support dynamic measurement name in InfluxDBResultMapper
|
@joelmarty, please consider to merge this PR |
|
due to no response from repo owner, I merged it |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
this PR to resolve the conflicts of the original PR influxdata#321
it also holds the fixes for other build and junit failures made by PR influxdata#321