CockroachDB support

[sergeyshaykhullin]

Sqlx has a known issue with transations and MARS
launchbadge/sqlx#933

Does sea orm has a workaround to do multiple inserts inside single transaction with CockroachDB?

tokio-postgres has no problems, so it might be fixed on client side somehow without waiting for cockroachdb/cockroach#40195

[billy1624]

Hey @sergeyshaykhullin, a workaround suggested on launchbadge/sqlx#933 (comment). This imply you can use .all() to avoid the error.

For example,

let chocolate: Vec<cake::Model> = Cake::find()
    .filter(cake::Column::Name.contains("chocolate"))
    .order_by_asc(cake::Column::Name)
    .all(db)  // Select `all` instead of `one`
    .await?;

https://www.sea-ql.org/SeaORM/docs/basic-crud/select#find-with-conditions-and-orders

[sergeyshaykhullin]

I need an insert

[billy1624]

I need an insert

Oh, right.

[billy1624]

Btw... can I take a look at the error messages?

[sergeyshaykhullin]

Yes

1. You need a running cockroachdb

version: "2"

services:
  crdb:
    container_name: crdb
    image: cockroachdb/cockroach:latest
    command: [ start-single-node, --insecure ]
    ports: [ 8080:8080, 26257:26257 ]
    volumes: [ crdb_data:/cockroach/cockroach-data ]

volumes:
  crdb_data:
    driver: local

2. Connection string

postgresql://root@127.0.0.1:26257/defaultdb?sslmode=disable

3. And something with the insert

sea-orm/examples/axum_example/src/main.rs

Line 131 in f418c4e

async fn create_post(

To reproduce you will need to wrap create_post with transaction

I moved all prototypes in trash folder a week ago, so i can't show something. But the error was same as here launchbadge/sqlx#933 unimplemented: multiple active portals not supported

[billy1624]

Thanks!! I will try reproduce it tmr morning. Gonna sleep now lol

[billy1624]

Hey @sergeyshaykhullin, I manage to insert rows into ChroachDB without hitting any errors. Check #531

[sergeyshaykhullin]

I can't see begin transaction here 😅

[billy1624]

Ok, now we have transaction, 340e3a0. Works as well loll

[sergeyshaykhullin]

@billy1624 Hmm, interesting... You need multiple inserts to catch this error...

[sergeyshaykhullin]

Also i found another one)

[billy1624]

Oh. Okay, hidden bug...

[AbhijithGanesh]

What all tasks need to be implemented for supporting Cockroach DB? It'd really be useful for new-contributors to learn

[billy1624]

What all tasks need to be implemented for supporting Cockroach DB? It'd really be useful for new-contributors to learn

Hey @AbhijithGanesh, do you mean what are the todo for supporting CockroachDB?

[mckingho]

unimplemented: multiple active portals not supported issue will likely be fixed in CockroachDB v22.2 according to launchbadge/sqlx#933 (comment)

[billy1624]

Thanks! @mckingho for the updates!

[tyt2y3]

I don't think we have the engineering resources (in terms of CI and manpower) to support CockroachDB at this point.
If there is enough interest let's do a crowdfunding may be.
It's something I think definitely a good addition to the SeaQL ecosystem.

[m-yosefpor]

I've noticed that the issue in CockroachDB (#40195) has been fixed. Given this update, is there any change in the priority of supporting CockroachDB with SeaORM? We are particularly interested in this feature and would appreciate any information on planned developments or necessary steps to move forward. Thank you for your consideration.

[NewtTheWolf]

Hello, also here to ask what would be exactly needed to Support CockroachDB? is it possible to Discuss what exactly would be needed?

So i just run into an Error while running a Migration with SeaORM v1

Execution Error: error returned from database: at or near "]": syntax error: unimplemented: this syntax thats not an Error of the Migration directly more what comes "after" the Mig, that SeaORM tries to write the Migrated files into its own Table (i think so)