8000 SQLAlchemy: Ignore `FOR UPDATE` clauses · Issue #95 · crate/sqlalchemy-cratedb · GitHub
[go: up one dir, main page]

Skip to content
SQLAlchemy: Ignore FOR UPDATE clauses #95
@amotl

Description

@amotl

About

At crate/mlflow-cratedb#7, we discovered that CrateDB does not support SELECT ... FOR UPDATE statements.

Details

mlflow-cratedb has a monkeypatch, which adds a visitor to the CrateDB SQLAlchemy dialect to just omit that clause.

Proposal

The monkeypatch needs to be converted into a real patch, might be supported by a corresponding software tests to verify the FOR UPDATE clause does not get rendered into the SQL statement, and a bit of documentation may also be added - one sentence is probably enough.

Followup

Determine if this is an applicable candidate to raise on the crate/crate issue tracker.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      0