Skip to content

Conversation

ydbot
Copy link
Collaborator

@ydbot ydbot commented Oct 15, 2025

PR was created by rightlib sync workflow run

andrey-khropov and others added 30 commits October 11, 2025 03:59
commit_hash:17b4088b80f1a7446525ed8eebf0566b3e3a7888
commit_hash:5be5816fa0d06087b51636b3acfaf53350e750da
Initial commit
commit_hash:d8d8a160ba64cc1e63e174430aec262d2359cf4e
commit_hash:5248f2af916d2474dfa485d3891e11f3b43d395b
commit_hash:97e25b8bd6a13444872620468246b26bba34ac22
commit_hash:567459b5b5eab7484ce33df9b5d2dc265d249ac7
Update tools: os_ymake, ymake
commit_hash:111186dda31d413702bdf419f07f83e378a881ed
Перенёс все ограничения на один механихм. Без этого ограничения про два отдельных файла в build умели перетирать ограничения на вложенны поддиректории.
commit_hash:2e5519f667ede068b45e40e0de1f7e92b79e0cf0
commit_hash:c0fc049f490c77f6663fa43931ad6a975922ef85
Update tools: ya_bin, os_ya, test_tool, os_test_tool
commit_hash:7f90cc5eee406b86163f41b70d41699fb08df4f0
commit_hash:cbb6bbe39c72e2322d57a38ded86cda07c68a419
commit_hash:da571bf1be08eab31e4940877506015e4d9122bf
commit_hash:4a9a87818416b167f43bcf622d3fa04584b8c218
commit_hash:9ef7c882d353697a1850f3b4f3155d7e34ad2aa0
commit_hash:0a2dda0b5dc87a8fccae9081a099d1fc2f1d06e1
commit_hash:c106c0b4122e4493aa70f0007e98dff850405f05
commit_hash:84bc074d576d0c9f0bcb29902ebbec3c9e040405
commit_hash:590666972371282e849bf6155969ac9aae4d06ec
commit_hash:7ea75a93d34c44851ea8dc7187c3a47593a1e63f
commit_hash:2616fa918b284019168fb12ab861499222d40647
commit_hash:61060f12ca550cc17700262f3d71a035558034f0
commit_hash:accc49558635e301bb0987c536dc15e8769f52e4
commit_hash:3f9ceb630b0ff9fe66db62cba7d3d73f9d4f86b5
commit_hash:9d9f0cc89424b60fbdf176953e5e0782e3a9cd3d
Add extend glob project
commit_hash:2b9549580a69f368df20b49e93d1a0115b8f9a77
Current `SQLv1` translation produces relatively low level `YQLs`
constructions that are hard to match during later optimizations, for
example, subqueries unnesting. Also it assumes that expressions are only
depend on corresponding source row, which is not true for correlated
subqueries. Both limitations blocks currelated subqueries implementation.

Although the problem exists for `SQLv1`, it is already solved for `PG`
syntax. There PostgreSQL-produced AST is converted to special `YQLs`
"bulk select" node called `PgSelect`. It is more declarative and
expanded later than translation. This fact helped to support correlated
subqueries, which with `PgSelect` are type-checkable and decorrelatable
(not generally, but heuristically).

This patch is the first step forward to "bulk select" translation
for `SQLv1`. As there are a lot of code already written for `PgSelect`
and `PG` and `SQLv1` relatively similar, I decided to rebrand `PgSelect`
into more general `SqlSelect` node. It seems that for some near future
goals `PgSelect` should be enough.

There was 2 problems solved:

1. `PgSelect` comes with `OrderedColumns` by default and its
   implementation tightly coupled with it.
2. `PgSelect` does Pg type casts.

This patch contains following changes:

- Added pragma `YqlSelect = 'disable' | 'auto' | 'force'`
- Added `YqlSelect` (`PgSelect` alias) translation
- Changed `PgSelect` wrappers to support `YqlSelect`
- Changed `PgSelect` expanders to support `YqlSelect`
commit_hash:8a55d63e06c22592b2029dd260bbd259194e92dc
* Changelog entry
  Type: feature
  Component: core

Make TAsyncExpiringCache sharded
commit_hash:b16531dd3fe638f878a6d906c25065e8b7df097c
commit_hash:ef97148212b5398ba2ed413d6056b6b8cf935cd8
commit_hash:bc75ab7ba0ee5a6571045c99062e8d4a996d16dd
commit_hash:013178051e5e4c0a99ded31893069fb928ba18f9
robot-piglet and others added 11 commits October 14, 2025 15:06
commit_hash:4a09f751cbc56778a76c45c92db1cfd783e3f574
commit_hash:2f5a44083a78a69d4d717b90d401af6f017e990d
Update tools: black
commit_hash:3ed401b3a410a02fdc38c17bc73be874adafc3f2
commit_hash:d40950630c21d95fa780df99e76f713cc694e705
commit_hash:3e2ff28f1560b865bf5a627c19f5f612dfd162ab
commit_hash:05a14bd2abc93819a93ac9adfe6bcf3c28711cbc
commit_hash:541f4e41807099de85307a15ce70dd6da9d952ff
commit_hash:ed70cd729998e0329c62e9c846104cc0529bf8d0
commit_hash:188f3c14b56695a6403d3c223755bb265feb008b
commit_hash:1bc8f82703745c9c7e06f2239735548ccd65997c
@ydbot ydbot requested a review from a team as a code owner October 15, 2025 00:52
@ydbot ydbot added rightlib automerge PR created by sync workflow and will be automerged labels Oct 15, 2025
Copy link

