Hi zookeepers,

Recently I've sent a pr[1] on enable checkstyle configuration on
zookeeper-promethus-metrics, Enrico(committer) and Dinesh(contributor)
kindly reviewed it but we still need another look from committer as
process required. Someone of you has spare time to have a look?

Besides, as ZOOKEEPER-3431 described, we're going to enable this
configuration on zookeeper-server as following. This is really a major
package which contains a lot of codes involved by multi prs. I'd like to
hear from the community about how to proceed.

(From where I stand, enable on the whole package at once
could solve the problem at once, but it would conflict quite a lot
ongoing prs. On the other hand, sending the pr without timely reviews
also causes a burden to rebase it on the master and solve conflict
again and again(since it likely conflicts with a merged pr))

Best,
tison.

[1] https://github.com/apache/zookeeper/pull/1028

Reply via email to