Skip to content

TEST: c-questdb-client vs questdb PR #6890 (do not merge)#159

Draft
ideoma wants to merge 2 commits into
mainfrom
test-questdb-6890
Draft

TEST: c-questdb-client vs questdb PR #6890 (do not merge)#159
ideoma wants to merge 2 commits into
mainfrom
test-questdb-6890

Conversation

@ideoma

@ideoma ideoma commented Jun 17, 2026

Copy link
Copy Markdown
Collaborator

Throwaway test branch — do not merge.

Builds QuestDB from the steve-rm-bins branch (questdb/questdb PR #6890, "remove pre-compiled native binaries from the repository") instead of master, to validate that the from-source build path still works for downstream client consumers after the committed native libraries are removed.

Changes:

  • ci/templates/clone_questdb.yaml: clone --branch steve-rm-bins instead of master (redirects all four QuestDB-building jobs).
  • ci/run_tests_pipeline.yaml: add ,build-binaries to the four Maven compile steps, since #6890 removed the committed native libs and the server jar otherwise has no native libraries to load.

Close once results are read.

@coderabbitai

coderabbitai Bot commented Jun 17, 2026

Copy link
Copy Markdown

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c605afbf-f43e-4ee8-89b1-ca8dd832259f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch test-questdb-6890

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.

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.

1 participant