Skip to content

docs: use arrayElement for random event_type in generating test data guide#5793

Merged
Blargian merged 1 commit intomainfrom
codex/linear-mention-doc-585-arrayjoin-usage-in-doc-generating
Mar 20, 2026
Merged

docs: use arrayElement for random event_type in generating test data guide#5793
Blargian merged 1 commit intomainfrom
codex/linear-mention-doc-585-arrayjoin-usage-in-doc-generating

Conversation

@Blargian
Copy link
Copy Markdown
Member

Motivation

  • Clarify the Simple uniform dataset example because arrayJoin expands rows rather than selecting a single random element, so the example and description were misleading.

Description

  • Replace arrayJoin(['click','view','purchase']) with arrayElement(['click','view','purchase'], toUInt32(rand()) % 3 + 1) and update the explanatory bullet to reference arrayElement(...) in docs/guides/starter_guides/generating-test-data.md.

Testing

  • Ran git diff --check -- docs/guides/starter_guides/generating-test-data.md, which reported no issues.
  • Attempted to fetch the published guide URL to confirm the change, but the HTTP request failed due to a proxy error in the execution environment.

Codex Task

@Blargian Blargian requested a review from a team as a code owner March 20, 2026 10:56
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 20, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
clickhouse-docs Ready Ready Preview, Comment Mar 20, 2026 11:07am
clickhouse-docs-jp Building Building Preview, Comment Mar 20, 2026 11:07am
3 Skipped Deployments
Project Deployment Actions Updated (UTC)
clickhouse-docs-ko Ignored Ignored Preview Mar 20, 2026 11:07am
clickhouse-docs-ru Ignored Ignored Preview Mar 20, 2026 11:07am
clickhouse-docs-zh Ignored Ignored Preview Mar 20, 2026 11:07am

Request Review

@Blargian Blargian merged commit 1346be9 into main Mar 20, 2026
14 of 15 checks passed
@Blargian Blargian mentioned this pull request Mar 20, 2026
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant