-1(binding) because https://github.com/apache/druid/pull/17697 prevents us from upgrading our old clusters to this release.
Our clusters are running in K8S, above PR skips the setting of druid.host properties, however, this property is defined as localhost by default in the common.properties, so after upgrading, all druid service names registered in the zookeeper are localhost, which can't be accessed by other services. Possible solutions might be: 1. if the druid_host environment is not provided, use the hostname command to set the druid.host property if it's under the K8S environment to override the default one in the property file 2. or remove the druid.host default property from the file and always set it via starting scripts BTW, I think we should add this change(use the hostname under the K8S environment) to the release note too. On Thu, Apr 17, 2025 at 8:25 AM Clint Wylie <cwy...@apache.org> wrote: > +1 (binding) > > src package: > * verified checksum and signature > * checked for LICENSE and NOTICE > * rat check successful > * built binary distribution, ran MSQ ingestion, kafka ingestion, > compaction, then some queries > > binary package: > * verified checksum and signature > * checked for LICENSE and NOTICE > * ran json batch ingestion and some queries > > docker: > * verified checksum > * started cluster with docker-compose, ran native json ingestion, some > queries > > I did see an out of memory failure on coordinator-overlord when using > the old 'micro quickstart' configuration when I was testing kafka > ingestion... I would guess it is related to the sampler? However, I > did not look much further into it since I think the recommended way to > run tutorials these days is with the auto sizing script, so not sure > if it is a big deal or not. Did not see the problem when using that > script instead. > > On Wed, Apr 16, 2025 at 12:39 AM Zoltan Haindrich <k...@rxd.hu> wrote: > > > > Hi all, > > > > I have created a build for Apache Druid 33.0.0, release candidate 2. > (RC1 was botched). > > > > Thanks for everyone who has helped contribute to the release! You can > read > > the proposed release notes here: > > https://github.com/apache/druid/pull/17872 > > > > The release candidate has been tagged in GitHub as druid-33.0.0-rc2 > (63477ea1b7866dc2f61c9d1d893dea150735cbff), > > available here: https://github.com/apache/druid/tree/druid-33.0.0-rc2 > > > > The artifacts to be voted on are located here: > > https://dist.apache.org/repos/dist/dev/druid/33.0.0-rc2/ > > > > A staged Maven repository is available for review at: > > https://repository.apache.org/content/repositories/orgapachedruid-1086/ > > > > Staged druid.apache.org website documentation is available here: > > https://druid.staged.apache.org/docs/33.0.0/ > > > > A Docker image containing the binary of the release candidate can be > > retrieved via: > > docker pull apache/druid:33.0.0-rc2 > > > > artifact checksums: > > 33.0.0-rc2/apache-druid-33.0.0-bin.tar.gz > > > cd6adc2840c87521312534cd6a786b0d92559a652f275a6d773e717cef1c5bd0f6aacac67626e2a26fdad96f2353c5314e53b8a2533b37cbdc953d96025a1e04 > > > > apache-druid-33.0.0-src.tar.gz > > > d4f78b627e0c46db0d328ce53460ae3f5639831f20efc50ed6c25c5999aa310dd089d4d7813c0994f7c6714a2df24594ab7b14ab087e330bb2fbc5992e6e7293 > > > > docker: > sha256:528a78a68b1dde47a5984e60975203d8cdf0bd4f477f8b97b84a8b2b44e3c683 > > > > > > Release artifacts are signed with the following key: > > https://people.apache.org/keys/committer/kgyrtkirk.asc > > > > This key and the key of other committers can also be found in the > project's > > KEYS file here: > > https://dist.apache.org/repos/dist/release/druid/KEYS > > > > (If you are a committer, please feel free to add your own key to that > file > > by following the instructions in the file's header.) > > > > Verify checksums: > > diff <(shasum -a512 apache-druid-33.0.0-src.tar.gz | cut -d ' ' -f1) > <(cat apache-druid-33.0.0-src.tar.gz.sha512 ; echo) > > diff <(shasum -a512 apache-druid-33.0.0-bin.tar.gz | cut -d ' ' -f1) > <(cat apache-druid-33.0.0-bin.tar.gz.sha512 ; echo) > > > > Verify signatures: > > gpg --verify apache-druid-33.0.0-src.tar.gz.asc > apache-druid-33.0.0-src.tar.gz > > gpg --verify apache-druid-33.0.0-bin.tar.gz.asc > apache-druid-33.0.0-bin.tar.gz > > > > Please review the proposed artifacts and vote. Note that Apache has > > specific requirements that must be met before +1 binding votes can be > cast > > by PMC members. Please refer to the policy at > > http://www.apache.org/legal/release-policy.html#policy for more details. > > > > As part of the validation process, the release artifacts can be generated > > from source by running: > > mvn clean install -Papache-release,dist -Dgpg.skip > > > > The RAT license check can be run from source by: > > mvn apache-rat:check -Prat > > > > This vote will be open for at least 72 hours. The vote will pass if a > > majority of at least three +1 PMC votes are cast. > > > > [ ] +1 Release this package as Apache Druid 0.17.0 > > [ ] 0 I don't feel strongly about it, but I'm okay with the release > > [ ] -1 Do not release this package because... > > > > cheers, > > Zoltan > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscr...@druid.apache.org > For additional commands, e-mail: dev-h...@druid.apache.org > >
