Skip to content

Comments

update data_substrate: eloqstore upload from memory and adjust cpu when eloqstore is in cloud mode#230

Merged
thweetkomputer merged 5 commits intoeloqdata:eloq-10.6.10from
thweetkomputer:eloqstore-memory-upload
Feb 12, 2026
Merged

update data_substrate: eloqstore upload from memory and adjust cpu when eloqstore is in cloud mode#230
thweetkomputer merged 5 commits intoeloqdata:eloq-10.6.10from
thweetkomputer:eloqstore-memory-upload

Conversation

@thweetkomputer
Copy link
Collaborator

@thweetkomputer thweetkomputer commented Feb 9, 2026

Summary by CodeRabbit

  • Chores
    • Updated an internal dependency pointer; no change to application behavior or public APIs.
    • Increased an internal store buffer pool size (8MB → 100MB), which may improve throughput and reduce latency under high load.
    • No exported/public entities were modified; low review effort expected.

@coderabbitai
Copy link

coderabbitai bot commented Feb 9, 2026

Walkthrough

Updated the data_substrate submodule pointer from commit 3be56b825812b9c4183614eff8b01acf87e0a26e to aa2703c81b190dd7a41a82e1c1c04170b6c28ffd and changed eloq_store_buffer_pool_size in concourse/scripts/dss_server.ini from 8MB to 100MB.

Changes

Cohort / File(s) Summary
Submodule Update
data_substrate
Submodule pointer advanced from 3be56b825812b9c4183614eff8b01acf87e0a26e to aa2703c81b190dd7a41a82e1c1c04170b6c28ffd; pointer-only change.
Config change
concourse/scripts/dss_server.ini
In [store] section, eloq_store_buffer_pool_size changed from 8MB to 100MB.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • liunyl
  • xiexiaoy
  • lzxddz

Poem

🐇 I hopped through commits, light and spry,
A pointer shifted, configs reached the sky,
Buffer pools grew, a joyful cheer,
I nibble code and clap my ear,
Carrots crunch — deploy is nigh!

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main changes: updating data_substrate submodule and adjusting eloq_store_buffer_pool_size configuration for cloud mode operations.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

No actionable comments were generated in the recent review. 🎉


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@thweetkomputer thweetkomputer force-pushed the eloqstore-memory-upload branch from f051a97 to af8744b Compare February 11, 2026 05:53
@thweetkomputer thweetkomputer changed the title update data_substrate: eloqstore upload from memory update data_substrate: eloqstore upload from memory and and adjust cpu when eloqstore is in cloud mode Feb 11, 2026
@thweetkomputer thweetkomputer changed the title update data_substrate: eloqstore upload from memory and and adjust cpu when eloqstore is in cloud mode update data_substrate: eloqstore upload from memory and adjust cpu when eloqstore is in cloud mode Feb 11, 2026
@thweetkomputer thweetkomputer force-pushed the eloqstore-memory-upload branch from 4cd49ef to fc679b7 Compare February 11, 2026 10:55
@thweetkomputer thweetkomputer merged commit e990447 into eloqdata:eloq-10.6.10 Feb 12, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants