Skip to content

fix: tags with commas #862

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.

Sign up for GitHub

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

Open
karel-rehor wants to merge 24 commits into
base: hotfix/v6.10.1
Choose a base branch
from
+363 −84
Open

fix: tags with commas #862

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
7c0d602
fix: (WIP) ensure tag keys and values with commas can be parsed corre…
karel-rehor Mar 13, 2026
03c959b
chore: remove unnecessary comma parsing method
karel-rehor Mar 16, 2026
0a5e4e9
chore: add new test asserts, clean and lint code.
karel-rehor Mar 17, 2026
970f47e
chore: remove commented code block
karel-rehor Mar 18, 2026
20269f0
chore: document logic of parseTags method.
karel-rehor Mar 18, 2026
c900bf6
chore: fix lint issue
karel-rehor Mar 18, 2026
10be978
chore: remove key and value lists from parseTags()
karel-rehor Mar 18, 2026
bab2453
tests: add assertion that all tags in series are accounted for
karel-rehor Mar 18, 2026
5b99621
chore: rewrite InfluxQLQueryApiImpl.parseTags to handle escaped equal…
karel-rehor Mar 19, 2026
b2ea118
tests: improve assert and readability of tags test
karel-rehor Mar 19, 2026
219a792
chore: remove needless encapsulation of escaped strings in double quo…
karel-rehor Mar 19, 2026
b4059ba
chore: fix final tag parser logic and clean unused imports in test
karel-rehor Mar 19, 2026
7fe6761
fix: adjust tag parser for escaped escapes and add integration test.
karel-rehor Mar 20, 2026
4213ef8
tests: add malformed and boundary tag test cases and checkstyle fixes.
karel-rehor Mar 23, 2026
e803d04
docs: update CHANGELOG.md and minor test tweak.
karel-rehor Mar 23, 2026
72fab9e
test: remove stale comment regarding returned data schema
karel-rehor Mar 23, 2026
f06f930
docs: minor edit in CHANGELOG.md
karel-rehor Mar 23, 2026
5951261
chore: skip tests completely in CI when generating docs web
alespour Mar 23, 2026
c716eba
chore: update release settings, tags and versions.
karel-rehor Mar 23, 2026
311b9b7
chore: update influxdb-client-java version in karaf poms.
karel-rehor Mar 23, 2026
6e9267f
chore: force latest version for influxdb-client-java in subproject poms.
karel-rehor Mar 23, 2026
3277934
chore: revert express dependencies and example/pom.xml
karel-rehor Mar 23, 2026
af5cb32
chore: in poms - replace version with 6.10.1-SNAPSHOT and tags with H…
karel-rehor Mar 24, 2026
5928b86
chore: update client-version in examples/pom.xml
karel-rehor Mar 24, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .circleci/config.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -200,7 +200,7 @@ jobs:
- maven-cache_v3-<< parameters.maven-image >>-
- run:
name: "Check generate site"
command: mvn clean site site:stage -DskipTests
command: mvn clean site site:stage -Dmaven.test.skip=true

deploy-snapshot:
docker:
Expand Down
9 changes: 9 additions & 0 deletions CHANGELOG.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,3 +1,12 @@
## 6.10.1 [2026-03-24]

### Bug Fixes

1. [#862](https://github.com/influxdata/influxdb-client-java/pull/862): Improved tags parsing.
- Supports InfluxQL queries involving tag keys and values that incorporate escaped special characters such as commas. e.g. (`"my_data,model\,\ uin=Droid\,\ C3PO ..."`)
- This support is stable for InfluxDB Enterprise v1.x databases and InfluxDB v2.x databases using legacy v1 query contexts - e.g (`/query?db=test_data`).
- Influxdb OSS v1.x uses slightly different escape semantics. Use of escaped commas or spaces in tag keys and values with this database can lead to unexpected behavior. Support for special tags with OSS 1.x is not included with this hotfix release.

## 6.10.0 [2023-07-28]

### Bug Fixes
Expand Down
4 changes: 2 additions & 2 deletions client-core/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@
<parent>
<artifactId>influxdb-client</artifactId>
<groupId>com.influxdb</groupId>
<version>6.10.0</version>
<version>6.10.1-SNAPSHOT</version>
</parent>

<artifactId>influxdb-client-core</artifactId>
Expand Down Expand Up @@ -66,7 +66,7 @@
<connection>scm:git:git@github.com:influxdata/influxdb-client-java.git</connection>
<developerConnection>scm:git:git@github.com:influxdata/influxdb-client-java.git</developerConnection>
<url>https://github.com/influxdata/influxdb-client-java/tree/master</url>
<tag>v6.10.0</tag>
<tag>HEAD</tag>
</scm>

<build>
Expand Down
10 changes: 5 additions & 5 deletions client-kotlin/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -303,26 +303,26 @@ The latest version for Maven dependency:
<dependency>
<groupId>com.influxdb</groupId>
<artifactId>influxdb-client-kotlin</artifactId>
<version>6.10.0</version>
<version>6.10.1</version>
</dependency>
```

Or when using with Gradle:
```groovy
dependencies {
implementation "com.influxdb:influxdb-client-kotlin:6.10.0"
implementation "com.influxdb:influxdb-client-kotlin:6.10.1"
}
```

### Snapshot Repository
The snapshots are deployed into [OSS Snapshot repository](https://oss.sonatype.org/content/repositories/snapshots/).
The snapshots are deployed into [OSS Snapshot repository](https://central.sonatype.com/repository/maven-snapshots/).

#### Maven
```xml
<repository>
<id>ossrh</id>
<name>OSS Snapshot repository</name>
<url>https://oss.sonatype.org/content/repositories/snapshots/</url>
<url>https://central.sonatype.com/repository/maven-snapshots/</url>
<releases>
<enabled>false</enabled>
</releases>
Expand All @@ -334,6 +334,6 @@ The snapshots are deployed into [OSS Snapshot repository](https://oss.sonatype.o
#### Gradle
```
repositories {
maven { url "https://oss.sonatype.org/content/repositories/snapshots" }
maven { url "https://central.sonatype.com/repository/maven-snapshots" }
}
```
4 changes: 2 additions & 2 deletions client-kotlin/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
<parent>
<artifactId>influxdb-client</artifactId>
<groupId>com.influxdb</groupId>
<version>6.10.0</version>
<version>6.10.1-SNAPSHOT</version>
</parent>
<modelVersion>4.0.0</modelVersion>

Expand Down Expand Up @@ -72,7 +72,7 @@
<connection>scm:git:git@github.com:influxdata/influxdb-client-java.git</connection>
<developerConnection>scm:git:git@github.com:influxdata/influxdb-client-java.git</developerConnection>
<url>https://github.com/influxdata/influxdb-client-java/tree/master</url>
<tag>v6.10.0</tag>
<tag>HEAD</tag>
</scm>

<build>
Expand Down
10 changes: 5 additions & 5 deletions client-legacy/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -162,26 +162,26 @@ The latest version for Maven dependency:
<dependency>
<groupId>com.influxdb</groupId>
<artifactId>influxdb-client-flux</artifactId>
<version>6.10.0</version>
<version>6.10.1</version>
</dependency>
```

Or when using with Gradle:
```groovy
dependencies {
implementation "com.influxdb:influxdb-client-flux:6.10.0"
implementation "com.influxdb:influxdb-client-flux:6.10.1"
}
```

### Snapshot Repository
The snapshots are deployed into [OSS Snapshot repository](https://oss.sonatype.org/content/repositories/snapshots/).
The snapshots are deployed into [OSS Snapshot repository](https://central.sonatype.com/repository/maven-snapshots/).

#### Maven
```xml
<repository>
<id>ossrh</id>
<name>OSS Snapshot repository</name>
<url>https://oss.sonatype.org/content/repositories/snapshots/</url>
<url>https://central.sonatype.com/repository/maven-snapshots/</url>
<releases>
<enabled>false</enabled>
</releases>
Expand All @@ -193,6 +193,6 @@ The snapshots are deployed into [OSS Snapshot repository](https://oss.sonatype.o
#### Gradle
```
repositories {
maven { url "https://oss.sonatype.org/content/repositories/snapshots" }
maven { url "https://central.sonatype.com/repository/maven-snapshots" }
}
```
4 changes: 2 additions & 2 deletions client-legacy/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@
<parent>
<groupId>com.influxdb</groupId>
<artifactId>influxdb-client</artifactId>
<version>6.10.0</version>
<version>6.10.1-SNAPSHOT</version>
</parent>

<artifactId>influxdb-client-flux</artifactId>
Expand Down Expand Up @@ -66,7 +66,7 @@
<connection>scm:git:git@github.com:influxdata/influxdb-client-java.git</connection>
<developerConnection>scm:git:git@github.com:influxdata/influxdb-client-java.git</developerConnection>
<url>https://github.com/influxdata/influxdb-client-java/tree/master</url>
<tag>v6.10.0</tag>
<tag>HEAD</tag>
</scm>

<build>
Expand Down
4 changes: 2 additions & 2 deletions client-osgi/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@
<parent>
<artifactId>influxdb-client</artifactId>
<groupId>com.influxdb</groupId>
<version>6.10.0</version>
<version>6.10.1-SNAPSHOT</version>
</parent>

<artifactId>influxdb-client-osgi</artifactId>
Expand Down Expand Up @@ -65,7 +65,7 @@
<connection>scm:git:git@github.com:influxdata/influxdb-client-java.git</connection>
<developerConnection>scm:git:git@github.com:influxdata/influxdb-client-java.git</developerConnection>
<url>https://github.com/influxdata/influxdb-client-java/tree/master</url>
<tag>v6.10.0</tag>
<tag>HEAD</tag>
</scm>

<build>
Expand Down
10 changes: 5 additions & 5 deletions client-reactive/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -433,26 +433,26 @@ The latest version for Maven dependency:
<dependency>
<groupId>com.influxdb</groupId>
<artifactId>influxdb-client-reactive</artifactId>
<version>6.10.0</version>
<version>6.10.1</version>
</dependency>
```

Or when using with Gradle:
```groovy
dependencies {
implementation "com.influxdb:influxdb-client-reactive:6.10.0"
implementation "com.influxdb:influxdb-client-reactive:6.10.1"
}
```

### Snapshot Repository
The snapshots are deployed into [OSS Snapshot repository](https://oss.sonatype.org/content/repositories/snapshots/).
The snapshots are deployed into [OSS Snapshot repository](https://central.sonatype.com/repository/maven-snapshots/).

#### Maven
```xml
<repository>
<id>ossrh</id>
<name>OSS Snapshot repository</name>
<url>https://oss.sonatype.org/content/repositories/snapshots/</url>
<url>https://central.sonatype.com/repository/maven-snapshots/</url>
<releases>
<enabled>false</enabled>
</releases>
Expand All @@ -464,6 +464,6 @@ The snapshots are deployed into [OSS Snapshot repository](https://oss.sonatype.o
#### Gradle
```
repositories {
maven { url "https://oss.sonatype.org/content/repositories/snapshots" }
maven { url "https://central.sonatype.com/repository/maven-snapshots" }
}
```
4 changes: 2 additions & 2 deletions client-reactive/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
<parent>
<artifactId>influxdb-client</artifactId>
<groupId>com.influxdb</groupId>
<version>6.10.0</version>
<version>6.10.1-SNAPSHOT</version>
</parent>
<modelVersion>4.0.0</modelVersion>

Expand Down Expand Up @@ -68,7 +68,7 @@
<connection>scm:git:git@github.com:influxdata/influxdb-client-java.git</connection>
<developerConnection>scm:git:git@github.com:influxdata/influxdb-client-java.git</developerConnection>
<url>https://github.com/influxdata/influxdb-client-java/tree/master</url>
<tag>v6.10.0</tag>
<tag>HEAD</tag>
</scm>

<build>
Expand Down
14 changes: 7 additions & 7 deletions client-scala/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -257,14 +257,14 @@ The latest version for Maven dependency:
<dependency>
<groupId>com.influxdb</groupId>
<artifactId>influxdb-client-scala_2.12</artifactId>
<version>6.10.0</version>
<version>6.10.1</version>
</dependency>
```

Or when using with Gradle:
```groovy
dependencies {
implementation "com.influxdb:influxdb-client-scala_2.12:6.10.0"
implementation "com.influxdb:influxdb-client-scala_2.12:6.10.1"
}
```

Expand All @@ -275,26 +275,26 @@ The latest version for Maven dependency:
<dependency>
<groupId>com.influxdb</groupId>
<artifactId>influxdb-client-scala_2.13</artifactId>
<version>6.10.0</version>
<version>6.10.1</version>
</dependency>
```

Or when using with Gradle:
```groovy
dependencies {
implementation "com.influxdb:influxdb-client-scala_2.13:6.10.0"
implementation "com.influxdb:influxdb-client-scala_2.13:6.10.1"
}
```

### Snapshot Repository
The snapshots are deployed into [OSS Snapshot repository](https://oss.sonatype.org/content/repositories/snapshots/).
The snapshots are deployed into [OSS Snapshot repository](https://central.sonatype.com/repository/maven-snapshots/).

#### Maven
```xml
<repository>
<id>ossrh</id>
<name>OSS Snapshot repository</name>
<url>https://oss.sonatype.org/content/repositories/snapshots/</url>
<url>https://central.sonatype.com/repository/maven-snapshots/</url>
<releases>
<enabled>false</enabled>
</releases>
Expand All @@ -306,6 +306,6 @@ The snapshots are deployed into [OSS Snapshot repository](https://oss.sonatype.o
#### Gradle
```
repositories {
maven { url "https://oss.sonatype.org/content/repositories/snapshots" }
maven { url "https://central.sonatype.com/repository/maven-snapshots" }
}
```
4 changes: 2 additions & 2 deletions client-scala/cross/2.12/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
<parent>
<artifactId>influxdb-client</artifactId>
<groupId>com.influxdb</groupId>
<version>6.10.0</version>
<version>6.10.1-SNAPSHOT</version>
<relativePath>../../../pom.xml</relativePath>
</parent>
<modelVersion>4.0.0</modelVersion>
Expand Down Expand Up @@ -68,7 +68,7 @@
<connection>scm:git:git@github.com:influxdata/influxdb-client-java.git</connection>
<developerConnection>scm:git:git@github.com:influxdata/influxdb-client-java.git</developerConnection>
<url>https://github.com/influxdata/influxdb-client-java/tree/master</url>
<tag>v6.10.0</tag>
<tag>HEAD</tag>
</scm>

<properties>
Expand Down
4 changes: 2 additions & 2 deletions client-scala/cross/2.13/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@
<parent>
<artifactId>influxdb-client</artifactId>
<groupId>com.influxdb</groupId>
<version>6.10.0</version>
<version>6.10.1-SNAPSHOT</version>
<relativePath>../../../pom.xml</relativePath>
</parent>
<modelVersion>4.0.0</modelVersion>
Expand Down Expand Up @@ -68,7 +68,7 @@
<connection>scm:git:git@github.com:influxdata/influxdb-client-java.git</connection>
<developerConnection>scm:git:git@github.com:influxdata/influxdb-client-java.git</developerConnection>
<url>https://github.com/influxdata/influxdb-client-java/tree/master</url>
<tag>v6.10.0</tag>
<tag>HEAD</tag>
</scm>

<properties>
Expand Down
4 changes: 2 additions & 2 deletions client-test/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@
<parent>
<artifactId>influxdb-client</artifactId>
<groupId>com.influxdb</groupId>
<version>6.10.0</version>
<version>6.10.1-SNAPSHOT</version>
</parent>

<artifactId>influxdb-client-test</artifactId>
Expand Down Expand Up @@ -68,7 +68,7 @@
<connection>scm:git:git@github.com:influxdata/influxdb-client-java.git</connection>
<developerConnection>scm:git:git@github.com:influxdata/influxdb-client-java.git</developerConnection>
<url>https://github.com/influxdata/influxdb-client-java/tree/master</url>
<tag>v6.10.0</tag>
<tag>HEAD</tag>
</scm>

<build>
Expand Down
4 changes: 2 additions & 2 deletions client-utils/pom.xml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,7 @@
<parent>
<artifactId>influxdb-client</artifactId>
<groupId>com.influxdb</groupId>
<version>6.10.0</version>
<version>6.10.1-SNAPSHOT</version>
</parent>

<artifactId>influxdb-client-utils</artifactId>
Expand Down Expand Up @@ -66,7 +66,7 @@
<connection>scm:git:git@github.com:influxdata/influxdb-client-java.git</connection>
<developerConnection>scm:git:git@github.com:influxdata/influxdb-client-java.git</developerConnection>
<url>https://github.com/influxdata/influxdb-client-java/tree/master</url>
<tag>v6.10.0</tag>
<tag>HEAD</tag>
</scm>

<build>
Expand Down
10 changes: 5 additions & 5 deletions client/README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1322,26 +1322,26 @@ The latest version for Maven dependency:
<dependency>
<groupId>com.influxdb</groupId>
<artifactId>influxdb-client-java</artifactId>
<version>6.10.0</version>
<version>6.10.1</version>
</dependency>
```

Or when using with Gradle:
```groovy
dependencies {
implementation "com.influxdb:influxdb-client-java:6.10.0"
implementation "com.influxdb:influxdb-client-java:6.10.1"
}
```

### Snapshot Repository
The snapshots are deployed into [OSS Snapshot repository](https://oss.sonatype.org/content/repositories/snapshots/).
The snapshots are deployed into [OSS Snapshot repository](https://central.sonatype.com/repository/maven-snapshots/).

#### Maven
```xml
<repository>
<id>ossrh</id>
<name>OSS Snapshot repository</name>
<url>https://oss.sonatype.org/content/repositories/snapshots/</url>
<url>https://central.sonatype.com/repository/maven-snapshots/</url>
<releases>
<enabled>false</enabled>
</releases>
Expand All @@ -1353,7 +1353,7 @@ The snapshots are deployed into [OSS Snapshot repository](https://oss.sonatype.o
#### Gradle
```
repositories {
maven { url "https://oss.sonatype.org/content/repositories/snapshots" }
maven { url "https://central.sonatype.com/repository/maven-snapshots" }
}
```

Loading