Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add max_result_set_writers and excess check #260

Merged
merged 5 commits into from
Feb 25, 2025

Conversation

YoshiakiNishimura
Copy link
Contributor

@YoshiakiNishimura YoshiakiNishimura marked this pull request as ready for review February 20, 2025 13:32
@kuron99
Copy link
Contributor

kuron99 commented Feb 20, 2025

an error message will be displayed とするよりも、must be equal or less than ... のように妥当な値の条件を書くだけがいいと思います。

@@ -75,12 +75,14 @@ parameter=value
|dev_rtx_parallel_scan| ブール(true/false) | RTXの場合にparallel scanを有効にするか。デフォルトはfalse | 開発用のため将来的に削除/変更される可能性あり|
|dev_thousandths_ratio_check_local_first| 整数 | タスクスケジューラのワーカーがローカルタスクキューとstickyタスクキューのどちらを優先的にデキューするかを決めるパラメータ。デキューの実行1000回のうちローカルタスクキューを優先する回数(1以上1000未満)を指定する。デフォルトは100|開発用のため将来的に削除/変更される可能性あり|
|dev_direct_commit_callback| ブール(true/false) | shirakamiのコミット処理を行うスレッドが直接クラインアント通知までおこなうか。`commit_response` が `ACCEPTED` または `AVAILABLE` の場合のみ有効。デフォルトはfalse | 開発用のため将来的に削除/変更される可能性あり|
|scan_default_parallel| 整数 | スキャンタスクの最大並列実行数。デフォルトは4|この値はdefault_partitionsの値より小さくしなければならない。開発用のため将来的に削除/変更される可能性あり|
|scan_default_parallel| 整数 | スキャンタスクの最大並列実行数。デフォルトは4。|この値はdefault_partitions及びmax_result_set_writers 以下でなければならない。開発用のため将来的に削除/変更される可能性あり|
Copy link
Contributor

Choose a reason for hiding this comment

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

default_partitions以下でなければならないというのはまだ有効なんでしょうか?max_result_set_writers以下であればいいのでは?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fix comment
02255c6

対応済み

@YoshiakiNishimura YoshiakiNishimura merged commit 7763bc2 into master Feb 25, 2025
10 checks passed
@YoshiakiNishimura YoshiakiNishimura deleted the max_result_set_writers branch February 25, 2025 10:44
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