Skip to content

fix: Allow Iterable[Table] in concat_tables#203

Merged
zen-xu merged 1 commit intozen-xu:mainfrom
dangotbanned:concat-tables-iterable
Apr 14, 2025
Merged

fix: Allow Iterable[Table] in concat_tables#203
zen-xu merged 1 commit intozen-xu:mainfrom
dangotbanned:concat-tables-iterable

Conversation

@dangotbanned
Copy link
Copy Markdown
Contributor

Per the docs (https://arrow.apache.org/docs/python/generated/pyarrow.concat_tables.html)

tables : iterable of pyarrow.Table objects

dangotbanned added a commit to narwhals-dev/narwhals that referenced this pull request Apr 12, 2025
@zen-xu zen-xu merged commit 067c042 into zen-xu:main Apr 14, 2025
2 checks passed
@dangotbanned dangotbanned deleted the concat-tables-iterable branch April 14, 2025 08:24
dangotbanned added a commit to narwhals-dev/narwhals that referenced this pull request Apr 14, 2025
* refactor: refactor: Simplify `ArrowNamespace.concat`

Related #2368

* chore(typing): Ignore stub issues

Fixed in zen-xu/pyarrow-stubs#203

* cov

https://github.com/narwhals-dev/narwhals/actions/runs/14422362924/job/40446526457?pr=2381

* refactor: Implement `EagerNamespace.concat`

* refactor: Avoid redundant unique check

Has no coverage, would be caught later with a nicer error anyway w/ https://github.com/narwhals-dev/narwhals/blob/41871f775589359c563150526d03935449709d7d/narwhals/utils.py#L1466-L1475

* refactor: Consistently use `chain.from_iterable`

* chore: Remove notes

One became a type, the other is moving to github
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