Skip to content

Conversation

@evanevanevanevannnn
Copy link
Collaborator

Changelog entry

...

Changelog category

  • Documentation (changelog entry is not required)

Description for reviewers

...

Copilot AI review requested due to automatic review settings November 10, 2025 23:55
@evanevanevanevannnn evanevanevanevannnn requested a review from a team as a code owner November 10, 2025 23:55
@github-actions github-actions bot added the documentation Improvements or additions to documentation label Nov 10, 2025
@github-actions
Copy link

github-actions bot commented Nov 10, 2025

2025-11-10 23:56:18 UTC Pre-commit check linux-x86_64-relwithdebinfo for 1aed018 has started.
2025-11-10 23:57:24 UTC Artifacts will be uploaded here
2025-11-10 23:59:27 UTC ya make is running...
🟢 2025-11-10 23:59:34 UTC Tests successful.

Ya make output | Test bloat

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

🟢 2025-11-10 23:59:39 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Nov 10, 2025

2025-11-10 23:57:40 UTC Pre-commit check linux-x86_64-release-asan for 1aed018 has started.
2025-11-10 23:58:02 UTC Artifacts will be uploaded here
2025-11-11 00:00:13 UTC ya make is running...
🟢 2025-11-11 00:00:21 UTC Tests successful.

Ya make output | Test bloat

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

🟢 2025-11-11 00:00:27 UTC Build successful.

Copilot finished reviewing on behalf of evanevanevanevannnn November 10, 2025 23:57
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR reorganizes the types mapping documentation by moving it from the general import_and_export.md file into a dedicated document within the S3-specific folder. Since types mapping is primarily relevant to S3 and Parquet format operations, this change improves documentation organization and discoverability.

Key changes:

  • Moved types mapping documentation from import_and_export.md to a dedicated s3/types_mapping.md file
  • Added the new types mapping document to the S3 section's table of contents

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
ydb/docs/ru/core/concepts/federated_query/s3/types_mapping.md Added header and introduction text for the standalone types mapping document
ydb/docs/ru/core/concepts/federated_query/s3/toc_i.yaml Added entry for the new types mapping document in the S3 documentation navigation
ydb/docs/ru/core/concepts/federated_query/import_and_export.md Removed types mapping section and include statement since it's now in a dedicated S3 document
Comments suppressed due to low confidence (1)

ydb/docs/ru/core/concepts/federated_query/s3/types_mapping.md:1

  • The page title mentions only export ('экспорте данных') but the content describes both import and export operations. The title should be updated to match the table of contents entry: 'Отображение типов при импорте и экспорте данных в формате Parquet' to accurately reflect the content.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link

🟢 2025-11-10 23:58:28 UTC The validation of the Pull Request description is successful.

@github-actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link

Build logs

Warnings (17)

⚠️ Title not found: bridge_config.md in ru/devops/configuration-management/configuration-v1/static-config.md

⚠️ Title not found: bridge_config.md in ru/devops/configuration-management/configuration-v2/config-settings.md

⚠️ Title not found: bridge_config.md in ru/reference/configuration/index.md

⚠️ en/concepts/analytics/concepts/etl.md: 49: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 3]

⚠️ en/concepts/analytics/concepts/ml.md: 16: MD030 / list-marker-space Spaces after list markers [Expected: 1; Actual: 3]

⚠️ en/concepts/analytics/concepts/ml.md: 17: MD030 / list-marker-space Spaces after list markers [Expected: 1; Actual: 3]

⚠️ en/concepts/analytics/concepts/store.md: 10: MD030 / list-marker-space Spaces after list markers [Expected: 1; Actual: 2]

⚠️ en/concepts/analytics/concepts/store.md: 9: MD030 / list-marker-space Spaces after list markers [Expected: 1; Actual: 2]

⚠️ ru/dev/timeouts.md: 28: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/dev/timeouts.md: 30: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/public-materials/videos/2025.md: 12: MD022 / blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Above] [Context: "## CI/CD как драйвер разработки"]

⚠️ ru/public-materials/videos/2025.md: 16: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/public-materials/videos/2025.md: 18: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/public-materials/videos/2025.md: 34: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/public-materials/videos/2025.md: 48: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

Log was truncated. (2 records)

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@github-actions
Copy link

github-actions bot commented Nov 11, 2025

2025-11-11 09:36:10 UTC Pre-commit check linux-x86_64-relwithdebinfo for 19900a7 has started.
2025-11-11 09:36:28 UTC Artifacts will be uploaded here
2025-11-11 09:38:33 UTC ya make is running...
🟢 2025-11-11 09:38:38 UTC Tests successful.

Ya make output | Test bloat

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

🟢 2025-11-11 09:38:44 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Nov 11, 2025

2025-11-11 09:37:14 UTC Pre-commit check linux-x86_64-release-asan for 19900a7 has started.
2025-11-11 09:37:32 UTC Artifacts will be uploaded here
2025-11-11 09:39:36 UTC ya make is running...
🟢 2025-11-11 09:39:42 UTC Tests successful.

Ya make output | Test bloat

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

🟢 2025-11-11 09:39:47 UTC Build successful.

@github-actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link

Build logs

Warnings (17)

⚠️ Title not found: bridge_config.md in ru/devops/configuration-management/configuration-v1/static-config.md

⚠️ Title not found: bridge_config.md in ru/devops/configuration-management/configuration-v2/config-settings.md

⚠️ Title not found: bridge_config.md in ru/reference/configuration/index.md

⚠️ en/concepts/analytics/concepts/etl.md: 49: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 3]

⚠️ en/concepts/analytics/concepts/ml.md: 16: MD030 / list-marker-space Spaces after list markers [Expected: 1; Actual: 3]

⚠️ en/concepts/analytics/concepts/ml.md: 17: MD030 / list-marker-space Spaces after list markers [Expected: 1; Actual: 3]

⚠️ en/concepts/analytics/concepts/store.md: 10: MD030 / list-marker-space Spaces after list markers [Expected: 1; Actual: 2]

⚠️ en/concepts/analytics/concepts/store.md: 9: MD030 / list-marker-space Spaces after list markers [Expected: 1; Actual: 2]

⚠️ ru/dev/timeouts.md: 28: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/dev/timeouts.md: 30: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/public-materials/videos/2025.md: 12: MD022 / blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Above] [Context: "## CI/CD как драйвер разработки"]

⚠️ ru/public-materials/videos/2025.md: 16: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/public-materials/videos/2025.md: 18: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/public-materials/videos/2025.md: 34: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/public-materials/videos/2025.md: 48: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

Log was truncated. (2 records)

@github-actions
Copy link

❌ Documentation build

Revision build failed

@github-actions
Copy link

github-actions bot commented Nov 11, 2025

2025-11-11 14:47:08 UTC Pre-commit check linux-x86_64-relwithdebinfo for 5cc92c4 has started.
2025-11-11 14:47:28 UTC Artifacts will be uploaded here
2025-11-11 14:49:02 UTC Check cancelled

@github-actions
Copy link

github-actions bot commented Nov 11, 2025

2025-11-11 14:50:33 UTC Pre-commit check linux-x86_64-release-asan for b014c14 has started.
2025-11-11 14:50:37 UTC Artifacts will be uploaded here
2025-11-11 14:52:52 UTC ya make is running...
🟢 2025-11-11 14:52:59 UTC Tests successful.

Ya make output | Test bloat

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

🟢 2025-11-11 14:53:04 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Nov 11, 2025

2025-11-11 14:51:18 UTC Pre-commit check linux-x86_64-relwithdebinfo for b014c14 has started.
2025-11-11 14:51:34 UTC Artifacts will be uploaded here
2025-11-11 14:53:52 UTC ya make is running...
🟢 2025-11-11 14:53:59 UTC Tests successful.

Ya make output | Test bloat

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

🟢 2025-11-11 14:54:04 UTC Build successful.

Copy link
Collaborator

Choose a reason for hiding this comment

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

давай переименуем файл. слишком абстрактное название. можно слово arrow дописать

@github-actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link

Build logs

Warnings (17)

⚠️ Title not found: bridge_config.md in ru/devops/configuration-management/configuration-v1/static-config.md

