morningman opened a new pull request, #4629: URL: https://github.com/apache/doris-website/pull/4629
The maintained 3.x English and Chinese introduction pages still show **Data Intergration** in their shared usage-scenario diagram. The newer dev/4.x illustrations spell it correctly, but do not fix the image used by 3.x. Add a corrected sibling PNG and update the two 3.x image references. Preserve the original JPEG because archived 2.1 pages also reference it. Dev and 4.x already use corrected illustrations and need no change. Fixes #2727 Validation: - The new bitmap was produced with the built-in imagegen tool, requesting only `Data Intergration` → `Data Integration` and preservation of every other label, box, and arrow. It is 1611 × 976 pixels. - Visually compared the original and corrected diagrams: all labels and seven arrow connections/directions are preserved. No functionality or architecture claims were added. - Used macOS Vision OCR to compare all recognized text. The only label difference is the intended spelling correction. Two original arrow shapes were recognized as `>` and `V`; excluding those non-label glyphs, the remaining OCR text matches exactly after the correction. OCR is an additional text check, not proof of pixel identity or arrow geometry; the layout and arrows were reviewed visually. - Verified the original shared JPEG has the same Git object hash as master, and both Markdown files only change the image reference (plus removal of existing trailing whitespace). - `git diff --check --cached` and changed-file locale synchronization pass. - The changed-file link checker reports four existing inbound anchor errors from the English/Chinese 3.x cluster-planning pages. This change adds no missing links and changes no headings or anchors. - No full website build was run for this asset/reference change. Review: - Goal: correct the reported spelling in the illustration actually used by 3.x. - Scope: one sibling image and two localized references. - Information architecture: English/Chinese 3.x updated together; dev/4.x checked; archived content and its shared original asset preserved. - Links/navigation: the new asset exists; page paths, metadata, headings, sidebars, and redirects are unchanged. - Frontend/build conventions: no React, styling, configuration, or build changes; the replacement retains the diagram's aspect ratio and visual style. - Validation: visual and OCR comparisons, original-asset hash, reference scope, locale synchronization, and whitespace checks completed; pre-existing link findings documented above. - Other concerns: generated output has a lower source resolution than the original, but exceeds the documentation content width and remains legible at its display size. -- 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]
