Skip to content

fix(kqp): use ReadTableRanges instead of LookupTable#1140

Merged
ulya-sidorina merged 1 commit intoydb-platform:mainfrom
ulya-sidorina:fix/KIKIMR-20810
Feb 1, 2024
Merged

fix(kqp): use ReadTableRanges instead of LookupTable#1140
ulya-sidorina merged 1 commit intoydb-platform:mainfrom
ulya-sidorina:fix/KIKIMR-20810

Conversation

@ulya-sidorina
Copy link
Collaborator

@ulya-sidorina ulya-sidorina commented Jan 18, 2024

No description provided.

@ulya-sidorina ulya-sidorina force-pushed the fix/KIKIMR-20810 branch 2 times, most recently from b88f829 to 96be887 Compare January 18, 2024 17:51
@github-actions
Copy link

github-actions bot commented Jan 18, 2024 •
edited
Loading

⚪ 2024-01-25 12:03:47 UTC Pre-commit check for 485e35c has started.
⚪ 2024-01-25 12:03:48 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-01-25 12:05:35 UTC Build successful.
⚪ 2024-01-25 12:05:44 UTC Tests are running...
🔴 2024-01-25 13:14:57 UTC Some tests failed, follow the links below.

Test history

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
60105 50804 0 36 9239 26

@github-actions
Copy link

github-actions bot commented Jan 18, 2024 •
edited
Loading

⚪ 2024-01-25 12:06:20 UTC Pre-commit check for 485e35c has started.
⚪ 2024-01-25 12:06:22 UTC Build linux-x86_64-release-asan is running...
🟢 2024-01-25 12:08:14 UTC Build successful.
⚪ 2024-01-25 12:08:23 UTC Tests are running...
🔴 2024-01-25 13:22:04 UTC Test run completed, no test results found for commit 2360165. Please check build logs.
⚫ 2024-01-25 13:22:07 UTC Check cancelled

@ulya-sidorina ulya-sidorina force-pushed the fix/KIKIMR-20810 branch 19 times, most recently from 5fe3429 to 2360165 Compare January 25, 2024 12:01
@ulya-sidorina ulya-sidorina changed the title fix(kqp): use ReadTableRange instead LookupTable fix(kqp): use ReadTable instead LookupTable Jan 25, 2024
@ulya-sidorina ulya-sidorina marked this pull request as ready for review January 25, 2024 13:17
@github-actions
Copy link

github-actions bot commented Jan 25, 2024 •
edited
Loading

⚪ 2024-01-25 13:24:33 UTC Pre-commit check for e949d43 has started.
⚪ 2024-01-25 13:24:35 UTC Build linux-x86_64-release-asan is running...
🟢 2024-01-25 13:26:22 UTC Build successful.
⚪ 2024-01-25 13:26:32 UTC Tests are running...
🔴 2024-01-25 14:02:27 UTC Test run completed, no test results found for commit 8fc522b. Please check build logs.
⚫ 2024-01-25 14:02:29 UTC Check cancelled

@github-actions
Copy link

github-actions bot commented Jan 25, 2024 •
edited
Loading

⚪ 2024-01-25 13:24:57 UTC Pre-commit check for e949d43 has started.
⚪ 2024-01-25 13:24:59 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-01-25 13:26:47 UTC Build successful.
⚪ 2024-01-25 13:26:57 UTC Tests are running...
🔴 2024-01-25 14:02:28 UTC Test run completed, no test results found for commit 8fc522b. Please check build logs.
⚫ 2024-01-25 14:02:31 UTC Check cancelled

@github-actions
Copy link

github-actions bot commented Jan 25, 2024 •
edited
Loading

⚪ 2024-01-25 14:03:40 UTC Pre-commit check for 04cc17d has started.
⚪ 2024-01-25 14:03:42 UTC Build linux-x86_64-relwithdebinfo is running...
🟢 2024-01-25 14:05:32 UTC Build successful.
⚪ 2024-01-25 14:05:41 UTC Tests are running...
🔴 2024-01-25 14:45:56 UTC Test run completed, no test results found for commit e5ff1dc. Please check build logs.
⚫ 2024-01-25 14:45:59 UTC Check cancelled

This was referenced Feb 7, 2024
Merged
Merged
Merged
@ildar-khisambeev ildar-khisambeev mentioned this pull request Feb 8, 2024
Merged
@vitstn vitstn mentioned this pull request Feb 8, 2024
Merged
@niksaveliev niksaveliev mentioned this pull request Feb 9, 2024
Merged
@GrigoriyPA GrigoriyPA mentioned this pull request Feb 9, 2024
Merged
ssmike pushed a commit to ssmike/ydb that referenced this pull request Feb 9, 2024
@ulya-sidorina @ssmike
fix(kqp): use ReadTableRange instead LookupTable (ydb-platform#1140)
cfd51dc
@ssmike ssmike mentioned this pull request Feb 9, 2024
Merged
@pavelvelikhov pavelvelikhov mentioned this pull request Feb 9, 2024
Merged
@niksaveliev niksaveliev mentioned this pull request Feb 12, 2024
Merged
@starlinskiy starlinskiy mentioned this pull request Feb 12, 2024
Closed
@vitstn vitstn mentioned this pull request Feb 12, 2024
Merged
@GrigoriyPA GrigoriyPA mentioned this pull request Feb 12, 2024
Merged
@vitstn vitstn mentioned this pull request Feb 12, 2024
Merged
@starlinskiy starlinskiy mentioned this pull request Feb 12, 2024
Merged
@pavelvelikhov pavelvelikhov mentioned this pull request Feb 13, 2024
Merged
This was referenced Feb 13, 2024
Merged
Merged
Merged
@pavelvelikhov pavelvelikhov mentioned this pull request Feb 15, 2024
Merged
@niksaveliev niksaveliev mentioned this pull request Feb 16, 2024
Merged
@vitstn vitstn mentioned this pull request Feb 16, 2024
Merged
This was referenced Feb 17, 2024
Merged
Merged
Merged
Merged
Merged
@aakulaga-ydb aakulaga-ydb mentioned this pull request Feb 24, 2024
Merged
@GrigoriyPA GrigoriyPA mentioned this pull request Feb 26, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@spuchin spuchin spuchin approved these changes

@gridnevvvit gridnevvvit Awaiting requested review from gridnevvvit

+1 more reviewer

@ssmike ssmike ssmike left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ulya-sidorina @ssmike @spuchin