search

sofastack / sofa-jraft

A production-grade java implementation of RAFT consensus algorithm.
https://www.sofastack.tech/projects/sofa-jraft/
Apache License 2.0
3.53k stars 1.12k forks source link

chore: remove deprecated method to adapt higher rocksdb version #1058

Closed leizhiyuan closed 6 months ago

leizhiyuan commented 6 months ago

Motivation:

Explain the context, and why you're making that change. To make others understand what is the problem you're trying to solve.

Modification:

after upgrade the new version， I removed all deprecated method

Result:

Fixes #1057

If there is no issue then describe the changes introduced by this PR.

leizhiyuan commented 6 months ago

after test case succeeded, I will restore the default rocksdb config

RongtongJin commented 6 months ago

Looking forward to merging soon, RocketMQ is introducing jraft, but jraft has employed deprecated methods from an older version of rocksdb, resulting in unexpected startup failures.

leizhiyuan commented 6 months ago

cc @fengjiachun

RongtongJin commented 6 months ago

Hi @fengjiachun , I'm curious to know whether there's an upcoming release planned in the near future. We are looking forward that JRaft can be applied to RocketMQ ASAP :-) https://github.com/apache/rocketmq/pull/7301

fengjiachun commented 6 months ago

I plan to release this month(maybe as soon as next week). I will first do a Jepsen validation, and once it has been verified, I will proceed to the release. The detailed process will be continuously updated in this issue.

fengjiachun commented 5 months ago

@RongtongJin @leizhiyuan https://github.com/sofastack/sofa-jraft/issues/1064

leizhiyuan commented 5 months ago

thanks，we will test soon