maxRows option for executeStatement is not taken into account
まだ誰も着手していません。
評価
- 難易度
- 3/5
- 見積もり時間
- 1〜2日
- 初心者へのやさしさ
- 45/100
- issue の種類
- バグ
- 明瞭さ
- おおむね明確
- 活発さ
- 停滞
- 技術スタック
- sql, typescript
- 領域
- api
調査の方向性
session.executeStatement エントリポイントから開始し、その maxRows オプションがどのように操作へ渡されるかを追跡します。その経路を fetchChunk と比較し、続いて issue にある TypeScript の再現コードを実行して、fetchAll が maxRows を尊重しつつ、文書化された動作を維持することを確認します。
索引モデルが issue の本文から書いたものです。
説明
Version
I am using the 1.8.4 version
Description
We want to limit the amount of RAM usage. passing a default maxRows option has no effect, the whole query is returned
How to reproduce
const operation = await this.session.executeStatement(
`WITH numbers AS (
SELECT num
FROM (SELECT EXPLODE(SEQUENCE(1, 100)) AS num) AS seq
)
SELECT num
FROM numbers
ORDER BY RAND()`,
{ maxRows: 10 },
);
const fetched = await operation.fetchAll();
console.log(fetched.length);
You will notice here that fetched has a size of 100, not 10.
Workaround
The workaround is to use fetchChunk instead:
const chunk = await operation.fetchChunk({
maxRows: limit - result.length,
});
Please deprecate or fix maxRows options for the executeStatement method of the session object
- 主要言語
- TypeScript
- スター
- 36
- フォーク
- 50
- 平均マージ
- 13時間 46分
- マージ済み PR（30日）
- 9
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
databricks/databricks-sql-nodejs のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
-
engineer-bot
難易度 2/5 1〜3時間 初心者へのやさしさ 64/100
databricks/databricks-sql-nodejs#274 · コメント 1 件 · リアクション 1 件 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
databricks/databricks-sql-nodejs の issue をすべて見る
似ている issue
-
VerificationGate: ATTRIBUTION quote guard never matches a normal quotation (\b around the quote) オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
danielmiessler/LifeOS#2234 ·
-
T: Bug
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
-
難易度 1/5 1時間未満 初心者へのやさしさ 85/100
-
Mend: dependency security vulnerability untriaged
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100