Skip to content

Conversation

@yew1eb
Copy link
Contributor

@yew1eb yew1eb commented Dec 22, 2025

Which issue does this PR close?

Closes #1778

Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

How was this patch tested?

@github-actions github-actions bot added the spark label Dec 22, 2025
@yew1eb yew1eb force-pushed the add_singleChildFallback branch from 418e057 to e700f7c Compare December 22, 2025 13:35
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a configuration flag spark.auron.expression.singleChildFallback.enabled to allow users to disable the single-child fallback optimization in expression conversion. This addresses issue #1778, which requested the ability to disable this optimization when it introduces overhead or complicates debugging.

Changes:

  • Added singleChildFallbackEnabled configuration method that defaults to true for backward compatibility
  • Modified convertExpr to check the config and skip single-child optimization when disabled

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@yew1eb
Copy link
Contributor Author

yew1eb commented Jan 27, 2026

@cxzl25 Could you help merge this PR?

@cxzl25 cxzl25 merged commit 814d4ad into apache:master Jan 27, 2026
94 checks passed
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.

Add config to disable single-child fallback in expression conversion

3 participants