Skip to content

Conversation

Git-Yang
Copy link
Contributor

@Git-Yang Git-Yang commented Sep 6, 2021

#94

Optimization logic
According to the configuration enableBatchPush, choose whether to synchronize data through batch. When a push request is received, it is judged whether it is a batch push according to the number of messages sent.

Signed-off-by: zhangyang21 <zhangyang21@xiaomi.com>
@Git-Yang
Copy link
Contributor Author

Git-Yang commented Sep 8, 2021

Verification Test

Test one:enableBatchPush(false --> true)

  • Initialization: The cluster has not yet started BatchPush
  • Upgrade content: enable enableBatchPush
    • Unoptimized version test: enableBatchPush=true, upgrade a slave (machine 05)
      选区_999(3028)
    • Optimized version test: enableBatchPush=true, upgrade a slave (machine machine)
      选区_999(3029)

Test two:enableBatchPush(true --> false)

  • Initialization: Start BatchPush in the cluster
  • Upgrade content: turn off enableBatchPush
    • Unoptimized version test: enableBatchPush=false, upgrade a slave (machine 05)
      选区_999(3030)
    • Optimized version test: enableBatchPush=false, upgrade a slave (machine machine)
      选区_999(3031)

@RongtongJin RongtongJin merged commit 10cacd4 into openmessaging:master Sep 26, 2021
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