Maintenance: Remove doc as database name across the board#517
Conversation
The default CrateDB database is called `crate`. `doc` is certainly wrong (default schema name), so let's remove it. If we would use `crate` instead, it would add more `crate` noise, and also suggest that the reader may use a different database name instead, which currently is not possible. So, the only viable conclusion is to remove it, unless there are different proposals.
WalkthroughThis PR removes the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Repository UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (6)
💤 Files with no reviewable changes (3)
🧰 Additional context used🧠 Learnings (8)📓 Common learnings📚 Learning: 2025-10-19T19:21:49.864ZApplied to files:
📚 Learning: 2025-10-16T10:58:18.401ZApplied to files:
📚 Learning: 2025-08-14T19:02:43.308ZApplied to files:
📚 Learning: 2025-10-20T21:49:18.785ZApplied to files:
📚 Learning: 2025-06-05T14:29:15.512ZApplied to files:
📚 Learning: 2025-08-05T07:14:57.416ZApplied to files:
📚 Learning: 2025-10-08T01:34:18.867ZApplied to files:
🪛 markdownlint-cli2 (0.18.1)docs/connect/fsharp/index.md92-92: Bare URL used (MD034, no-bare-urls) ⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
🔇 Additional comments (6)
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 |
surister
left a comment
There was a problem hiding this comment.
Fair enough, nothing agains't this
About
The default CrateDB database is called
crate.docis certainly wrong (default schema name), so let's remove it.Synopsis
Thoughts
If we would use
crateinstead, it would add morecratenoise, and also suggest that the reader may use a different database name instead, which currently is not possible. So, the only viable conclusion is to remove it, unless there are different proposals.References
docas database name across the board #475