[GitHub] [zookeeper] eolivelli commented on issue #950: ZOOKEEPER-3263. JAVA9/11 Warnings: Illegal reflective access in zookeer's kerberosUtil

2019-05-17 Thread GitBox
eolivelli commented on issue #950: ZOOKEEPER-3263. JAVA9/11 Warnings: Illegal reflective access in zookeer's kerberosUtil URL: https://github.com/apache/zookeeper/pull/950#issuecomment-493597736 @anmolnar the result is the same, still the same warning. I feel we can live with it.

[GitHub] [zookeeper] eolivelli edited a comment on issue #949: ZOOKEEPER-3396: Flaky test in RestoreCommittedLogTest

2019-05-17 Thread GitBox
eolivelli edited a comment on issue #949: ZOOKEEPER-3396: Flaky test in RestoreCommittedLogTest URL: https://github.com/apache/zookeeper/pull/949#issuecomment-493594696 @anmolnar can you please commit this one ? that test is very annoying on CI

[GitHub] [zookeeper] eolivelli commented on issue #949: ZOOKEEPER-3396: Flaky test in RestoreCommittedLogTest

2019-05-17 Thread GitBox
eolivelli commented on issue #949: ZOOKEEPER-3396: Flaky test in RestoreCommittedLogTest URL: https://github.com/apache/zookeeper/pull/949#issuecomment-493594696 @anmolnar can you commit this one ? that test is very annoying on CI

RE: [VOTE] Apache ZooKeeper release 3.5.5 candidate 6

2019-05-17 Thread Gracia, Raul
+1 - Built Zookeeper 3.5.5-rc6 from source correctly. - Exercised ZooKeeper standalone + zkCli with/without SSL configured. - Pravega branch with ZooKeeper 3.5.5 (used by Curator 4.0.1) passed the build tests. - All distributed tests of Pravega in a Kubernetes cluster with ZooKeeper 3.5.5 (used

Re: [VOTE] Separate mailing lists for Jira and Github/Gitbox notifications - Take 2

2019-05-17 Thread Enrico Olivelli
+1 Enrico Il ven 17 mag 2019, 17:11 Norbert Kalmar ha scritto: > +1 (non-binding) > > Whoever will do the change (if voted), needs to make sure stuff like > markmail are updated/checked. > I see Apache projects going for separate mailing lists for automated > notifications (I subscribed to

Re: [VOTE] Apache ZooKeeper release 3.5.5 candidate 6

2019-05-17 Thread Rakesh Radhakrishnan
+1 - built ZooKeeper jar from source - tested using 3 node cluster, ran smoke test scenarios, ran command line client, checked jmx beans and LGTM. - verified sig/xsum, release notes looks fine. Thanks Andor and others who have worked on making this release happen! Rakesh On Fri, May 3, 2019 at

Re: [VOTE] Separate mailing lists for Jira and Github/Gitbox notifications - Take 2

2019-05-17 Thread Norbert Kalmar
+1 (non-binding) Whoever will do the change (if voted), needs to make sure stuff like markmail are updated/checked. I see Apache projects going for separate mailing lists for automated notifications (I subscribed to HBase and Thrift for example, they have this separation). My +1 is to separate

Re: Github & Jira notifications

2019-05-17 Thread Flavio Junqueira
To add to this conversation, we have 5 mailing lists as of today: dev@z.a.o commits@z.a.o private@z.a.o security@z.a.o user@z.a.o The commits@ one was historically supposed to be for notifications. -Flavio > On 17 May 2019, at 16:50, Lars Francke wrote: > > Alright, I will do so, thank you

[VOTE] Separate mailing lists for Jira and Github/Gitbox notifications - Take 2

2019-05-17 Thread Lars Francke
Hi ZooKeeper devs, I'd like to call a vote on whether we want to have separate mailing lists for dev chatter (mostly humans) and automated notifications Andor summarized this in another thread: - Jenkins build notifications: failures / successes, - Jira notifications: Created, Updated, Status

Re: [VOTE] Separate mailing lists for Jira and Github/Gitbox notifications - Take 2

2019-05-17 Thread Lars Francke
And here is my +1 (non-binding) On Fri, May 17, 2019 at 4:55 PM Lars Francke wrote: > Hi ZooKeeper devs, > > I'd like to call a vote on whether we want to have separate mailing lists > for dev chatter (mostly humans) and automated notifications > > Andor summarized this in another thread: > -

Re: Github & Jira notifications

2019-05-17 Thread Lars Francke
Alright, I will do so, thank you On Fri, May 17, 2019 at 11:46 AM Norbert Kalmar wrote: > I think we should start a vote. > > Lars, feel free to start it, as it is your initiative, you can place your > argument in the initial email. > Thanks for bringing this up! > > Regards, > Norbert > > > On

[GitHub] [zookeeper] anmolnar commented on issue #950: ZOOKEEPER-3263. JAVA9/11 Warnings: Illegal reflective access in zookeer's kerberosUtil

2019-05-17 Thread GitBox
anmolnar commented on issue #950: ZOOKEEPER-3263. JAVA9/11 Warnings: Illegal reflective access in zookeer's kerberosUtil URL: https://github.com/apache/zookeeper/pull/950#issuecomment-493474795 retest this please This is an

[GitHub] [zookeeper] anmolnar commented on issue #950: ZOOKEEPER-3263. JAVA9/11 Warnings: Illegal reflective access in zookeer's kerberosUtil

2019-05-17 Thread GitBox
anmolnar commented on issue #950: ZOOKEEPER-3263. JAVA9/11 Warnings: Illegal reflective access in zookeer's kerberosUtil URL: https://github.com/apache/zookeeper/pull/950#issuecomment-493459800 Updated. Please check. This is

[GitHub] [zookeeper] eolivelli commented on issue #950: ZOOKEEPER-3263. JAVA9/11 Warnings: Illegal reflective access in zookeer's kerberosUtil

2019-05-17 Thread GitBox
eolivelli commented on issue #950: ZOOKEEPER-3263. JAVA9/11 Warnings: Illegal reflective access in zookeer's kerberosUtil URL: https://github.com/apache/zookeeper/pull/950#issuecomment-493453739 @anmolnar maybe it helps, I don't know. Please try

[GitHub] [zookeeper] anmolnar commented on issue #950: ZOOKEEPER-3263. JAVA9/11 Warnings: Illegal reflective access in zookeer's kerberosUtil

2019-05-17 Thread GitBox
anmolnar commented on issue #950: ZOOKEEPER-3263. JAVA9/11 Warnings: Illegal reflective access in zookeer's kerberosUtil URL: https://github.com/apache/zookeeper/pull/950#issuecomment-493453441 @eolivelli shall we upgrade BouncyCastle?

[GitHub] [zookeeper] eolivelli commented on issue #950: ZOOKEEPER-3263. JAVA9/11 Warnings: Illegal reflective access in zookeer's kerberosUtil

2019-05-17 Thread GitBox
eolivelli commented on issue #950: ZOOKEEPER-3263. JAVA9/11 Warnings: Illegal reflective access in zookeer's kerberosUtil URL: https://github.com/apache/zookeeper/pull/950#issuecomment-493429920 @anmolnar the issue you are trying to fix is fixed with this patch. I am seeing errors

[GitHub] [zookeeper] eolivelli commented on a change in pull request #923: ZOOKEEPER-1426: add version command to the zookeeper server

2019-05-17 Thread GitBox
eolivelli commented on a change in pull request #923: ZOOKEEPER-1426: add version command to the zookeeper server URL: https://github.com/apache/zookeeper/pull/923#discussion_r285089642 ## File path: zookeeper-server/src/test/resources/test-scripts.sh ## @@ -217,6 +217,9

Re: Github & Jira notifications

2019-05-17 Thread Norbert Kalmar
I think we should start a vote. Lars, feel free to start it, as it is your initiative, you can place your argument in the initial email. Thanks for bringing this up! Regards, Norbert On Fri, May 17, 2019 at 11:38 AM Lars Francke wrote: > I don't really have anything else to add to this

Re: Github & Jira notifications

2019-05-17 Thread Lars Francke
I don't really have anything else to add to this conversation. I agree with Norbert & Andor. Separating the lists is (slowly) becoming the standard at the ASF in the projects I participate in and it makes it easier for newcomers. Do we want to put this to a vote or abandon this? On Mon, May 13,