rich7420 opened a new pull request, #6352: URL: https://github.com/apache/datafusion-comet/pull/6352
## Which issue does this PR close? Closes #5832. ## Rationale for this change The operator guide omits plan identity and exchange reuse. Its incomplete examples can lead contributors to include serialization state in equality or omit semantic parameters, allowing incorrect exchange reuse and wrong query results. ## What changes are included in this PR? Complete the Filter and Project examples with their current `stringArgs`, `equals` and `hashCode` overrides. Explain semantic parameters, serialization state, canonicalization, and the alternative `originalPlan` convention used by scan and broadcast operators. Add guidance for exchange-reuse regressions that verify native execution, distinguish different plans and retain reuse for equivalent plans, including optimizer rules that can otherwise hide an omission. ## How are these changes tested? Reviewed the examples against the current operators and canonicalization code, and checked the linked regression examples. Prettier and whitespace checks pass. [Fork CI](https://github.com/rich7420/datafusion-comet/actions/runs/36451214692) passed for head `f5bf0f9fd`. This changes only the contributor guide; runtime jobs were skipped by the documentation path filters, and no JVM/native runtime suites were run locally for this documentation 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]
