Skip to content

Conversation

@merlimat
Copy link
Contributor

Motivation

Upgrade to newest version of RocksDb which has support for Mac with M1 architecture.

@merlimat merlimat added release/note-required doc-not-needed Your PR changes do not impact docs labels Mar 26, 2022
@merlimat merlimat added this to the 2.11.0 milestone Mar 26, 2022
@merlimat merlimat self-assigned this Mar 26, 2022
Copy link
Contributor

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

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

Lgtm

@merlimat merlimat merged commit 12faf2d into apache:master Mar 28, 2022
@merlimat merlimat deleted the upgrade-rocksdb branch March 28, 2022 15:54
lhotari added a commit to lhotari/pulsar that referenced this pull request Mar 31, 2022
lhotari added a commit that referenced this pull request Mar 31, 2022
Fixes #14961

This reverts commit 12faf2d.

Code compiled with RocksDB <6.17.3 isn't compatible with a newer RocksDB version at runtime.
Nicklee007 pushed a commit to Nicklee007/pulsar that referenced this pull request Apr 20, 2022
Nicklee007 pushed a commit to Nicklee007/pulsar that referenced this pull request Apr 20, 2022
Fixes apache#14961

This reverts commit 12faf2d.

Code compiled with RocksDB <6.17.3 isn't compatible with a newer RocksDB version at runtime.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants