Boyang,

Thanks for the proposed change, the wiki page lgtm. One minor comment
otherwise I'm +1:

For the new API, we now also have a constructor that accepts both a
clientSupplier and a Time, so we should consider having four overloads in
total:
// New API (using Properties)
public KafkaStreams(final Topology, final Properties props)
public KafkaStreams(final Topology, final Properties props, final Time time)
public KafkaStreams(final Topology, final Properties props, final
KafkaClientSupplier
clientSupplier)
public KafkaStreams(final Topology, final Properties props, final
KafkaClientSupplier
clientSupplier, final Time time)

Guozhang

On Tue, Dec 26, 2017 at 7:26 PM, Satish Duggana <[EMAIL PROTECTED]>
wrote:

--
-- Guozhang