Hey Matt and Guozhang,
I have already updated the pull request: https://github.com/apache/kafka/pull/4354

and the KIP: https://cwiki.apache.org/confluence/display/KAFKA/KIP-245%3A+Use+Properties+instead+of+StreamsConfig+in+KafkaStreams+constructor
to reflect the change proposed by Guozhang(adding a 4th constructor)

Let me know your thoughts!
Best,

Boyang
________________________________
From: Boyang Chen <[EMAIL PROTECTED]>
Sent: Saturday, January 13, 2018 9:37 AM
To: Matthias J. Sax
Subject: Re: Vote for KIP-245: Use Properties instead of StreamsConfig in KafkaStreams constructor
Sounds good, will do. However I don't receive the +1 emails, interesting...
________________________________
From: Matthias J. Sax <[EMAIL PROTECTED]>
Sent: Saturday, January 13, 2018 9:32 AM
To: Boyang Chen
Subject: Re: Vote for KIP-245: Use Properties instead of StreamsConfig in KafkaStreams constructor

Guozhang left a comment about having a 4th overload. I agree that we
should add this 4th overload.

Please update the KIP accordingly and follow up on the mailing list
thread. Than we can vote it through.

Thx.

-Matthias

On 1/12/18 4:48 PM, Boyang Chen wrote:
issues.apache.org<https://issues.apache.org/>
issues.apache.org
Apache currently hosts two different issue tracking systems, Bugzilla and Jira. To find out how to report an issue for a particular project, please visit the project ...

[KAFKA-6386] Deprecate KafkaStreams constructor taking ...<https://issues.apache.org/jira/browse/KAFKA-6386>
issues.apache.org
Currently, KafkaStreams constructor has overloads that take either Properties or StreamsConfig a parameters. Because StreamsConfig is immutable and is created from a ...

[KAFKA-6386] Deprecate KafkaStreams constructor taking ...<https://issues.apache.org/jira/browse/KAFKA-6386>
issues.apache.org
Currently, KafkaStreams constructor has overloads that take either Properties or StreamsConfig a parameters. Because StreamsConfig is immutable and is created from a ...

[https://avatars3.githubusercontent.com/u/5845561?s=400&v=4]<https://github.com/apache/kafka/pull/4354>

KAFKA-6386:use Properties instead of StreamsConfig in KafkaStreams constructor by abbccdda · Pull Request #4354 · apache/kafka<https://github.com/apache/kafka/pull/4354>
github.com
This pull request targets https://issues.apache.org/jira/browse/KAFKA-6386 The minor fix to deprecate usage of StreamsConfig in favor of java.util.Properties. I created separate public constructors...

[https://avatars3.githubusercontent.com/u/5845561?s=400&v=4]<https://github.com/apache/kafka/pull/4354>

KAFKA-6386:use Properties instead of StreamsConfig in KafkaStreams constructor by abbccdda · Pull Request #4354 · apache/kafka<https://github.com/apache/kafka/pull/4354>
github.com
This pull request targets https://issues.apache.org/jira/browse/KAFKA-6386 The minor fix to deprecate usage of StreamsConfig in favor of java.util.Properties. I created separate public constructors...

[https://avatars3.githubusercontent.com/u/5845561?s=400&v=4]<https://github.com/apache/kafka/pull/4354>

KAFKA-6386:use Properties i