Skip to content

custom-scan: Support multiple conditions#840

Draft
abetomo wants to merge 1 commit intopgroonga:mainfrom
abetomo:cs-support-multiple-condition
Draft

custom-scan: Support multiple conditions#840
abetomo wants to merge 1 commit intopgroonga:mainfrom
abetomo:cs-support-multiple-condition

Conversation

@abetomo
Copy link
Copy Markdown
Member

@abetomo abetomo commented Oct 6, 2025

Columns that are not included in the index are stored separately and evaluated in ExecQual() for filtering.

  • sql/custom-scan/multiple-condition/mix-and.sql
    • It is when there are columns that are not included in the index.
    • This commit adds support for this case.
  • sql/custom-scan/multiple-condition/index-only-and.sql
    • Conditions with multiple columns included in the index were already supported.
    • It worked as expected even without this commit.

* sql/custom-scan/multiple-condition/mix-and.sql
  * It is when there are columns that are not included in the index.
  * This commit adds support for this case.
* sql/custom-scan/multiple-condition/index-only-and.sql
  * Conditions with multiple columns included in the index were already supported.
  * It worked as expected even without this commit.
@kou
Copy link
Copy Markdown
Member

kou commented Oct 6, 2025

自分で処理できない条件はPostgreSQLにおまかせということはできないの?
自分で処理できない条件も自分でPostgreSQLの処理を呼び出さなきゃいけないの?

@abetomo
Copy link
Copy Markdown
Member Author

abetomo commented Oct 6, 2025

やはりPostgreSQLにおまかせしたいですよね。
もう少し調べます。

@abetomo abetomo marked this pull request as draft October 6, 2025 08:47
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