⚠️ Title not found: bridge_config.md in ru/devops/configuration-management/configuration-v2/config-settings.md

⚠️ Title not found: bridge_config.md in ru/reference/configuration/index.md

⚠️ en/concepts/analytics/concepts/etl.md: 49: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 3]

⚠️ en/concepts/analytics/concepts/ml.md: 16: MD030 / list-marker-space Spaces after list markers [Expected: 1; Actual: 3]

⚠️ en/concepts/analytics/concepts/ml.md: 17: MD030 / list-marker-space Spaces after list markers [Expected: 1; Actual: 3]

⚠️ en/concepts/analytics/concepts/store.md: 10: MD030 / list-marker-space Spaces after list markers [Expected: 1; Actual: 2]

⚠️ en/concepts/analytics/concepts/store.md: 9: MD030 / list-marker-space Spaces after list markers [Expected: 1; Actual: 2]

⚠️ ru/dev/timeouts.md: 28: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/dev/timeouts.md: 30: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/public-materials/videos/2025.md: 12: MD022 / blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Above] [Context: "## CI/CD как драйвер разработки"]

⚠️ ru/public-materials/videos/2025.md: 16: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/public-materials/videos/2025.md: 18: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/public-materials/videos/2025.md: 34: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/public-materials/videos/2025.md: 48: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

Log was truncated. (2 records)

@github-actions
Copy link

github-actions bot commented Nov 11, 2025

2025-11-11 15:14:16 UTC Pre-commit check linux-x86_64-relwithdebinfo for 9f9067e has started.
2025-11-11 15:14:34 UTC Artifacts will be uploaded here
2025-11-11 15:16:46 UTC ya make is running...
🟢 2025-11-11 15:16:53 UTC Tests successful.

Ya make output | Test bloat

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

🟢 2025-11-11 15:16:58 UTC Build successful.

@github-actions
Copy link

github-actions bot commented Nov 11, 2025

2025-11-11 15:16:08 UTC Pre-commit check linux-x86_64-release-asan for 9f9067e has started.
2025-11-11 15:16:24 UTC Artifacts will be uploaded here
2025-11-11 15:18:32 UTC ya make is running...
🟢 2025-11-11 15:18:39 UTC Tests successful.

Ya make output | Test bloat

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

🟢 2025-11-11 15:18:45 UTC Build successful.

@github-actions
Copy link

✅ Documentation build

Revision built successfully
Revision preview link

Build logs

Warnings (17)

⚠️ Title not found: bridge_config.md in ru/devops/configuration-management/configuration-v1/static-config.md

⚠️ Title not found: bridge_config.md in ru/devops/configuration-management/configuration-v2/config-settings.md

⚠️ Title not found: bridge_config.md in ru/reference/configuration/index.md

⚠️ en/concepts/analytics/concepts/etl.md: 49: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 3]

⚠️ en/concepts/analytics/concepts/ml.md: 16: MD030 / list-marker-space Spaces after list markers [Expected: 1; Actual: 3]

⚠️ en/concepts/analytics/concepts/ml.md: 17: MD030 / list-marker-space Spaces after list markers [Expected: 1; Actual: 3]

⚠️ en/concepts/analytics/concepts/store.md: 10: MD030 / list-marker-space Spaces after list markers [Expected: 1; Actual: 2]

⚠️ en/concepts/analytics/concepts/store.md: 9: MD030 / list-marker-space Spaces after list markers [Expected: 1; Actual: 2]

⚠️ ru/dev/timeouts.md: 28: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/dev/timeouts.md: 30: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/public-materials/videos/2025.md: 12: MD022 / blanks-around-headings Headings should be surrounded by blank lines [Expected: 1; Actual: 0; Above] [Context: "## CI/CD как драйвер разработки"]

⚠️ ru/public-materials/videos/2025.md: 16: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/public-materials/videos/2025.md: 18: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/public-materials/videos/2025.md: 34: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

⚠️ ru/public-materials/videos/2025.md: 48: MD009 / no-trailing-spaces Trailing spaces [Expected: 0 or 2; Actual: 1]

Log was truncated. (2 records)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants