Skip to content

basic example query service #13477

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

Jump to bottom
Merged
merged 47 commits into from
Mar 11, 2025
Merged

basic example query service #13477

merged 47 commits into from
Mar 11, 2025

Conversation

rekby
Copy link
Member

@rekby rekby commented Jan 17, 2025 •
edited
Loading

Changelog entry

...

Changelog category

  • Documentation (changelog entry is not required)

Additional information

...

@rekby rekby requested a review from a team as a code owner January 17, 2025 09:33
@github-actions github-actions bot added new-feature documentation Improvements or additions to documentation and removed new-feature labels Jan 17, 2025
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 17, 2025 •
edited
Loading

⚪ 2025-01-17 09:37:24 UTC Pre-commit check linux-x86_64-relwithdebinfo for 02d9186 has started.
⚪ 2025-01-17 09:37:35 UTC Artifacts will be uploaded here
⚪ 2025-01-17 09:39:56 UTC ya make is running...
🟢 2025-01-17 09:41:16 UTC Tests successful.

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-01-17 09:41:22 UTC Build successful.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 17, 2025 •
edited
Loading

⚪ 2025-01-17 09:40:07 UTC Pre-commit check linux-x86_64-release-asan for 02d9186 has started.
⚪ 2025-01-17 09:40:18 UTC Artifacts will be uploaded here
⚪ 2025-01-17 09:42:46 UTC ya make is running...
🟢 2025-01-17 09:42:52 UTC Tests successful.

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-01-17 09:42:58 UTC Build successful.

