Hey <@UQ9S6U14Z> we recently found a bug in this P...
# pinot-dev
j
Hey @User we recently found a bug in this PR (https://github.com/apache/incubator-pinot/pull/6418), and I opened a hotfix for that (https://github.com/apache/incubator-pinot/pull/6540). Could you review it?
a
Hey Jack, thanks for the PR. Quick question - did you check with the most recent master? we just merged a related PR 2 days ago: https://github.com/apache/incubator-pinot/pull/6515
j
Yes, I noticed that. But this time it’s on pinot-server. The PR you pointed out is for pinot-controller and pinot-broker.
a
I could have done a better job with the PR description. we actually fixed the default behavior for server admin port too (see bottom change to ListenerConfigUtil.java)
j
Cool, I’ve verified that the latest fix works. Thanks for pointing that out! I can close the current PR now.
👌 1