Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
WalkthroughThis PR introduces documentation for connecting to CrateDB from Julia applications. It adds a Julia driver card to the connectivity page and creates a new Julia-specific documentation page with setup instructions, code examples using LibPQ.jl, and Cloud connection details. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
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. Comment |
ea2e8d0 to
d3cb2a6
Compare
kneth
left a comment
There was a problem hiding this comment.
Let's discuss https://github.com/crate/tech-content/issues/124#issuecomment-3485676563 before merging
|
I think it's nice to have a dedicated page about Julia, so that all downstream entities (humans and machines) can easily learn how to connect to CrateDB using the PostgreSQL interface, and use a ready-to-run example to exercise it right away. In the long run, this will also improve network effects, because CrateDB gains a dedicated representation within the corresponding search- and vectorspaces. Currently, relevant search keywords like cratedb julia will not yield concise results, but this will improve a few days after merging this patch. |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@docs/connect/julia/index.md`:
- Line 28: Update the cloud connection example that constructs the LibPQ
connection string so it omits the database path component; locate the
LibPQ.Connection call (the line assigning conn = LibPQ.Connection(...)) that
currently contains "/doc" in the URL and remove "/doc" so the URI ends with
"/?sslmode=require", matching the local example and other language docs and
CrateDB's schema model.
docs/connect/julia/index.md
Outdated
| :::{rubric} CrateDB Cloud | ||
| ::: | ||
|
|
||
| For connecting to CrateDB Cloud, use the `sslmode=require` |
There was a problem hiding this comment.
same comment as in the other PR, to not indicate this is only for Cloud.
d73ce49 to
69e8baf
Compare
docs/connect/index.md
Outdated
| :::: | ||
|
|
||
| ::::{grid-item-card} | ||
| <<<<<<< HEAD |
There was a problem hiding this comment.
please resolve the conflicts here.
1eec2c8 to
2813777
Compare
About
What the title says.
Preview
https://cratedb-guide--417.org.readthedocs.build/connect/julia/