[SPARK-42486][BUILD] Upgrade ZooKeeper to 3.6.4

[bjornjorgensen]

What changes were proposed in this pull request?

Upgrade ZooKeeper from 3.6.3 to 3.6.4

Release notes

Why are the changes needed?

ZooKeeper 3.6 is EoL since 30th December, 2022

Does this PR introduce any user-facing change?

No.

How was this patch tested?

Pass GA

[bjornjorgensen]

@yaooqinn you did the last upgrade on this one in #38733 will you have a look at this one?

[dongjoon-hyun]

According to the PR description, does this PR aims to upgrade from one EoL version (3.6.3) to another EoL version (3.6.4), @bjornjorgensen ?

[ZooKeeper 3.6 is EoL since 30th December, 2022(https://zookeeper.apache.org/releases.html)

[bjornjorgensen]

@dongjoon-hyun yes, that's right.
I think we can have this small update now to master and branch 3.4. When 3.4 is released, we can try to update to a newer version.

[dongjoon-hyun]

Got it. Thank you for the clarification. Sounds safe.

[dongjoon-hyun]

Oh, I missed that you wrote branch-3.4 explicitly. Are you sure, @bjornjorgensen ?

I think we can have this small update now to master and branch 3.4. When 3.4 is released, we can try to update to a newer version.

If you want to claim this PR as a blocker, you need to do that properly. In general, you had better do two things at least.

• Ping, @xinrong-meng , in order to inform her the existence of the blocker.
• Update SPARK-42486 properly. Currently, the JIRA Affected Version is quite opposite from your comment (or intention) here.

[bjornjorgensen]

I did upgrade the JIRA ticket now to include 3.4.0.
This is not a blocker.
I think this is more a nice too have upgrade for 3.4.0.

[dongjoon-hyun]

I'm not sure new 3.6.4 Zookeepr is urgent or safe in branch-3.4. According to the Maven Central, 3.6.4 is the least adopted (unverified) release among the recent versions. Do we have a test case to support that we need this?

[HyukjinKwon]

Let's probably don't add it to branch-3.4 ...

[xinrong-meng]

Since branch-3.4 CI is under fix so I may have to re-create the tag later. Please let me know if we shall wait for this PR or not.

[dongjoon-hyun]

IMO, we don't need to wait for this PR, @xinrong-meng .

[xinrong-meng]

Thanks @dongjoon-hyun!

[bjornjorgensen]

Ok, thank you, @dongjoon-hyun

[github-actions]

We're closing this PR because it hasn't been updated in a while. This isn't a judgement on the merit of the PR in any way. It's just a way of keeping the PR queue manageable.
If you'd like to revive this PR, please reopen it and ask a committer to remove the Stale tag!