Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

KAFKA-15044: Upgrade to snappy v1.1.10.0 #13786

Merged
merged 1 commit into from May 31, 2023
Merged

Conversation

splett2
Copy link
Contributor

@splett2 splett2 commented May 31, 2023

Snappy v1.1.9.1 has some issues around arm compatibility and glibc library versions shipped by default with certain OS distributions. Snappy v1.1.10.0 uses a glibc LTS version, see: xerial/snappy-java#417 xerial/snappy-java#440

This needs to be cherry-picked to the 3.5 branch as well.

Committer Checklist (excluded from commit message)

  • Verify design and implementation
  • Verify test coverage and CI build status
  • Verify documentation (including upgrade notes)

Copy link
Contributor

@jolshan jolshan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for fixing this.

@jolshan
Copy link
Contributor

jolshan commented May 31, 2023

Test failures are unrelated. Will merge and pick to 3.5.

@jolshan jolshan merged commit f9b2b3a into apache:trunk May 31, 2023
1 check failed
jolshan pushed a commit that referenced this pull request May 31, 2023
Snappy v1.1.9.1 has some issues around arm compatibility and glibc library versions shipped by default with certain OS distributions. Snappy v1.1.10.0 uses a glibc LTS version, see: xerial/snappy-java#417 xerial/snappy-java#440

Reviewers: Justine Olshan <jolshan@confluent.io>
@divijvaidya
Copy link
Contributor

divijvaidya commented Jun 1, 2023

As a thought exercise, could we have detected this ARM specific bug if we were running our tests on ARM machines? What requirement do we have to enable testing on ARM again?

@jolshan @splett2 @mimaison thoughts?

@mimaison
Copy link
Member

mimaison commented Jun 1, 2023

We used to run our tests on ARM machines too but we disabled it a few months ago because they were stuck. I believe the owner of the ARM machines fixed them but we never re-enabled it. I looked at properly supporting non x86_64 platforms (KIP-826) but did not have time to follow up.

emissionnebula added a commit to confluentinc/kafka that referenced this pull request Jun 16, 2023
…tream-3.5

* commit 'c2f6f29ca6e1306ac77ec726bac4cd09bd1aa80b': (76 commits)
  KAFKA-15019: Improve handling of broker heartbeat timeouts (apache#13759)
  KAFKA-15003: Fix ZK sync logic for partition assignments (apache#13735)
  MINOR: Add 3.5 upgrade steps for ZK and KRaft (apache#13792)
  KAFKA-15010 ZK migration failover support (apache#13758)
  KAFKA-15017 Fix snapshot load in dual write mode for ClientQuotas and SCRAM  (apache#13757)
  MINOR: Update LICENSE-binary following snappy upgrade (apache#13791)
  Upgrade to snappy v1.1.10.0 (apache#13786)
  KAFKA-15004: Fix configuration dual-write during migration (apache#13767)
  KAFKA-8713: JsonConverter replace.null.with.default should prevent emitting default for Struct fields (apache#13781)
  KAFKA-14996: Handle overly large user operations on the kcontroller (apache#13742)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants