You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@kafka.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/07/08 16:23:00 UTC
[jira] [Commented] (KAFKA-5572) ConfigDef.Type.List should support
escaping comma character
[ https://issues.apache.org/jira/browse/KAFKA-5572?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16079212#comment-16079212 ]
ASF GitHub Bot commented on KAFKA-5572:
---------------------------------------
GitHub user jcustenborder opened a pull request:
https://github.com/apache/kafka/pull/3508
KAFKA-5572 - ConfigDef should be able to escape comma(s)
Allow escaping ',' character with '\,' as an escape sequence.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jcustenborder/kafka KAFKA-5572
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/kafka/pull/3508.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #3508
----
commit f22af833e6e9ce7fb6437994e29ae3dc8bfd6611
Author: Jeremy Custenborder <jc...@gmail.com>
Date: 2017-07-08T16:19:00Z
KAFKA-5572 - Allow escaping ',' character with '\,' as an escape sequence.
----
> ConfigDef.Type.List should support escaping comma character
> -----------------------------------------------------------
>
> Key: KAFKA-5572
> URL: https://issues.apache.org/jira/browse/KAFKA-5572
> Project: Kafka
> Issue Type: Improvement
> Reporter: Jeremy Custenborder
> Assignee: Jeremy Custenborder
> Priority: Minor
>
> You should be able to include a comma in a list. Currently the split regex is only looks for comma. This should be escapable with as something like \,.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)