Re: [PR] [SPARK-51326][CONNECT] Remove LazyExpression proto message [spark]

2025-02-27 Thread via GitHub
ueshin commented on PR #50093: URL: https://github.com/apache/spark/pull/50093#issuecomment-2688893677 Thanks! merging to master. -- 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 comm

Re: [PR] [SPARK-51326][CONNECT] Remove LazyExpression proto message [spark]

2025-02-27 Thread via GitHub
ueshin closed pull request #50093: [SPARK-51326][CONNECT] Remove LazyExpression proto message URL: https://github.com/apache/spark/pull/50093 -- 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 spec

Re: [PR] [SPARK-51326][CONNECT] Remove LazyExpression proto message [spark]

2025-02-27 Thread via GitHub
ueshin commented on PR #50093: URL: https://github.com/apache/spark/pull/50093#issuecomment-2688892681 I reran the compatibility test after #50094 was merged and it passed. - https://github.com/ueshin/apache-spark/actions/runs/13554631961/job/37946210197 -- This is an automated messag

Re: [PR] [SPARK-51326][CONNECT] Remove LazyExpression proto message [spark]

2025-02-26 Thread via GitHub
ueshin commented on PR #50093: URL: https://github.com/apache/spark/pull/50093#issuecomment-2686499500 As "Protobuf breaking change detection and Python CodeGen check" failed with: ``` Error: Previously present message "LazyExpression" was deleted from file. Error: Previously pre

[PR] [SPARK-51326][CONNECT] Remove LazyExpression proto message [spark]

2025-02-26 Thread via GitHub
ueshin opened a new pull request, #50093: URL: https://github.com/apache/spark/pull/50093 ### What changes were proposed in this pull request? Removes `LazyExpression` proto message. Any features using this proto message is not released yet, so it's safe to remove from master a