Abhishek-Mohan opened a new pull request, #7163:
URL: https://github.com/apache/datafusion-comet/pull/7163
## Which issue does this PR close?
Part of #4419.
## Rationale for this change
#4419 lists `gluten_comparison.md` among the pages to update. It used
"native" for both projects, sometimes meaning Rust or C++, sometimes Comet as a
whole, and sometimes Velox, and it called both engines "vectorized". The rule
IDs refer to the style guide in #7151.
## What changes are included in this PR?
- **"native" now names what each sentence means (N1):**
- for Comet: "the Comet pipeline" or "Rust";
- for Gluten: "Velox", so a sentence about Gluten doesn't borrow Comet's
vocabulary.
- **Velox and DataFusion are described as columnar query engines, not
vectorized ones (T10).**
- **Factual fix: Delta Lake.** The page said Comet has no Delta Lake
integration, but `main` has an experimental, opt-in Delta Lake scan (#5365).
The page now says so and links the Delta Lake guide. If you'd prefer this PR to
change wording only, I can move this to a separate PR.
- **Link fix:** the compatibility guide is now linked by its path in the
docs rather than its published URL (P4).
## How are these changes tested?
Docs only.
- `prettier --check` passes.
- I built the docs locally with Sphinx and checked the new Delta Lake link.
AI assistance: OpenCode helped draft this change.
--
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]
