Skip to content

Small test fix #1311

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
martijnvg merged 1 commit into from
Jul 2, 2019
Merged

Small test fix #1311

martijnvg merged 1 commit into from
Jul 2, 2019

Conversation

@martijnvg
Copy link
Member

@martijnvg martijnvg commented Jul 1, 2019

Assert results variable instead of dataFrame

@martijnvg martijnvg requested a review from jbaiera July 1, 2019 17:33
jbaiera
jbaiera approved these changes Jul 1, 2019
View reviewed changes
Copy link
Member

@jbaiera jbaiera left a comment

Choose a reason for hiding this comment

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

LGTM, nice find!

@martijnvg martijnvg merged commit d4340b8 into elastic:master Jul 2, 2019
martijnvg added a commit that referenced this pull request Jul 2, 2019
@martijnvg
Assert results variable instead of dataFrame. (#1311)
2b9db87
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jbaiera jbaiera jbaiera approved these changes

Assignees

No one assigned

Labels

:Spark :Testing

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@martijnvg @jbaiera