Skip to content

Conversation

@ivasio
Copy link
Contributor

@ivasio ivasio commented Oct 17, 2025

Resolves #3193

  • Bumped the connectorx dependency under python_version >= '3.10' from >=0.4.2 to >=0.4.4 - the arrow_stream return_type does indeed provide functional parity with other backends and potentially a serious memory footprint optimization. At the same time, between 0.4.2 and 0.4.4 connectorx didn't change any dependency/support specification, so we can upgrade to this version "for free"

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Oct 17, 2025

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
docs 7ac6834 Commit Preview URL

Branch Preview URL
Nov 13 2025, 09:47 PM

@ivasio ivasio force-pushed the feat/3193-connectorx-arrow-stream branch from e782320 to 5266e84 Compare November 10, 2025 16:05
@ivasio ivasio changed the title Feat: make arrow_stream default return_type for connectorx backend Feat: support return_type = arrow_stream for connectorx backend Nov 10, 2025
@ivasio ivasio force-pushed the feat/3193-connectorx-arrow-stream branch from c75ea6d to 6468294 Compare November 11, 2025 10:24
@ivasio ivasio marked this pull request as ready for review November 12, 2025 15:04
@ivasio ivasio requested a review from rudolfix November 12, 2025 15:04
Copy link
Collaborator

@rudolfix rudolfix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! thanks for fixing precision tests! I've added my old fix for date64 (botched timestamps) which came back with streaming backend and updated the tests. pls take a look. otherwise ready to merge

@rudolfix rudolfix merged commit dc1a046 into devel Nov 14, 2025
120 of 123 checks passed
@rudolfix rudolfix deleted the feat/3193-connectorx-arrow-stream branch November 14, 2025 18:57
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.

Dlt clubs multiple files into one at Normalization and Extraction step when we use connectorx as backend an arroe_stream return type

3 participants