-
Notifications
You must be signed in to change notification settings - Fork 1
Driver: Add page about Julia #417
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
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.
matriv
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thx!
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.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same comment as in the other PR, to not indicate this is only for Cloud.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks. Fixed.
d73ce49 to
69e8baf
Compare
docs/connect/index.md
Outdated
| :::: | ||
|
|
||
| ::::{grid-item-card} | ||
| <<<<<<< HEAD |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please resolve the conflicts here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Resolved, thanks.
1eec2c8 to
2813777
Compare
About
What the title says.
Preview
https://cratedb-guide--417.org.readthedocs.build/connect/julia/