Skip to content

参考BDBLogStorage 基于ChronicleMap、MapDB、H2MVStore、LevelDB-Java实现的日志存储 #1216

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.

Sign up for GitHub

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

Open
KnightBlood wants to merge 13 commits into
base: master
Choose a base branch
from
Open

参考BDBLogStorage 基于ChronicleMap、MapDB、H2MVStore、LevelDB-Java实现的日志存储 #1216

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
77331c9
增加LevelDB、H2 MVStore、MapDB、Chronicle Map的实现以及测试类
Aug 27, 2025
3c12cb7
修改一下Author，抄代码的时候忘改了
Aug 27, 2025
92dc366
修改代码能够通过单元测试
Aug 28, 2025
ca6f290
Update build.yml
KnightBlood Aug 28, 2025
ea79171
修改ChronicleMapLogStorage实现能够通过单元测试
Aug 28, 2025
ba14494
Merge remote-tracking branch 'origin/master'
Aug 28, 2025
13f5300
Update build.yml
KnightBlood Aug 28, 2025
7e3af16
Update build.yml
KnightBlood Aug 28, 2025
b142feb
Update build.yml
KnightBlood Aug 28, 2025
70fa0d7
Create build-windows.yml
KnightBlood Aug 28, 2025
6365b3f
Create build-macos.yml
KnightBlood Aug 28, 2025
471bbe0
修改 build*.yml 目前ChronicleMapLogStorage只能在JDK8上通过测试
Aug 28, 2025
d3a33ec
update actions/checkout v2 to v5
Aug 29, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
165 changes: 165 additions & 0 deletions .github/workflows/build-macos.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,165 @@
name: build-macos

on:
push:
branches: [ master ]
pull_request:
branches: [ master ]

jobs:
check_format:
runs-on: macos-latest
steps:
- uses: actions/checkout@v5
- name: Set up JDK 17
uses: actions/setup-java@v5
with:
distribution: temurin
java-version: 17
Comment on lines +13 to +18
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot Aug 29, 2025

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion

Enable Maven cache on macOS jobs too.

Same rationale as Linux: add cache: maven to all setup-java steps.

       - name: Set up JDK 17
-        uses: actions/setup-java@v5
+        uses: actions/setup-java@v5
         with:
           distribution: temurin
           java-version: 17
+          cache: maven

Also applies to: 27-32, 43-48, 59-64, 75-80, 91-96, 107-112, 123-128, 139-144, 155-160

🤖 Prompt for AI Agents 
.github/workflows/build-macos.yml around lines 13-18: the actions/setup-java@v5
step is missing the Maven cache option; add "cache: maven" under the with: block
for that step, and repeat the same change for the other setup-java occurrences
referenced (lines 27-32, 43-48, 59-64, 75-80, 91-96, 107-112, 123-128, 139-144,
155-160) so every setup-java invocation includes with: distribution: temurin,
java-version: 17 and cache: maven.

- name: Check format
run: mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
&& sh ./tools/check_format.sh

test_jraft_core:
needs: check_format
runs-on: macos-latest
steps:
- uses: actions/checkout@v5
- name: Set up JDK 17
uses: actions/setup-java@v5
with:
distribution: temurin
java-version: 17
- name: Maven Test
run: mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
&& (mvn --projects jraft-core test
|| mvn --projects jraft-core test
|| mvn --projects jraft-core test)
Comment on lines +34 to +37
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot Aug 29, 2025

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion

Switch from chained “||” retries to a proper retry with backoff; also build only required modules.

Mirror the Linux suggestions: use a retry step and -pl/-am to avoid building the whole repo per job.

-      - name: Maven Test
-        run: mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
-          && (mvn --projects jraft-core test
-            || mvn --projects jraft-core test
-            || mvn --projects jraft-core test)
+      - name: Maven Test (targeted + retry)
+        shell: bash
+        run: |
+          for i in 1 2 3; do
+            mvn -pl jraft-core -am -Dmaven.javadoc.skip=true -B -V test && break
+            echo "Retry #$i failed; sleeping..." && sleep $((i*10))
+          done

Apply for each module section accordingly.

Also applies to: 50-53, 66-69, 82-85, 98-101, 114-117, 130-133, 146-149, 162-165


test_rheakv_core:
needs: check_format
runs-on: macos-latest
steps:
- uses: actions/checkout@v5
- name: Set up JDK 17
uses: actions/setup-java@v5
with:
distribution: temurin
java-version: 17
- name: Maven Test
run: mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
&& (mvn --projects jraft-rheakv/rheakv-core test
|| mvn --projects jraft-rheakv/rheakv-core test
|| mvn --projects jraft-rheakv/rheakv-core test)

test_rheakv_pd:
needs: check_format
runs-on: macos-latest
steps:
- uses: actions/checkout@v5
- name: Set up JDK 17
uses: actions/setup-java@v5
with:
distribution: temurin
java-version: 17
- name: Maven Test
run: mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
&& (mvn --projects jraft-rheakv/rheakv-pd test
|| mvn --projects jraft-rheakv/rheakv-pd test
|| mvn --projects jraft-rheakv/rheakv-pd test)

test_rpc_grpc_impl:
needs: check_format
runs-on: macos-latest
steps:
- uses: actions/checkout@v5
- name: Set up JDK 17
uses: actions/setup-java@v5
with:
distribution: temurin
java-version: 17
- name: Maven Test
run: mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
&& (mvn --projects jraft-extension/rpc-grpc-impl test
|| mvn --projects jraft-extension/rpc-grpc-impl test
|| mvn --projects jraft-extension/rpc-grpc-impl test)

test_leveldb_log_storage_impl:
needs: check_format
runs-on: macos-latest
steps:
- uses: actions/checkout@v5
- name: Set up JDK 17
uses: actions/setup-java@v5
with:
distribution: temurin
java-version: 17
- name: Maven Test
run: mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
&& (mvn --projects jraft-extension/leveldb-log-storage-impl test
|| mvn --projects jraft-extension/leveldb-log-storage-impl test
|| mvn --projects jraft-extension/leveldb-log-storage-impl test)

test_h2mvstore_log_storage_impl:
needs: check_format
runs-on: macos-latest
steps:
- uses: actions/checkout@v5
- name: Set up JDK 17
uses: actions/setup-java@v5
with:
distribution: temurin
java-version: 17
- name: Maven Test
run: mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
&& (mvn --projects jraft-extension/h2mvstore-log-storage-impl test
|| mvn --projects jraft-extension/h2mvstore-log-storage-impl test
|| mvn --projects jraft-extension/h2mvstore-log-storage-impl test)

test_mapdb_log_storage_impl:
needs: check_format
runs-on: macos-latest
steps:
- uses: actions/checkout@v5
- name: Set up JDK 17
uses: actions/setup-java@v5
with:
distribution: temurin
java-version: 17
- name: Maven Test
run: mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
&& (mvn --projects jraft-extension/mapdb-log-storage-impl test
|| mvn --projects jraft-extension/mapdb-log-storage-impl test
|| mvn --projects jraft-extension/mapdb-log-storage-impl test)

test_chroniclemap_log_storage_impl:
needs: check_format
runs-on: macos-latest
steps:
- uses: actions/checkout@v5
- name: Set up JDK 8
uses: actions/setup-java@v5
with:
distribution: temurin
java-version: 8
- name: Maven Test
run: mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
&& (mvn --projects jraft-extension/chronicle-map-log-storage-impl test
|| mvn --projects jraft-extension/chronicle-map-log-storage-impl test
|| mvn --projects jraft-extension/chronicle-map-log-storage-impl test)

test_bdb_log_storage_impl:
needs: check_format
runs-on: macos-latest
steps:
- uses: actions/checkout@v5
- name: Set up JDK 17
uses: actions/setup-java@v5
with:
distribution: temurin
java-version: 17
- name: Maven Test
run: mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
&& (mvn --projects jraft-extension/bdb-log-storage-impl test
|| mvn --projects jraft-extension/bdb-log-storage-impl test
|| mvn --projects jraft-extension/bdb-log-storage-impl test)
185 changes: 185 additions & 0 deletions .github/workflows/build-windows.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,185 @@
name: build-windows

on:
push:
branches: [ master ]
pull_request:
branches: [ master ]

jobs:
check_format:
runs-on: windows-latest
steps:
- uses: actions/checkout@v5
- name: Set up JDK 17
uses: actions/setup-java@v5
with:
distribution: temurin
java-version: 17
- name: Check format
shell: bash
run: |
mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
sh ./tools/check_format.sh

