Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit6384052

Browse files
committed
Fix outdated convert_saop_to_hashed_saop comment
In29f45e2, we added support for optimizing the execution of NOTIN(values) by using a hash table instead of a linear search over thearray. That commit neglected to update the header comment forconvert_saop_to_hashed_saop() to mention this fact. Here we fix that.Author: James ColemanDiscussion:https://postgr.es/m/CAAaqYe99NUpAPcxgchGstgM23fmiGjqQPot8627YgkBgNt=BfA@mail.gmail.comBackpatch-through: 15, where29f45e2 was added.
1 parent111d954 commit6384052

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

‎src/backend/optimizer/util/clauses.c

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2117,7 +2117,8 @@ eval_const_expressions(PlannerInfo *root, Node *node)
21172117
*
21182118
* We'll use a hash table if all of the following conditions are met:
21192119
* 1. The 2nd argument of the array contain only Consts.
2120-
* 2. useOr is true.
2120+
* 2. useOr is true or there is a valid negator operator for the
2121+
* ScalarArrayOpExpr's opno.
21212122
* 3. There's valid hash function for both left and righthand operands and
21222123
* these hash functions are the same.
21232124
* 4. If the array contains enough elements for us to consider it to be

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp