Skip to content

Conversation

@vanzin
Copy link
Contributor

@vanzin vanzin commented May 5, 2018

It was missing the jax-rs annotation.

It was missing the jax-rs annotation.
@SparkQA
Copy link

SparkQA commented May 5, 2018

Test build #90239 has finished for PR 21245 at commit 81a1180.

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

@mgaido91
Copy link
Contributor

mgaido91 commented May 5, 2018

LGTM

Copy link
Member

@dongjoon-hyun dongjoon-hyun left a comment

Choose a reason for hiding this comment

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

+1, LGTM.

Copy link
Contributor

@jiangxb1987 jiangxb1987 left a comment

Choose a reason for hiding this comment

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

LGTM

@jerryshao
Copy link
Contributor

LGTM, merging to master and branch 2.3.

asfgit pushed a commit that referenced this pull request May 8, 2018
It was missing the jax-rs annotation.

Author: Marcelo Vanzin <[email protected]>

Closes #21245 from vanzin/SPARK-24188.

Change-Id: Ib338e34b363d7c729cc92202df020dc51033b719
(cherry picked from commit 05eb19b)
Signed-off-by: jerryshao <[email protected]>
@asfgit asfgit closed this in 05eb19b May 8, 2018
@vanzin vanzin deleted the SPARK-24188 branch May 18, 2018 21:28
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.

7 participants