test_jraft_core:
needs: check_format
runs-on: windows-latest
steps:
- uses: actions/checkout@v5
- name: Set up JDK 17
uses: actions/setup-java@v5
with:
distribution: temurin
java-version: 17
- name: Maven Test
shell: bash
run: |
mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
(mvn --projects jraft-core test \
|| mvn --projects jraft-core test \
|| mvn --projects jraft-core test)

test_rheakv_core:
needs: check_format
runs-on: windows-latest
steps:
- uses: actions/checkout@v5
- name: Set up JDK 17
uses: actions/setup-java@v5
with:
distribution: temurin
java-version: 17
- name: Maven Test
shell: bash
run: |
mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
(mvn --projects jraft-rheakv/rheakv-core test \
|| mvn --projects jraft-rheakv/rheakv-core test \
|| mvn --projects jraft-rheakv/rheakv-core test)

test_rheakv_pd:
needs: check_format
runs-on: windows-latest
steps:
- uses: actions/checkout@v5
- name: Set up JDK 17
uses: actions/setup-java@v5
with:
distribution: temurin
java-version: 17
- name: Maven Test
shell: bash
run: |
mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
(mvn --projects jraft-rheakv/rheakv-pd test \
|| mvn --projects jraft-rheakv/rheakv-pd test \
|| mvn --projects jraft-rheakv/rheakv-pd test)

test_rpc_grpc_impl:
needs: check_format
runs-on: windows-latest
steps:
- uses: actions/checkout@v5
- name: Set up JDK 17
uses: actions/setup-java@v5
with:
distribution: temurin
java-version: 17
- name: Maven Test
shell: bash
run: |
mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
(mvn --projects jraft-extension/rpc-grpc-impl test \
|| mvn --projects jraft-extension/rpc-grpc-impl test \
|| mvn --projects jraft-extension/rpc-grpc-impl test)

test_leveldb_log_storage_impl:
needs: check_format
runs-on: windows-latest
steps:
- uses: actions/checkout@v5
- name: Set up JDK 17
uses: actions/setup-java@v5
with:
distribution: temurin
java-version: 17
- name: Maven Test
shell: bash
run: |
mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
(mvn --projects jraft-extension/leveldb-log-storage-impl test \
|| mvn --projects jraft-extension/leveldb-log-storage-impl test \
|| mvn --projects jraft-extension/leveldb-log-storage-impl test)

test_h2mvstore_log_storage_impl:
needs: check_format
runs-on: windows-latest
steps:
- uses: actions/checkout@v5
- name: Set up JDK 17
uses: actions/setup-java@v5
with:
distribution: temurin
java-version: 17
- name: Maven Test
shell: bash
run: |
mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
(mvn --projects jraft-extension/h2mvstore-log-storage-impl test \
|| mvn --projects jraft-extension/h2mvstore-log-storage-impl test \
|| mvn --projects jraft-extension/h2mvstore-log-storage-impl test)

test_mapdb_log_storage_impl:
needs: check_format
runs-on: windows-latest
steps:
- uses: actions/checkout@v5
- name: Set up JDK 17
uses: actions/setup-java@v5
with:
distribution: temurin
java-version: 17
- name: Maven Test
shell: bash
run: |
mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
(mvn --projects jraft-extension/mapdb-log-storage-impl test \
|| mvn --projects jraft-extension/mapdb-log-storage-impl test \
|| mvn --projects jraft-extension/mapdb-log-storage-impl test)

test_chroniclemap_log_storage_impl:
needs: check_format
runs-on: windows-latest
steps:
- uses: actions/checkout@v5
- name: Set up JDK 8
uses: actions/setup-java@v5
with:
distribution: temurin
java-version: 8
- name: Maven Test
shell: bash
run: |
mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
(mvn --projects jraft-extension/chronicle-map-log-storage-impl test \
|| mvn --projects jraft-extension/chronicle-map-log-storage-impl test \
|| mvn --projects jraft-extension/chronicle-map-log-storage-impl test)

test_bdb_log_storage_impl:
needs: check_format
runs-on: windows-latest
steps:
- uses: actions/checkout@v5
- name: Set up JDK 17
uses: actions/setup-java@v5
with:
distribution: temurin
java-version: 17
- name: Maven Test
shell: bash
run: |
mvn clean install -DskipTests=true -Dmaven.javadoc.skip=true -B -V
(mvn --projects jraft-extension/bdb-log-storage-impl test \
|| mvn --projects jraft-extension/bdb-log-storage-impl test \
|| mvn --projects jraft-extension/bdb-log-storage-impl test)
Loading
Loading