-
Notifications
You must be signed in to change notification settings - Fork 128
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
CockroachDB support #1291
Comments
Quoting cockroachdb/cockroach#23061 (comment) in a thread about CockroachDB vs. PostgreSQL performance:
Now some points about dcrdata:
|
@chappjc is this still a concern? so far there hasn't been a clear use case for cockroachDB, plus dcrdata has been using postgresql for this long. |
No we are ditching cockroach. It's not performant. |
@chappjc there are unused function for cockroachDB, a new issue should be opened to clean-up codebase? |
Yah, that would be fine |
CockroachDB support should be a new feature for 5.0.
I'm fairly certain this is doable since CockroachDB is largely compatible with PostgreSQL, and
github.com/lib/pq
seems to support CockroachDB (see this recent pq PR).However, there are a number of required changes, including:
vin_t
) since cockroach does not support them. CockroachDB support #1294synchronous_commit
and almost all the pg_settings dumped in the table at startup). CockroachDB support #1294CREATE DATABASE dcrdata OWNER dcrdata
.) https://github.com/decred/dcrdata/wiki/CockroachDBSERIAL
in CockroachDB does not create a sequence (by default). CockroachDB support #1294DELETE
in CockroachDB does not support theUSING
clause (like aJOIN
). https://www.cockroachlabs.com/docs/stable/delete.html CockroachDB support #1294ARRAY[]
constructor in CockroachDB require type annotation (e.g.ARRAY[]:::BIGINT[]
with three:
characters for annotation instead of conversion), a concept that does not exist in PostgreSQL). May need to use the literal format likeSELECT '{}'::int[];
or whatever works for both PostgreSQL and CockroachDB. CockroachDB support #1294unknown function: cardinality()
forRetrieveUTXOs
CockroachDB support #1294CASCADE
clause. CockroachDB support #1294https://www.cockroachlabs.com/docs/stable/performance-best-practices-overview.html
to no avail: https://github.com/decred/dcrdata/compare/3625acdcd48121bfe5b80c45a7fb7710cf6bdd40...chappjc:multi-line-inserts?expand=1These changes are just what I can come up with now, so it is very likely that there will be many more roadblocks along the way.
The text was updated successfully, but these errors were encountered: