[jira] [Commented] (KAFKA-3373) Add `log` prefix to KIP-31/32 configs
[ https://issues.apache.org/jira/browse/KAFKA-3373?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15200370#comment-15200370 ] ASF GitHub Bot commented on KAFKA-3373: --- Github user asfgit closed the pull request at: https://github.com/apache/kafka/pull/1072 > Add `log` prefix to KIP-31/32 configs > - > > Key: KAFKA-3373 > URL: https://issues.apache.org/jira/browse/KAFKA-3373 > Project: Kafka > Issue Type: Bug >Reporter: Ismael Juma >Assignee: Jiangjie Qin >Priority: Blocker > Fix For: 0.10.0.0 > > > [~jjkoshy] suggested that we should prefix the configs introduced as part of > KIP-31/32 to include a `log` prefix: > message.format.version > message.timestamp.type > message.timestamp.difference.max.ms > If we do it, we must update the KIP. > Marking it as blocker because we should decide either way before 0.10.0.0. > Discussion here: > https://github.com/apache/kafka/pull/907#issuecomment-193950768 -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (KAFKA-3373) Add `log` prefix to KIP-31/32 configs
[ https://issues.apache.org/jira/browse/KAFKA-3373?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15195191#comment-15195191 ] ASF GitHub Bot commented on KAFKA-3373: --- GitHub user ijuma opened a pull request: https://github.com/apache/kafka/pull/1072 MINOR: KAFKA-3373 follow-up, a few val renames remaining You can merge this pull request into a Git repository by running: $ git pull https://github.com/ijuma/kafka kafka-3373-follow-up Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/1072.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 #1072 commit 13b9fbc6a60a3ace789794072b3ee7ed5c72e713 Author: Ismael Juma Date: 2016-03-15T12:19:12Z MINOR: KAFKA-3373 follow-up, a few val renames remaining > Add `log` prefix to KIP-31/32 configs > - > > Key: KAFKA-3373 > URL: https://issues.apache.org/jira/browse/KAFKA-3373 > Project: Kafka > Issue Type: Bug >Reporter: Ismael Juma >Assignee: Jiangjie Qin >Priority: Blocker > Fix For: 0.10.0.0 > > > [~jjkoshy] suggested that we should prefix the configs introduced as part of > KIP-31/32 to include a `log` prefix: > message.format.version > message.timestamp.type > message.timestamp.difference.max.ms > If we do it, we must update the KIP. > Marking it as blocker because we should decide either way before 0.10.0.0. > Discussion here: > https://github.com/apache/kafka/pull/907#issuecomment-193950768 -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (KAFKA-3373) Add `log` prefix to KIP-31/32 configs
[ https://issues.apache.org/jira/browse/KAFKA-3373?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15194577#comment-15194577 ] ASF GitHub Bot commented on KAFKA-3373: --- Github user asfgit closed the pull request at: https://github.com/apache/kafka/pull/1049 > Add `log` prefix to KIP-31/32 configs > - > > Key: KAFKA-3373 > URL: https://issues.apache.org/jira/browse/KAFKA-3373 > Project: Kafka > Issue Type: Bug >Reporter: Ismael Juma >Assignee: Jiangjie Qin >Priority: Blocker > Fix For: 0.10.0.0 > > > [~jjkoshy] suggested that we should prefix the configs introduced as part of > KIP-31/32 to include a `log` prefix: > message.format.version > message.timestamp.type > message.timestamp.difference.max.ms > If we do it, we must update the KIP. > Marking it as blocker because we should decide either way before 0.10.0.0. > Discussion here: > https://github.com/apache/kafka/pull/907#issuecomment-193950768 -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (KAFKA-3373) Add `log` prefix to KIP-31/32 configs
[ https://issues.apache.org/jira/browse/KAFKA-3373?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15190031#comment-15190031 ] ASF GitHub Bot commented on KAFKA-3373: --- GitHub user becketqin opened a pull request: https://github.com/apache/kafka/pull/1049 KAFKA-3373 add 'log' prefix to configurations in KIP-31/32 You can merge this pull request into a Git repository by running: $ git pull https://github.com/becketqin/kafka KAFKA-3373 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/kafka/pull/1049.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 #1049 commit ff4a509902a888563a1dc39fd129eba7f0c53bb9 Author: Jiangjie Qin Date: 2016-03-10T21:57:46Z KAFKA-3373 add 'log' prefix to configurations in KIP-31/32 > Add `log` prefix to KIP-31/32 configs > - > > Key: KAFKA-3373 > URL: https://issues.apache.org/jira/browse/KAFKA-3373 > Project: Kafka > Issue Type: Bug >Reporter: Ismael Juma >Assignee: Jiangjie Qin >Priority: Blocker > Fix For: 0.10.0.0 > > > [~jjkoshy] suggested that we should prefix the configs introduced as part of > KIP-31/32 to include a `log` prefix: > message.format.version > message.timestamp.type > message.timestamp.difference.max.ms > If we do it, we must update the KIP. > Marking it as blocker because we should decide either way before 0.10.0.0. > Discussion here: > https://github.com/apache/kafka/pull/907#issuecomment-193950768 -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (KAFKA-3373) Add `log` prefix to KIP-31/32 configs
[ https://issues.apache.org/jira/browse/KAFKA-3373?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15188982#comment-15188982 ] Ismael Juma commented on KAFKA-3373: cc [~becket_qin] [~junrao] > Add `log` prefix to KIP-31/32 configs > - > > Key: KAFKA-3373 > URL: https://issues.apache.org/jira/browse/KAFKA-3373 > Project: Kafka > Issue Type: Bug >Reporter: Ismael Juma > Fix For: 0.10.0.0 > > > [~jjkoshy] suggested that we should prefix the configs introduced as part of > KIP-31/32 to include a `log` prefix: > message.format.version > message.timestamp.type > message.timestamp.difference.max.ms > If we do it, we must update the KIP. > Marking it as blocker because we should decide either way before 0.10.0.0. > Discussion here: > https://github.com/apache/kafka/pull/907#issuecomment-193950768 -- This message was sent by Atlassian JIRA (v6.3.4#6332)