Skip to content

lake: incremental updates 0531#22968

Merged
ti-chi-bot[bot] merged 8 commits into
pingcap:feature/preview-cloud-lakefrom
lilin90:update-0531
Jun 8, 2026
Merged

lake: incremental updates 0531#22968
ti-chi-bot[bot] merged 8 commits into
pingcap:feature/preview-cloud-lakefrom
lilin90:update-0531

Conversation

@lilin90

@lilin90 lilin90 commented May 29, 2026

Copy link
Copy Markdown
Member

What is changed, added or deleted? (Required)

Incremental updates till 05/31/2026

Which TiDB version(s) do your changes apply to? (Required)

Tips for choosing the affected version(s):

By default, CHOOSE MASTER ONLY so your changes will be applied to the next TiDB major or minor releases. If your PR involves a product feature behavior change or a compatibility change, CHOOSE THE AFFECTED RELEASE BRANCH(ES) AND MASTER.

For details, see tips for choosing the affected versions.

  • master (the latest development version)
  • v9.0 (TiDB 9.0 versions)
  • v8.5 (TiDB 8.5 versions)
  • v8.1 (TiDB 8.1 versions)
  • v7.5 (TiDB 7.5 versions)
  • v7.1 (TiDB 7.1 versions)
  • v6.5 (TiDB 6.5 versions)
  • v6.1 (TiDB 6.1 versions)

What is the related PR or file link(s)?

  • This PR is translated from:
  • Other reference link(s):

Do your changes match any of the following descriptions?

  • Delete files
  • Change aliases
  • Need modification after applied to another branch
  • Might cause conflicts after applied to another branch

@lilin90 lilin90 self-assigned this May 29, 2026
@lilin90 lilin90 added translation/no-need No need to translate this PR. area/tidb-cloud This PR relates to the area of TiDB Cloud. labels May 29, 2026
@ti-chi-bot ti-chi-bot Bot added the size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. label May 29, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request introduces comprehensive documentation for Amazon SQS (S3) integration, including new guides for the SQS (S3) IAM Role data source and integration tasks, as well as a detailed guide on Data Protection Policies (masking and row access policies). It also updates several SQL reference files and guides to clarify regular expression pattern matching for staged files. The review feedback highlights opportunities to better align with the style guide by rewriting passive voice sentences into the active voice, addressing the user in the second person ('you'), applying sentence case to headings, and consistently formatting function names, keywords, and paths with backticks.

Comment thread tidb-cloud-lake/guides/amazon-sqs-s3-iam-role.md
Comment thread tidb-cloud-lake/guides/amazon-sqs-s3-iam-role.md
Comment thread tidb-cloud-lake/guides/amazon-sqs-s3-iam-role.md
Comment thread tidb-cloud-lake/guides/amazon-sqs-s3-iam-role.md
Comment thread tidb-cloud-lake/guides/data-protection-policies.md
Comment thread tidb-cloud-lake/sql/system-history-tables.md Outdated
Comment thread tidb-cloud-lake/guides/task-management.md
Comment thread tidb-cloud-lake/guides/integration-tasks.md
Comment thread tidb-cloud-lake/sql/create-masking-policy.md
Comment thread tidb-cloud-lake/guides/data-sources.md
lilin90 and others added 6 commits May 29, 2026 15:04
Replace instances of 'AWS - Credentials' with 'Amazon S3 - Credentials' in tidb-cloud-lake/guides/integrate-with-amazon-s3.md to clarify the credential type; updates link text and data source field descriptions.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Update the Amazon SQS + S3 IAM role guide to use 'platform' naming instead of 'databend'. Changes include env var renames (ROLE_NAME, PLATFORM_SETUP_ROLE_ARN, PLATFORM_LOAD_ROLE_ARN, EXTERNAL_ID), updated guidance text, trust-policy jq variable names and Statement Sids, IAM policy name (platform-s3-sqs-access), and example test filenames/object keys. This clarifies that the roles and ARNs are provided by Platform and keeps naming consistent across the doc.
Add missing blank lines to improve readability in two documentation pages: tidb-cloud-lake/guides/row-access-policy.md (under 'Test with multiple roles') and tidb-cloud-lake/sql/system-history-tables.md (under 'Required Permissions'). No substantive content changes; only whitespace/formatting adjustments.
@lilin90 lilin90 added the lgtm label Jun 8, 2026
Update the link in tidb-cloud-lake/guides/integrate-with-amazon-sqs-s3.md from a relative path ('./integrate-with-amazon-s3.md') to an absolute path ('/tidb-cloud-lake/guides/integrate-with-amazon-s3.md') to ensure correct navigation and consistent linking across the site.
@lilin90

lilin90 commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

/approve

@ti-chi-bot

ti-chi-bot Bot commented Jun 8, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: lilin90

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ti-chi-bot ti-chi-bot Bot added the approved label Jun 8, 2026
@ti-chi-bot ti-chi-bot Bot merged commit b1bdc6b into pingcap:feature/preview-cloud-lake Jun 8, 2026
10 of 11 checks passed
@lilin90 lilin90 deleted the update-0531 branch June 8, 2026 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved area/tidb-cloud This PR relates to the area of TiDB Cloud. lgtm size/XXL Denotes a PR that changes 1000+ lines, ignoring generated files. translation/no-need No need to translate this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant