Copilot commented on code in PR #3362: URL: https://github.com/apache/doris-website/pull/3362#discussion_r2789387495
########## i18n/zh-CN/docusaurus-plugin-content-docs/version-4.x/data-operate/import/data-source/azure-storage.md: ########## @@ -46,10 +46,10 @@ DISTRIBUTED BY HASH(user_id) BUCKETS 10; ### 第 3 步：使用 S3 Load 导入数据 :::caution Caution Review Comment: This is a zh-CN page but the admonition title is still in English (`:::caution Caution`). For consistency with other zh-CN docs (e.g., the MinIO page uses `:::caution 注意`), consider localizing the title to `注意` here as well. ########## i18n/zh-CN/docusaurus-plugin-content-docs/version-4.x/data-operate/import/data-source/azure-storage.md: ########## @@ -46,10 +46,10 @@ DISTRIBUTED BY HASH(user_id) BUCKETS 10; ### 第 3 步：使用 S3 Load 导入数据 :::caution Caution -Azure Storage 默认要求 HTTPS 传输，对应的存储账户配置是 `需要安全传输：已启用`。 -必须在 Doris `be.conf` 中设置 `s3_client_http_scheme = https` 才能正常访问 Azure Storage。 +使用 S3 Load 导入 Azure Storage 数据时，请注意： -Azure S3 properties 中的 `s3.region` 可以省略。 +- Azure Storage 默认要求 HTTPS 传输（对应存储账户配置为 `需要安全传输：已启用`），必须在 Doris `be.conf` 中设置 `s3_client_http_scheme = https` 才能正常访问。 +- Azure S3 properties 中的 `s3.region` 可以省略。 Review Comment: 这里提示 `s3.region` 可以省略，但紧随其后的示例（以及 TVF 章节示例）仍包含 `"s3.region" = ...`，可能让读者困惑。建议要么从示例中移除 `s3.region`，要么明确说明该参数可选但示例中保留用于兼容/说明。 ```suggestion - Azure S3 properties 中的 `s3.region` 可以省略，以下示例中保留该参数仅用于兼容性和示例说明。 ``` ########## i18n/zh-CN/docusaurus-plugin-content-docs/version-2.1/data-operate/import/data-source/azure-storage.md: ########## @@ -46,10 +46,10 @@ DISTRIBUTED BY HASH(user_id) BUCKETS 10; ### 第 3 步：使用 S3 Load 导入数据 :::caution Caution Review Comment: This is a zh-CN page but the admonition title is still in English (`:::caution Caution`). For consistency with other zh-CN docs (e.g., the MinIO page uses `:::caution 注意`), consider localizing the title to `注意` here as well. ########## i18n/zh-CN/docusaurus-plugin-content-docs/current/data-operate/import/data-source/azure-storage.md: ########## @@ -46,10 +46,10 @@ DISTRIBUTED BY HASH(user_id) BUCKETS 10; ### 第 3 步：使用 S3 Load 导入数据 :::caution Caution -Azure Storage 默认要求 HTTPS 传输，对应的存储账户配置是 `需要安全传输：已启用`。 -必须在 Doris `be.conf` 中设置 `s3_client_http_scheme = https` 才能正常访问 Azure Storage。 +使用 S3 Load 导入 Azure Storage 数据时，请注意： -Azure S3 properties 中的 `s3.region` 可以省略。 +- Azure Storage 默认要求 HTTPS 传输（对应存储账户配置为 `需要安全传输：已启用`），必须在 Doris `be.conf` 中设置 `s3_client_http_scheme = https` 才能正常访问。 +- Azure S3 properties 中的 `s3.region` 可以省略。 Review Comment: 这里提示 `s3.region` 可以省略，但紧随其后的示例（以及 TVF 章节示例）仍包含 `"s3.region" = ...`，可能让读者困惑。建议要么从示例中移除 `s3.region`，要么明确说明该参数可选但示例中保留用于兼容/说明。 ```suggestion - Azure S3 properties 中的 `s3.region` 为可选参数，下方示例中保留该参数仅用于演示（或兼容性）说明，实际使用时可根据需要省略。 ``` ########## versioned_docs/version-2.1/data-operate/import/data-source/azure-storage.md: ########## @@ -46,10 +46,10 @@ DISTRIBUTED BY HASH(user_id) BUCKETS 10; ### Step 3: Load data using S3 Load :::caution Caution -Azure Storage requires HTTPS transmission by default, with the corresponding storage account configuration being `Secure transfer required: Enabled`. -To access Azure Storage properly, you need to set `s3_client_http_scheme = https` in Doris's `be.conf`. +When importing data from Azure Storage with S3 Load, note the following: -The `s3.region` setting for Azure S3 properties can be omitted. +- Azure Storage requires HTTPS transmission by default (`Secure transfer required: Enabled`). To access Azure Storage properly, set `s3_client_http_scheme = https` in Doris `be.conf`. +- The `s3.region` setting in Azure S3 properties can be omitted. Review Comment: The caution note says `s3.region` can be omitted, but the example configs immediately below (and in the TVF section) still include `"s3.region" = ...` without explanation. Consider either removing `s3.region` from the examples or explicitly noting that it’s optional but kept in the sample for compatibility. ```suggestion - The `s3.region` setting in Azure S3 properties can be omitted; it is included in the examples below only for compatibility with existing configurations. ``` ########## i18n/zh-CN/docusaurus-plugin-content-docs/version-4.x/data-operate/import/data-source/azure-storage.md: ########## @@ -136,10 +136,10 @@ DISTRIBUTED BY HASH(user_id) BUCKETS 10; ### 第 3 步：使用 TVF 导入数据 :::caution Caution Review Comment: This is a zh-CN page but the admonition title is still in English (`:::caution Caution`). For consistency with other zh-CN docs (e.g., the MinIO page uses `:::caution 注意`), consider localizing the title to `注意` here as well. ########## i18n/zh-CN/docusaurus-plugin-content-docs/version-3.x/data-operate/import/data-source/azure-storage.md: ########## @@ -46,10 +46,10 @@ DISTRIBUTED BY HASH(user_id) BUCKETS 10; ### 第 3 步：使用 S3 Load 导入数据 :::caution Caution Review Comment: This is a zh-CN page but the admonition title is still in English (`:::caution Caution`). For consistency with other zh-CN docs (e.g., the MinIO page uses `:::caution 注意`), consider localizing the title to `注意` here as well. ########## i18n/zh-CN/docusaurus-plugin-content-docs/version-3.x/data-operate/import/data-source/azure-storage.md: ########## @@ -46,10 +46,10 @@ DISTRIBUTED BY HASH(user_id) BUCKETS 10; ### 第 3 步：使用 S3 Load 导入数据 :::caution Caution -Azure Storage 默认要求 HTTPS 传输，对应的存储账户配置是 `需要安全传输：已启用`。 -必须在 Doris `be.conf` 中设置 `s3_client_http_scheme = https` 才能正常访问 Azure Storage。 +使用 S3 Load 导入 Azure Storage 数据时，请注意： -Azure S3 properties 中的 `s3.region` 可以省略。 +- Azure Storage 默认要求 HTTPS 传输（对应存储账户配置为 `需要安全传输：已启用`），必须在 Doris `be.conf` 中设置 `s3_client_http_scheme = https` 才能正常访问。 +- Azure S3 properties 中的 `s3.region` 可以省略。 Review Comment: 这里提示 `s3.region` 可以省略，但紧随其后的示例（以及 TVF 章节示例）仍包含 `"s3.region" = ...`，可能让读者困惑。建议要么从示例中移除 `s3.region`，要么明确说明该参数可选但示例中保留用于兼容/说明。 ```suggestion - Azure S3 properties 中的 `s3.region` 为可选参数；本文示例中保留该参数是出于兼容性和说明目的，实际使用时可根据需要省略。 ``` ########## docs/data-operate/import/data-source/azure-storage.md: ########## @@ -46,10 +46,10 @@ DISTRIBUTED BY HASH(user_id) BUCKETS 10; ### Step 3: Load data using S3 Load :::caution Caution -Azure Storage requires HTTPS transmission by default, with the corresponding storage account configuration being `Secure transfer required: Enabled`. -To access Azure Storage properly, you need to set `s3_client_http_scheme = https` in Doris's `be.conf`. +When importing data from Azure Storage with S3 Load, note the following: -The `s3.region` setting for Azure S3 properties can be omitted. +- Azure Storage requires HTTPS transmission by default (`Secure transfer required: Enabled`). To access Azure Storage properly, set `s3_client_http_scheme = https` in Doris `be.conf`. +- The `s3.region` setting in Azure S3 properties can be omitted. Review Comment: The caution note says `s3.region` can be omitted, but the example configs immediately below (and in the TVF section) still include `"s3.region" = ...` without explanation. Consider either removing `s3.region` from the examples or explicitly noting that it’s optional but kept in the sample for compatibility. ```suggestion - The `s3.region` setting in Azure S3 properties can be omitted. It is included in the examples below only for compatibility with existing configurations. ``` ########## i18n/zh-CN/docusaurus-plugin-content-docs/version-3.x/data-operate/import/data-source/azure-storage.md: ########## @@ -136,10 +136,10 @@ DISTRIBUTED BY HASH(user_id) BUCKETS 10; ### 第 3 步：使用 TVF 导入数据 :::caution Caution Review Comment: This is a zh-CN page but the admonition title is still in English (`:::caution Caution`). For consistency with other zh-CN docs (e.g., the MinIO page uses `:::caution 注意`), consider localizing the title to `注意` here as well. ########## i18n/zh-CN/docusaurus-plugin-content-docs/version-2.1/data-operate/import/data-source/azure-storage.md: ########## @@ -136,10 +136,10 @@ DISTRIBUTED BY HASH(user_id) BUCKETS 10; ### 第 3 步：使用 TVF 导入数据 :::caution Caution Review Comment: This is a zh-CN page but the admonition title is still in English (`:::caution Caution`). For consistency with other zh-CN docs (e.g., the MinIO page uses `:::caution 注意`), consider localizing the title to `注意` here as well. ########## i18n/zh-CN/docusaurus-plugin-content-docs/current/data-operate/import/data-source/azure-storage.md: ########## @@ -136,10 +136,10 @@ DISTRIBUTED BY HASH(user_id) BUCKETS 10; ### 第 3 步：使用 TVF 导入数据 :::caution Caution Review Comment: This is a zh-CN page but the admonition title is still in English (`:::caution Caution`). For consistency with other zh-CN docs (e.g., the MinIO page uses `:::caution 注意`), consider localizing the title to `注意` here as well. ########## i18n/zh-CN/docusaurus-plugin-content-docs/version-2.1/data-operate/import/data-source/azure-storage.md: ########## @@ -46,10 +46,10 @@ DISTRIBUTED BY HASH(user_id) BUCKETS 10; ### 第 3 步：使用 S3 Load 导入数据 :::caution Caution -Azure Storage 默认要求 HTTPS 传输，对应的存储账户配置是 `需要安全传输：已启用`。 -必须在 Doris `be.conf` 中设置 `s3_client_http_scheme = https` 才能正常访问 Azure Storage。 +使用 S3 Load 导入 Azure Storage 数据时，请注意： -Azure S3 properties 中的 `s3.region` 可以省略。 +- Azure Storage 默认要求 HTTPS 传输（对应存储账户配置为 `需要安全传输：已启用`），必须在 Doris `be.conf` 中设置 `s3_client_http_scheme = https` 才能正常访问。 +- Azure S3 properties 中的 `s3.region` 可以省略。 Review Comment: 这里提示 `s3.region` 可以省略，但紧随其后的示例（以及 TVF 章节示例）仍包含 `"s3.region" = ...`，可能让读者困惑。建议要么从示例中移除 `s3.region`，要么明确说明该参数可选但示例中保留用于兼容/说明。 ```suggestion - Azure S3 properties 中的 `s3.region` 可以省略；下方示例中保留该参数仅用于说明和兼容性。 ``` ########## versioned_docs/version-4.x/data-operate/import/data-source/azure-storage.md: ########## @@ -46,10 +46,10 @@ DISTRIBUTED BY HASH(user_id) BUCKETS 10; ### Step 3: Load data using S3 Load :::caution Caution -Azure Storage requires HTTPS transmission by default, with the corresponding storage account configuration being `Secure transfer required: Enabled`. -To access Azure Storage properly, you need to set `s3_client_http_scheme = https` in Doris's `be.conf`. +When importing data from Azure Storage with S3 Load, note the following: -The `s3.region` setting for Azure S3 properties can be omitted. +- Azure Storage requires HTTPS transmission by default (`Secure transfer required: Enabled`). To access Azure Storage properly, set `s3_client_http_scheme = https` in Doris `be.conf`. +- The `s3.region` setting in Azure S3 properties can be omitted. Review Comment: The caution note says `s3.region` can be omitted, but the example configs immediately below (and in the TVF section) still include `"s3.region" = ...` without explanation. Consider either removing `s3.region` from the examples or explicitly noting that it’s optional but kept in the sample for compatibility. ```suggestion - The `s3.region` setting in Azure S3 properties can be omitted; it is included in the examples below only for backward compatibility with older Doris versions that required it. ``` ########## versioned_docs/version-3.x/data-operate/import/data-source/azure-storage.md: ########## @@ -46,10 +46,10 @@ DISTRIBUTED BY HASH(user_id) BUCKETS 10; ### Step 3: Load data using S3 Load :::caution Caution -Azure Storage requires HTTPS transmission by default, with the corresponding storage account configuration being `Secure transfer required: Enabled`. -To access Azure Storage properly, you need to set `s3_client_http_scheme = https` in Doris's `be.conf`. +When importing data from Azure Storage with S3 Load, note the following: -The `s3.region` setting for Azure S3 properties can be omitted. +- Azure Storage requires HTTPS transmission by default (`Secure transfer required: Enabled`). To access Azure Storage properly, set `s3_client_http_scheme = https` in Doris `be.conf`. +- The `s3.region` setting in Azure S3 properties can be omitted. Review Comment: The caution note says `s3.region` can be omitted, but the example configs immediately below (and in the TVF section) still include `"s3.region" = ...` without explanation. Consider either removing `s3.region` from the examples or explicitly noting that it’s optional but kept in the sample for compatibility. ```suggestion - The `s3.region` setting in Azure S3 properties can be omitted; it is included in the examples below only for compatibility with other S3-compatible storage and older Doris versions. ``` ########## i18n/zh-CN/docusaurus-plugin-content-docs/current/data-operate/import/data-source/azure-storage.md: ########## @@ -46,10 +46,10 @@ DISTRIBUTED BY HASH(user_id) BUCKETS 10; ### 第 3 步：使用 S3 Load 导入数据 :::caution Caution Review Comment: This is a zh-CN page but the admonition title is still in English (`:::caution Caution`). For consistency with other zh-CN docs (e.g., the MinIO page uses `:::caution 注意`), consider localizing the title to `注意` here as well. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
