Skip to content

Support for cockroach db #8662

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.

Sign up for GitHub

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

Draft
RCasatta wants to merge 4 commits into
base: master
Choose a base branch
from
+32 −15
Draft

Support for cockroach db #8662

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
ceb4cca
Some db are stricter with casting
RCasatta Nov 4, 2025
b619ab2
replace _ROWID_ with standard SQL
RCasatta Nov 4, 2025
b90fcd4
support cockroach db 64 bits integer
RCasatta Nov 4, 2025
d584cb9
use OID to determine int column type
RCasatta Nov 7, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion wallet/db.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -943,7 +943,7 @@ static struct migration dbmigrations[] = {
" in_channel_scid"
", COALESCE("
" (SELECT channel_htlc_id FROM channel_htlcs WHERE id = forwarded_payments.in_htlc_id),"
" -_ROWID_"
" -row_number() OVER ()"
Comment on lines -939 to +946
Copy link
Contributor

@rustyrussell rustyrussell Nov 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah I think you missed the translation layer?

See devtools/sql-rewrite.py...

Copy link
Contributor Author

@RCasatta RCasatta Nov 7, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see that sql-rewrite.py rewrite _ROWID_, but I don't get what is exactly the flow to use it, how do I bootstrap a lightning node with cockroach db?

Copy link
Contributor

@rustyrussell rustyrussell Nov 13, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It gets run over all db source files to map anything in a SQL() macro. These get looked up at runtime.

" )"
", out_channel_scid"
", (SELECT channel_htlc_id FROM channel_htlcs WHERE id = forwarded_payments.out_htlc_id)"
Expand Down