Conversation
amotl
added
the
cross linking
|
Warning Rate limit exceeded@amotl has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 7 minutes and 34 seconds before requesting another review. ⌛ 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 selected for processing (5)
WalkthroughThis pull request adds comprehensive Go driver documentation to CrateDB, introducing pgx, pq, and KSQL packages as connectivity options. It updates the main connect index page to include a new Go card and navigation entry, while creating dedicated documentation pages for each driver with usage examples, features, and cloud connection instructions. Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~20 minutes
Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
amotl
force-pushed
the
go
branch
3 times, most recently
from
a5568d4 to
8f7a983
Compare
amotl
force-pushed
the
go
branch
2 times, most recently
from
6e459e5 to
6e33ebd
Compare
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
This comment was marked as resolved.
This comment was marked as resolved.
docs/connect/go/index.md
Outdated
| :::{rubric} About | ||
| ::: | ||
|
|
||
| [pgx] is a pure Go driver and toolkit for PostgreSQL. |
There was a problem hiding this comment.
I just discovered lib/pq, a pure Go PostgreSQL driver for database/sql.
There was a problem hiding this comment.
0c1d156 adds information about it. /cc @coderabbitai
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Show the commit details for 0c1d156b9c
git show 0c1d156b9c
Length of output: 2003
@amotl, thanks for the heads up! I've reviewed commit 0c1d156.
Great addition! The documentation now covers both major PostgreSQL drivers for Go:
- pgx: A more modern, feature-rich driver with advanced features like connection pooling
- pq: The traditional, widely-adopted
database/sqldriver
The new pq section follows the same clear structure as the pgx section with:
- Basic usage example showing standard
database/sqlpatterns - Connection string examples for both local and CrateDB Cloud setups
- Proper error handling guidance (using
sslmode=disablefor local,sslmode=requirefor cloud)
This gives users flexibility to choose based on their needs - pgx for more advanced features or pq for traditional database/sql compatibility. The documentation is comprehensive and consistent! 👍
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
amotl
force-pushed
the
go
branch
4 times, most recently
from
58488ca to
776e417
Compare
seut
left a comment
seut
left a comment
There was a problem hiding this comment.
Looks good to me, I guess all code is verified to work as expected?
3 participants
About
What the title says.
Preview
https://cratedb-guide--406.org.readthedocs.build/connect/go/