anton-bobkov
anton-bobkov reviewed Jan 17, 2025
View reviewed changes
ydb/docs/ru/core/dev/example-app/go/index.md Outdated
Comment on lines 168 to 170
Если ожидаемое количество данных от запроса велико, не следует пытаться загружать их полностью в оперативную память с помощью вспомогательных методов клиента Query-сервиса, таких как `query.Client.Query` и `query.Client.QueryResultSet`. Эти методы отдают уже материализованный результат (весь результат запроса прочитан с сервера в локальную память). При большом количестве возвращаемых строк материализация результата может привести к известной проблеме [OOM killed](https://en.wikipedia.org/wiki/Out_of_memory).

Для запросов, предполагающих большое количество данных, следует пользоваться методами сессии YDB `query.TxActor.Query` или `query.TxActor.QueryResultSet`, которые отдают "потоковый" результат без материализации. Сессия `query.Session` доступна только из метода `query.Client.Do`, реализующего механизмы выполнения повторных попыток при ошибках. Нужно учитывать, что чтение может быть прервано в любой момент, и в таком случае весь процесс выполнения запроса начнётся заново. Т.е. функция, переданная в `Do` может вызываться больше одного раза.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я бы эти два параграфа обернул в заметку важно:

{% note warning %}

...

{% endnote %}

blinkov
blinkov requested changes Jan 21, 2025
View reviewed changes
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 21, 2025 •
edited
Loading

⚪ 2025-01-21 17:23:42 UTC Pre-commit check linux-x86_64-relwithdebinfo for 5d60620 has started.
⚪ 2025-01-21 17:23:56 UTC Artifacts will be uploaded here
⚫ 2025-01-21 17:24:17 UTC Check cancelled

rekby and others added 7 commits January 21, 2025 20:23
@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 21, 2025 •
edited
Loading

⚪ 2025-01-21 17:26:43 UTC Pre-commit check linux-x86_64-relwithdebinfo for 15481a3 has started.
⚪ 2025-01-21 17:26:48 UTC Artifacts will be uploaded here
⚪ 2025-01-21 17:29:16 UTC ya make is running...
🟢 2025-01-21 17:30:51 UTC Tests successful.

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-01-21 17:31:01 UTC Build successful.

@github-actions GitHub Actions
Copy link

github-actions bot commented Jan 21, 2025 •
edited
Loading

⚪ 2025-01-21 17:26:44 UTC Pre-commit check linux-x86_64-release-asan for 15481a3 has started.
⚪ 2025-01-21 17:26:48 UTC Artifacts will be uploaded here
⚪ 2025-01-21 17:29:18 UTC ya make is running...
🟢 2025-01-21 17:30:50 UTC Tests successful.

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-01-21 17:30:57 UTC Build successful.

@github-actions GitHub Actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 4, 2025

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 4, 2025 •
edited
Loading

⚪ 2025-03-04 11:20:05 UTC Pre-commit check linux-x86_64-release-asan for c9d24ae has started.
⚪ 2025-03-04 11:20:21 UTC Artifacts will be uploaded here
⚪ 2025-03-04 11:22:36 UTC ya make is running...
🟢 2025-03-04 11:22:42 UTC Tests successful.

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-03-04 11:22:49 UTC Build successful.

@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 4, 2025

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 4, 2025 •
edited
Loading

⚪ 2025-03-04 11:25:35 UTC Pre-commit check linux-x86_64-relwithdebinfo for c9d24ae has started.
⚪ 2025-03-04 11:25:50 UTC Artifacts will be uploaded here
⚪ 2025-03-04 11:28:07 UTC ya make is running...
🟢 2025-03-04 11:28:13 UTC Tests successful.

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-03-04 11:28:20 UTC Build successful.

@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 4, 2025 •
edited
Loading

⚪ 2025-03-04 13:15:30 UTC Pre-commit check linux-x86_64-relwithdebinfo for 8554ab2 has started.
⚪ 2025-03-04 13:15:45 UTC Artifacts will be uploaded here
⚪ 2025-03-04 13:18:12 UTC ya make is running...
🟢 2025-03-04 13:18:19 UTC Tests successful.

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-03-04 13:18:25 UTC Build successful.

@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 4, 2025

✅ Documentation build

Revision built successfully
Revision preview link

@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 4, 2025 •
edited
Loading

⚪ 2025-03-04 13:18:43 UTC Pre-commit check linux-x86_64-release-asan for 8554ab2 has started.
⚪ 2025-03-04 13:18:50 UTC Artifacts will be uploaded here
⚪ 2025-03-04 13:21:07 UTC ya make is running...
🟢 2025-03-04 13:21:13 UTC Tests successful.

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-03-04 13:21:19 UTC Build successful.

blinkov
blinkov previously requested changes Mar 7, 2025
View reviewed changes
blinkov
blinkov reviewed Mar 7, 2025
View reviewed changes
@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 7, 2025 •
edited
Loading

⚪ 2025-03-07 09:14:56 UTC Pre-commit check linux-x86_64-release-asan for 72c2082 has started.
⚪ 2025-03-07 09:15:10 UTC Artifacts will be uploaded here
⚪ 2025-03-07 09:17:23 UTC ya make is running...
🟢 2025-03-07 09:17:29 UTC Tests successful.

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-03-07 09:17:35 UTC Build successful.

@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 7, 2025 •
edited
Loading

⚪ 2025-03-07 09:14:57 UTC Pre-commit check linux-x86_64-relwithdebinfo for 72c2082 has started.
⚪ 2025-03-07 09:15:12 UTC Artifacts will be uploaded here
⚪ 2025-03-07 09:17:34 UTC ya make is running...
🟢 2025-03-07 09:17:40 UTC Tests successful.

Test history | Ya make output | Test bloat

TESTS PASSED ERRORS FAILED SKIPPED MUTED?
0 0 0 0 0 0

🟢 2025-03-07 09:17:47 UTC Build successful.

@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 7, 2025

✅ Documentation build

Revision built successfully
Revision preview link

@blinkov blinkov enabled auto-merge (squash) March 11, 2025 10:43
@blinkov blinkov merged commit 9aaa1a9 into ydb-platform:main Mar 11, 2025
13 checks passed
@serbel324 serbel324 mentioned this pull request Mar 13, 2025
Merged
@alexnick88 alexnick88 mentioned this pull request Mar 19, 2025
Closed
adameat pushed a commit to adameat/ydb that referenced this pull request Mar 20, 2025
@rekby @asmyasnikov
@blinkov @anton-bobkov @adameat
basic example query service (ydb-platform#13477)
4a61e9c 
Co-authored-by: Aleksey Myasnikov <[email protected]>
Co-authored-by: Ivan Blinkov <[email protected]>
Co-authored-by: anton-bobkov <[email protected]>
blinkov added a commit that referenced this pull request Mar 21, 2025
@rekby @asmyasnikov
@blinkov @anton-bobkov
basic example query service (#13477)
be87fde 
Co-authored-by: Aleksey Myasnikov <[email protected]>
Co-authored-by: Ivan Blinkov <[email protected]>
Co-authored-by: anton-bobkov <[email protected]>
lberserq pushed a commit to lberserq/ydb that referenced this pull request Mar 28, 2025
@rekby @asmyasnikov
@blinkov @anton-bobkov
basic example query service (ydb-platform#13477)
4a2f4e1 
Co-authored-by: Aleksey Myasnikov <[email protected]>
Co-authored-by: Ivan Blinkov <[email protected]>
Co-authored-by: anton-bobkov <[email protected]>
@asmyasnikov asmyasnikov mentioned this pull request May 6, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@blinkov blinkov blinkov left review comments

@anton-bobkov anton-bobkov anton-bobkov left review comments

@fomichev3000 fomichev3000 fomichev3000 approved these changes

Assignees
No one assigned
Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@rekby @blinkov @fomichev3000 @anton-bobkov @asmyasnikov