Skip to content

Conversation

@marmbrus
Copy link
Contributor

No description provided.

@marmbrus
Copy link
Contributor Author

/cc @rxin @davies

Copy link
Contributor

Choose a reason for hiding this comment

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

data source should be the last argument (we did it for python)

@SparkQA
Copy link

SparkQA commented Mar 10, 2015

Test build #28415 has finished for PR 4958 at commit 19c2735.

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

@rxin
Copy link
Contributor

rxin commented Mar 11, 2015

lgtm

@SparkQA
Copy link

SparkQA commented Mar 11, 2015

Test build #28451 has finished for PR 4958 at commit d81b7e7.

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

@SparkQA
Copy link

SparkQA commented Mar 11, 2015

Test build #28452 has finished for PR 4958 at commit 9351dbc.

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

@SparkQA
Copy link

SparkQA commented Mar 11, 2015

Test build #28456 has finished for PR 4958 at commit 04b35cb.

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

asfgit pushed a commit that referenced this pull request Mar 11, 2015
Author: Michael Armbrust <[email protected]>

Closes #4958 from marmbrus/sqlDocs and squashes the following commits:

9351dbc [Michael Armbrust] fix parquet example
6877e13 [Michael Armbrust] add sql examples
d81b7e7 [Michael Armbrust] rxins comments
e393528 [Michael Armbrust] fix order
19c2735 [Michael Armbrust] more on data source load/store
00d5914 [Michael Armbrust] Update SQL Docs with JDBC and Migration Guide
@asfgit asfgit closed this in 2672374 Mar 11, 2015
@marmbrus marmbrus deleted the sqlDocs branch August 3, 2015 22:55
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