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.

Sign up for GitHub

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

index_column_diff: increase timeout for too_many_token's tests on Windows #2090

Merged
merged 2 commits into from
Nov 27, 2024
Merged

index_column_diff: increase timeout for too_many_token's tests on Windows #2090

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
546d111
index_column_diff: increase timeout for too_many_token's tests on Win…
otegami Nov 21, 2024
a229ce8
kick ci for the test
otegami Nov 27, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
index_column_diff: increase timeout for too_many_token's tests on Win… 
…dows

In CI, Test: HTTP: reference count: Apache Arrow: chunked step
for Windows failed as follows. This test isn't stable using
the current timeout time.

```
...
--- (expected)
+++ (actual)
@@ -6,6 +6,5 @@
 [[0,0.0,0.0],true]
 column_create Deltas index COLUMN_INDEX|WITH_POSITION PostingLists deltas
 [[0,0.0,0.0],true]
-#|w| [ii][update][one] too many postings: <Deltas.index>: <1>(1): record:<PostingLists>(1:1), n-postings:<131072>, n-discarded-postings:<1>
 index_column_diff Deltas index
 [[0,0.0,0.0],[]]

===============================================================================
[2] index_column_diff/too_many_tokens
  int64_vector                                                56.2620s [failed]
===============================================================================
===============================================================================
...
```

So we will try to extend the timeout from 60 seconds to 120 seconds
for these too_many_token's tests.
  • Loading branch information
@otegami
otegami committed Nov 26, 2024
commit 546d111f68fdab6f962f7c5dde4b611b857b62ff
2 changes: 1 addition & 1 deletion test/command/suite/index_column_diff/too_many_tokens/int64_vector.test
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ column_create PostingLists deltas COLUMN_VECTOR Int64
table_create Deltas TABLE_PAT_KEY Int64
column_create Deltas index COLUMN_INDEX|WITH_POSITION PostingLists deltas

#@timeout 60
#@timeout 120
#@disable-logging
#@generate-series 1 1 PostingLists '{"deltas" => [1] * (0x1ffff + 1)}'
#@enable-logging
Expand Down
2 changes: 1 addition & 1 deletion test/command/suite/index_column_diff/too_many_tokens/reference_vector.test
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ column_create Memos tags COLUMN_VECTOR Tags

column_create Tags memos_tags COLUMN_INDEX|WITH_POSITION Memos tags

#@timeout 60
#@timeout 120
#@disable-logging
#@generate-series 1 1 Memos '{"tags" => ["tag"] * (0x1ffff + 1)}'
#@enable-logging
Expand Down
2 changes: 1 addition & 1 deletion test/command/suite/index_column_diff/too_many_tokens/text.test
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ column_create Diaries content COLUMN_SCALAR LongText
table_create Terms TABLE_PAT_KEY ShortText --default_tokenizer TokenDelimit
column_create Terms index COLUMN_INDEX|WITH_POSITION Diaries content

#@timeout 60
#@timeout 120
#@disable-logging
#@generate-series 1 1 Diaries '{"content" => "a " * (0x1ffff + 1)}'
#@enable-logging
Expand Down
2 changes: 1 addition & 1 deletion test/command/suite/index_column_diff/too_many_tokens/text_multi_sources.test
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ column_create Terms index \
COLUMN_INDEX|WITH_POSITION|WITH_SECTION \
Diaries title,content

#@timeout 60
#@timeout 120
#@disable-logging
#@generate-series 1 1 Diaries '{"title" => "a", "content" => "a " * (0x1ffff + 1)}'
#@enable-logging
Expand Down
2 changes: 1 addition & 1 deletion test/command/suite/index_column_diff/too_many_tokens/text_multi_tokens.test
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ column_create Diaries content COLUMN_SCALAR LongText
table_create Terms TABLE_PAT_KEY ShortText --default_tokenizer TokenDelimit
column_create Terms index COLUMN_INDEX|WITH_POSITION Diaries content

#@timeout 60
#@timeout 120
#@disable-logging
#@generate-series 1 1 Diaries '{"content" => "a b " * (0x1ffff + 1)}'
#@enable-logging
Expand Down
2 changes: 1 addition & 1 deletion test/command/suite/index_column_diff/too_many_tokens/text_vector.test
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ column_create Diaries contents COLUMN_VECTOR Text
table_create Terms TABLE_PAT_KEY ShortText
column_create Terms index COLUMN_INDEX|WITH_POSITION Diaries contents

#@timeout 60
#@timeout 120
#@disable-logging
#@generate-series 1 1 Diaries '{"contents" => ["a"] * (0x1ffff + 1)}'
#@enable-logging
Expand Down
Loading