Skip to content
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.

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

[SPARK-19843] [SQL] [Followup] Classdoc for IntWrapper and LongWrapper #17205

Closed
wants to merge 1 commit into from

Conversation

tejasapatil
Copy link
Contributor

What changes were proposed in this pull request?

This is as per suggestion by @rxin at : #17184 (comment)

How was this patch tested?

NA as this is a documentation change

@SparkQA
Copy link

SparkQA commented Mar 8, 2017

Test build #74183 has started for PR 17205 at commit f1e09ce.

@cloud-fan
Copy link
Contributor

LGTM

@rxin
Copy link
Contributor

rxin commented Mar 8, 2017

LGTM too

@cloud-fan
Copy link
Contributor

retest this please

@SparkQA
Copy link

SparkQA commented Mar 8, 2017

Test build #74194 has finished for PR 17205 at commit f1e09ce.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
  • public static class IntWrapper

@rxin
Copy link
Contributor

rxin commented Mar 8, 2017

Merging in master.

@asfgit asfgit closed this in e420fd4 Mar 8, 2017
@tejasapatil tejasapatil deleted the SPARK-19843_followup branch March 8, 2017 18:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants