Skip to content

Harmonize drivers to crate<2.2 and sqlalchemy-cratedb<0.43#843

Open
amotl wants to merge 1 commit into
crate:mainfrom
zerotired:harmonize-drivers
Open

Harmonize drivers to crate<2.2 and sqlalchemy-cratedb<0.43#843
amotl wants to merge 1 commit into
crate:mainfrom
zerotired:harmonize-drivers

Conversation

@amotl

@amotl amotl commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Problem

One nightly scheduled CI run started failing. A subsystem that still uses SQLAlchemy 1.4 raised this error.

crate.client.exceptions.ProgrammingError:
SQLParseException[line 1:49: no viable alternative at input 'VALUES (%']

Solution

Use previous driver tandem.

References

/cc @bgunebakan, @kneth

@coderabbitai

coderabbitai Bot commented Jun 18, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 1473ec1d-4066-4cb9-a13e-e4fb1481281d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@bgunebakan

Copy link
Copy Markdown

Hi,
Thank you for your suggestion but I don't understand the motivation behind this change. current cratedb-toolkit 0.0.49 version works correctly because it depends on older packages, pipeline is failing because there is paramstyle changes on new packages but this packages is not used in current cratedb-toolkit version so there is no effect on user experience. We should implement new features that introduced in other packages here to use them other than downgrading and be dependent on older packages.

Best

@bgunebakan bgunebakan self-requested a review June 19, 2026 08:41
@amotl

amotl commented Jun 19, 2026

Copy link
Copy Markdown
Contributor Author

Hi Bilal. This patch was meant as a mitigation against a single test slot going south after package releases of crate and sqlalchemy-cratedb that will also display the same behaviour when used in production. It will only fix that very slot, as dependency adjustments are exclusively constrained to the corresponding "extra" section optional-dependencies.pymongo.

If you think this is enough of a signal to conduct further actions, together with the other signal coming from the Fivetran adapter, please go ahead.

Hereby, I am asking to either work around the issue by adding version constraints (this patch), or about your decision to possibly yank crate-python another time.

crate.client.exceptions.ProgrammingError:
SQLParseException[line 1:49: no viable alternative at input 'VALUES (%']
@amotl amotl force-pushed the harmonize-drivers branch from 6960d1d to 848384b Compare June 19, 2026 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants