Re: [VOTE] Pulsar Release 2.1.1-incubating Candidate 1

2018-09-06 Thread Jerry Peng
Sorry there was a typo in my previous email: "v2.2.1-incubating-candidate-1" => "v2.1.1-incubating-candidate-1" On Thu, Sep 6, 2018 at 5:31 PM Jerry Peng wrote: > This is the first release candidate for Apache Pulsar, version > 2.1.1-incubating. > > It fixes the following issues: >

Re: [DRAFT] Podling report

2018-09-06 Thread Jerry Peng
+1 Very nicely done! Best, Jerry On Thu, Sep 6, 2018 at 8:20 AM Rajan Dhabalia wrote: > +1. > > Thanks, > Rajan > > Sent from my iPhone > > > On Sep 5, 2018, at 5:36 PM, Matteo Merli wrote: > > > > Sorry for sending out at very last minute (deadline is today). Please > take > > a look. > >

[VOTE] Pulsar Release 2.1.1-incubating Candidate 1

2018-09-06 Thread Jerry Peng
This is the first release candidate for Apache Pulsar, version 2.1.1-incubating. It fixes the following issues: https://github.com/apache/incubator-pulsar/milestone/17?closed=1 Mostly bug fixes for 2.1.0 *** Please download, test and vote on this release. This vote will stay open for at least

Re: [DRAFT] Podling report

2018-09-06 Thread Rajan Dhabalia
+1. Thanks, Rajan Sent from my iPhone > On Sep 5, 2018, at 5:36 PM, Matteo Merli wrote: > > Sorry for sending out at very last minute (deadline is today). Please take > a look. > > I have added a paragraph regarding the branding discussion that happen in > early June (as discussed with

Re: [DRAFT] Podling report

2018-09-06 Thread David Kjerrumgaard
+1 looks good On Wed, Sep 5, 2018 at 5:36 PM, Matteo Merli wrote: > Sorry for sending out at very last minute (deadline is today). Please take > a look. > > I have added a paragraph regarding the branding discussion that happen in > early June (as discussed with Taylor then) since the previous

Re: PIP 20: Mechanism to revoke TLS authentication

2018-09-06 Thread Ivan Kelly
That said, it is stored in Json so it wouldn't be much work to sort one of the values as a list of strings. -Ivan On Thu, Sep 6, 2018 at 10:55 AM, Ivan Kelly wrote: > PIP updated. The one thing that isn't 100% clean with the dynamic > configuration keys is that they only support string as

Re: PIP 20: Mechanism to revoke TLS authentication

2018-09-06 Thread Ivan Kelly
PIP updated. The one thing that isn't 100% clean with the dynamic configuration keys is that they only support string as value, so the list of revoked keys needs some sort of serialization. -Ivan On Thu, Sep 6, 2018 at 10:41 AM, Ivan Kelly wrote: > On Tue, Aug 21, 2018 at 3:57 AM, Rajan

Re: PIP 20: Mechanism to revoke TLS authentication

2018-09-06 Thread Ivan Kelly
On Tue, Aug 21, 2018 at 3:57 AM, Rajan Dhabalia wrote: >> I will consider both this and system topics and update the PIP. > > I think we should avoid more complexity by introducing system topic to > store "subject key" and then process it. Instead we can store it to > global/config zk as it

Re: [DRAFT] Podling report

2018-09-06 Thread Ivan Kelly
lgtm +! On Thu, Sep 6, 2018 at 9:52 AM, Sijie Guo wrote: > +1 looks good > > - Sijie > > On Wed, Sep 5, 2018 at 5:36 PM Matteo Merli wrote: > >> Sorry for sending out at very last minute (deadline is today). Please take >> a look. >> >> I have added a paragraph regarding the branding discussion

Re: [DRAFT] Podling report

2018-09-06 Thread Ivan Kelly
that should be +1 On Thu, Sep 6, 2018 at 10:38 AM, Ivan Kelly wrote: > lgtm +! > > On Thu, Sep 6, 2018 at 9:52 AM, Sijie Guo wrote: >> +1 looks good >> >> - Sijie >> >> On Wed, Sep 5, 2018 at 5:36 PM Matteo Merli wrote: >> >>> Sorry for sending out at very last minute (deadline is today).

Re: [DRAFT] Podling report

2018-09-06 Thread Sijie Guo
+1 looks good - Sijie On Wed, Sep 5, 2018 at 5:36 PM Matteo Merli wrote: > Sorry for sending out at very last minute (deadline is today). Please take > a look. > > I have added a paragraph regarding the branding discussion that happen in > early June (as discussed with Taylor then) since the