duckdb / duckdb/duckdb-python

`read_json` accepts a list of paths at runtime but the type stub only allows a single path.

オープン 初心者向け
#576 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る
needs triage
主要言語
Python
スター
186
フォーク
113
平均マージ
13時間 29分
マージ済み PR(30日)
17

説明

### What happens?

Hi duckdb team, big fan of the product!

I'm experiencing a small typing issue:

```python
import duckdb

con = duckdb.connect()
rel = con.read_json(
["a.jsonl", "b.jsonl"],
format="newline_delimited",
)
print(rel.count("*").fetchone())
```

This runs fine and returns the union of both files.

However, pyright throws:
`Argument of type "list[str]" cannot be assigned to parameter "path_or_buffer" of type "str | bytes | PathLike[str] | PathLike[bytes] | IO[bytes] | IO[str]"`

mypy throws a similar error.

Adding `list[str]` (and probably `list[PathLike]`) to `path_or_buffer` in the Python stub would fix this

### To Reproduce

```python
import duckdb

con = duckdb.connect()
rel = con.read_json(
["a.jsonl", "b.jsonl"],
format="newline_delimited",
)
print(rel.count("*").fetchone())
```

### OS:

macOS arm64

### DuckDB Package Version:

1.5.3

### Python Version:

3.10

### Full Name:

Amir Refaee

### Affiliation:

Railtracks

### What is the latest build you tested with? If possible, we recommend testing with the latest nightly build.

I have tested with a stable release

### Did you include all relevant data sets for reproducing the issue?

Not applicable - the reproduction does not require a data set

### Did you include all code required to reproduce the issue?

- [x] Yes, I have

### Did you include all relevant configuration to reproduce the issue?

- [x] Yes, I have

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

まず con.read_json の Python スタブを見つけ、path_or_buffer のアノテーションを確認します。pyright と mypy で再現を確認します。両方の型チェッカーが JSON パスのリストを受け入れ、既存の単一パス形式も引き続き有効であれば完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
python
領域
developer-experience
issue の種類
バグ
難易度
2/5
見積もり時間
1〜3時間
活発さ
静か
明瞭さ
明確に書かれている
初心者へのやさしさ
78/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。