kafka-jira mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (KAFKA-5572) ConfigDef.Type.List should support escaping comma character
Date Sat, 08 Jul 2017 16:23:00 GMT

    [ 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:


    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:


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 <jcustenborder@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

View raw message