Skip to content

Conversation

HyukjinKwon
Copy link
Member

What changes were proposed in this pull request?

This PR updates other missed lists in branch-2.1 for dependency update of Janino.

How was this patch tested?

Jenkins tests

@HyukjinKwon HyukjinKwon changed the title [SPARK-22373][BUILD][BRANCH-2.1] Updates other dependency lists too for Janino [SPARK-22373][BUILD][FOLLOWUP][BRANCH-2.1] Updates other dependency lists too for Janino Dec 2, 2017
Copy link
Member

@srowen srowen left a comment

Choose a reason for hiding this comment

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

Ack, yeah I have forgotten this before, that branch 2.1 has additional deps files. Thanks for patching this one!

@SparkQA
Copy link

SparkQA commented Dec 2, 2017

Test build #84388 has finished for PR 19866 at commit 5fdf624.

  • This patch fails build dependency tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@HyukjinKwon HyukjinKwon force-pushed the branch-2.1-SPARK-22373 branch from 5fdf624 to 2b467c2 Compare December 2, 2017 12:09
@SparkQA
Copy link

SparkQA commented Dec 2, 2017

Test build #84389 has finished for PR 19866 at commit 2b467c2.

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

@srowen
Copy link
Member

srowen commented Dec 2, 2017

Merged to 2.1

asfgit pushed a commit that referenced this pull request Dec 2, 2017
…ists too for Janino

## What changes were proposed in this pull request?

This PR updates other missed lists in branch-2.1 for dependency update of Janino.

## How was this patch tested?

Jenkins tests

Author: hyukjinkwon <[email protected]>

Closes #19866 from HyukjinKwon/branch-2.1-SPARK-22373.
@HyukjinKwon HyukjinKwon closed this Dec 2, 2017
@HyukjinKwon HyukjinKwon deleted the branch-2.1-SPARK-22373 branch January 2, 2018 03:37
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