Skip to content

Comments

Update argument naming#540

Merged
bevzzz merged 1 commit intofeat/async-replication-configfrom
async-replication-config-updates
Feb 23, 2026
Merged

Update argument naming#540
bevzzz merged 1 commit intofeat/async-replication-configfrom
async-replication-config-updates

Conversation

@g-despot
Copy link
Contributor

Fix inconsistency:

  • Rename loggingFrequencyMillis to loggingFrequencySeconds as it is in seconds in Weaviate

Remove replication from argument name:

  • Rename replicationConcurrency to maxWorkers
  • Rename replicationFrequencyMillis to frequencyMillis
  • Rename replicationFrequencyMillisWhilePropagating to frequencyMillisWhilePropagating

Other:

  • Rename nodePingFrequencyMillis to aliveNodesCheckingFrequencyMillis

Copy link

@orca-security-eu orca-security-eu bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca

@g-despot g-despot requested a review from bevzzz February 23, 2026 10:50
@bevzzz bevzzz merged commit 65750f4 into feat/async-replication-config Feb 23, 2026
8 checks passed
@bevzzz bevzzz deleted the async-replication-config-updates branch February 23, 2026 10:58
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