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

[feat] curvebs: The formatting and deployment phases support separate… #238

Open
wants to merge 3 commits into
base: tombstone
Choose a base branch
from

Conversation

zyb521
Copy link

@zyb521 zyb521 commented Jun 12, 2023

… partitioning of WAL and data

change format.yaml,add format_type,only can be “wal” or “data”

host:
  - machine1
  - machine2
  - machine3
disk:
  - data:/dev/sda:/data/chunkserver0:10  # fortmat_type:device:mount_path:format_percent
  - data:/dev/sdb:/data/chunkserver1:10
  - data:/dev/sdc:/data/chunkserver2:10
  - wal:/dev/nvme0n1p1:/data/wal/chunkserver0:10
  - wal:/dev/nvme0n1p2:/data/wal/chunkserver1:10
  - wal:/dev/nvme0n1p3:/data/wal/chunkserver2:10

change topology.yaml,add wal_dir config item,and change some default config

chunkserver_services:
  config:
    listen.ip: ${service_host}
    listen.port: 820${service_replicas_sequence}  # 8200,8201,8202
    data_dir: /data/chunkserver${service_replicas_sequence}  # /data/chunkserver0, /data/chunksever1
    wal_dir: /data/wal/chunkserver${service_replicas_sequence}  # /wal/chunkserver0, /wal/chunksever1
    copyset.snapshot_interval_s: 300
    walfilepool.use_chunk_file_pool: false
    walfilepool.enable_get_segment_from_pool: true
    walfilepool.segment_size: 16777216

@@ -236,6 +240,9 @@ type (
ChunkfilePoolRootDir string // /curvebs/chunkserver/data
Copy link

@legionxiong legionxiong Jun 21, 2023

Choose a reason for hiding this comment

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

missing WalfilePoolRootDir initialization, else the layout.WalfilePoolRootDir is an empty string.

Copy link

@legionxiong legionxiong Jun 21, 2023

Choose a reason for hiding this comment

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

the same as WalfilePoolDir and WalfilePoolMetaPath.

Copy link
Author

Choose a reason for hiding this comment

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

fixed

@zyb521 zyb521 force-pushed the develop branch 2 times, most recently from b0422f4 to e2eea99 Compare July 20, 2023 02:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants