Skip to content

Conversation

@amotl
Copy link
Contributor

@amotl amotl commented Jul 3, 2025

Problem

When using uv pip install --upgrade ingestr, only version v0.13.52 gets installed.

Details

Adding support for CrateDB as a source adapter per GH-232 added the dependency sqlalchemy-cratedb==0.42.0.dev2, which is apparently problematic in some cases.

To install the latest version of ingestr with sqlalchemy-cratedb==0.42.0.dev2 successfully, uv needs the --prerelease=allow option. Otherwise, it will install the much older ingestr version v0.13.52.

Solution

Downgrade to a previous GA version of sqlalchemy-cratedb (v0.41.0), to mitigate any such obstacles.

Outlook

Note to self: We'll need to wrap up another patch'es ingredients to fully conclude the operation making 0.42.0 available.

References

@amotl
Copy link
Contributor Author

amotl commented Jul 4, 2025

To install the latest version of ingestr with sqlalchemy-cratedb==0.42.0.dev2 successfully, uv needs the --prerelease=allow option. Otherwise, it will install the much older ingestr version v0.13.52.

In this spirit, the impact is also on users not using the CrateDB adapter. Apologies for introducing this hiccup the other day.

@amotl amotl force-pushed the cratedb-sqlalchemy-no-prerelease branch from e0f1017 to b6afc90 Compare July 7, 2025 10:43
sqlalchemy-cratedb v0.42.0.dev2 needs uv's `--prerelease=allow` option
when installing ingestr. Otherwise, it will install the much older
version v0.13.52.
@amotl amotl force-pushed the cratedb-sqlalchemy-no-prerelease branch from b6afc90 to d1163fc Compare July 7, 2025 10:45
@amotl amotl changed the title Chore: Downgrade to sqlalchemy-cratedb==0.41.0 GA Dependencies: Downgrade to sqlalchemy-cratedb==0.41.0 GA Jul 7, 2025
@turtleDev turtleDev merged commit a8f16b7 into bruin-data:main Jul 8, 2025
4 checks passed
@amotl
Copy link
Contributor Author

amotl commented Jul 8, 2025

Thanks a stack for merging and releasing so swiftly 💯, and sorry again for the hiccup.

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