Skip to content

Conversation

@mgaido91
Copy link
Contributor

@mgaido91 mgaido91 commented Mar 9, 2018

What changes were proposed in this pull request?

Backport of ea48099 to branch 2.3.

How was this patch tested?

added UT

cc @cloud-fan @hvanhovell

@mgaido91 mgaido91 changed the title [SPARK-23628][SQL] Fix calculateParamLength [SPARK-23628][SQL][BACKPORT-2.3] calculateParamLength should not return 1 + num of expressions Mar 9, 2018
@SparkQA
Copy link

SparkQA commented Mar 9, 2018

Test build #88120 has finished for PR 20783 at commit fbe9db6.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@cloud-fan
Copy link
Contributor

thanks, merging to 2.3!

asfgit pushed a commit that referenced this pull request Mar 9, 2018
…rn 1 + num of expressions

## What changes were proposed in this pull request?

Backport of ea48099 to branch 2.3.

## How was this patch tested?

added UT

cc cloud-fan hvanhovell

Author: Marco Gaido <[email protected]>

Closes #20783 from mgaido91/SPARK-23628_2.3.
@mgaido91 mgaido91 closed this Mar 10, 2018
@mgaido91 mgaido91 deleted the SPARK-23628_2.3 branch March 10, 2018 12:04
peter-toth pushed a commit to peter-toth/spark that referenced this pull request Oct 6, 2018
…rn 1 + num of expressions

## What changes were proposed in this pull request?

Backport of ea48099 to branch 2.3.

## How was this patch tested?

added UT

cc cloud-fan hvanhovell

Author: Marco Gaido <[email protected]>

Closes apache#20783 from mgaido91/SPARK-23628_2.3.
otterc pushed a commit to linkedin/spark that referenced this pull request Mar 22, 2023
…rn 1 + num of expressions

Backport of ea48099 to branch 2.3.

added UT

cc cloud-fan hvanhovell

Author: Marco Gaido <[email protected]>

Closes apache#20783 from mgaido91/SPARK-23628_2.3.

(cherry picked from commit 3ec25d5)

RB=1512778
BUG=LIHADOOP-42994
G=superfriends-reviewers
R=fli,mshen,yezhou,edlu,zolin
A=zolin
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants