Skip to content

Conversation

@wangyum
Copy link
Member

@wangyum wangyum commented May 20, 2017

What changes were proposed in this pull request?

Add cot test in MathExpressionsSuite as #17999 (comment).

How was this patch tested?

unit tests

@SparkQA
Copy link

SparkQA commented May 20, 2017

Test build #77113 has started for PR 18039 at commit 57719f1.

@wangyum
Copy link
Member Author

wangyum commented May 20, 2017

Jenkins, retest this please

@wzhfy
Copy link
Contributor

wzhfy commented May 20, 2017

It's a followup pr for SPARK-20751. Could you add a followup tag in the title?

@wangyum wangyum changed the title [SPARK-20751][SQL] Add cot test in MathExpressionsSuite [SPARK-20751][SQL][FOLLOWUP] Add cot test in MathExpressionsSuite May 20, 2017
@SparkQA
Copy link

SparkQA commented May 20, 2017

Test build #77117 has finished for PR 18039 at commit 57719f1.

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

Copy link
Member

@gatorsmile gatorsmile left a comment

Choose a reason for hiding this comment

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

LGTM

@gatorsmile
Copy link
Member

Thanks! Merging to master.

@asfgit asfgit closed this in 9b09101 May 22, 2017
lycplus pushed a commit to lycplus/spark that referenced this pull request May 24, 2017
## What changes were proposed in this pull request?

Add cot test in MathExpressionsSuite as apache#17999 (comment).

## How was this patch tested?

unit tests

Author: Yuming Wang <[email protected]>

Closes apache#18039 from wangyum/SPARK-20751-test.
@wangyum wangyum deleted the SPARK-20751-test branch May 27, 2017 02:17
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.

5 participants