🟢 2025-10-15 00:55:17 UTC The validation of the Pull Request description is successful.

Copy link

github-actions bot commented Oct 15, 2025

2025-10-15 00:56:09 UTC Pre-commit check linux-x86_64-relwithdebinfo for fae812a has started.
2025-10-15 00:56:23 UTC Artifacts will be uploaded here
2025-10-15 01:03:23 UTC ya make is running...

Copy link

github-actions bot commented Oct 15, 2025

2025-10-15 00:56:31 UTC Pre-commit check linux-x86_64-release-asan for fae812a has started.
2025-10-15 00:56:45 UTC Artifacts will be uploaded here
2025-10-15 01:04:05 UTC ya make is running...
🟡 2025-10-15 04:18:21 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
20109 19597 0 186 312 14

🟢 2025-10-15 04:18:27 UTC Build successful.
🟡 2025-10-15 04:18:50 UTC ydbd size 3.8 GiB changed* by +901.6 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 8336376 merge: fae812a diff diff %
ydbd size 4 031 971 560 Bytes 4 032 894 752 Bytes +901.6 KiB +0.023%
ydbd stripped size 1 497 479 296 Bytes 1 497 783 456 Bytes +297.0 KiB +0.020%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

@ydbot
Copy link
Collaborator Author

ydbot commented Oct 15, 2025

Check checks_integrated failed. All future check are suspended, please remove the automerge-blocked label to enable checks. Sync workflow logs can be found here.

@ydbot ydbot added the automerge-blocked PR check failed and it will be ignored by automerge label Oct 15, 2025
Copy link

github-actions bot commented Oct 15, 2025

2025-10-15 04:26:45 UTC Pre-commit check linux-x86_64-relwithdebinfo for efee4df has started.
2025-10-15 04:26:58 UTC Artifacts will be uploaded here
2025-10-15 04:34:11 UTC ya make is running...
🟡 2025-10-15 06:30:42 UTC Some tests failed, follow the links below. Going to retry failed tests...

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
43498 40710 0 7 2754 27

2025-10-15 06:30:52 UTC ya make is running... (failed tests rerun, try 2)
🟡 2025-10-15 06:46:14 UTC Some tests failed, follow the links below. Going to retry failed tests...

Ya make output | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
588 (only retried tests) 564 0 1 0 23

2025-10-15 06:46:17 UTC ya make is running... (failed tests rerun, try 3)
🟢 2025-10-15 07:00:03 UTC Tests successful.

Ya make output | Test bloat | Test bloat | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
565 (only retried tests) 544 0 0 0 21

🟢 2025-10-15 07:00:06 UTC Build successful.
🟡 2025-10-15 07:00:25 UTC ydbd size 2.3 GiB changed* by +512.6 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: 8336376 merge: efee4df diff diff %
ydbd size 2 423 463 432 Bytes 2 423 988 336 Bytes +512.6 KiB +0.022%
ydbd stripped size 515 882 568 Bytes 515 997 032 Bytes +111.8 KiB +0.022%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

Copy link

github-actions bot commented Oct 15, 2025

2025-10-15 04:27:19 UTC Pre-commit check linux-x86_64-release-asan for efee4df has started.
2025-10-15 04:27:34 UTC Artifacts will be uploaded here
2025-10-15 04:35:02 UTC ya make is running...
🟡 2025-10-15 06:49:10 UTC Some tests failed, follow the links below. This fail is not in blocking policy yet

Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
20109 19504 0 273 314 18

🟢 2025-10-15 06:49:17 UTC Build successful.
🟡 2025-10-15 06:49:45 UTC ydbd size 3.8 GiB changed* by +901.6 KiB, which is >= 100.0 KiB vs main: Warning

ydbd size dash main: a6ba11e merge: efee4df diff diff %
ydbd size 4 032 011 928 Bytes 4 032 935 184 Bytes +901.6 KiB +0.023%
ydbd stripped size 1 497 497 056 Bytes 1 497 801 280 Bytes +297.1 KiB +0.020%

*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation

@PavelEfarinov PavelEfarinov removed the automerge-blocked PR check failed and it will be ignored by automerge label Oct 15, 2025
@ydbot ydbot merged commit e0234c3 into main Oct 15, 2025
20 of 23 checks passed
@ydbot
Copy link
Collaborator Author

ydbot commented Oct 15, 2025

The PR was successfully merged into main using workflow

@ydbot ydbot deleted the merge-rightlib-251015-0050 branch October 15, 2025 07:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge PR created by sync workflow and will be automerged rightlib

Projects

None yet

Development

Successfully merging this pull request may close these issues.