Skip to content

Conversation

@jackylk
Copy link
Contributor

@jackylk jackylk commented Mar 13, 2016

What changes were proposed in this pull request?

fix typo in DataSourceRegister

How was this patch tested?

found when going through latest code

found when going through latest code
@jackylk jackylk changed the title fix typo in DataSourceRegister [SQL] fix typo in DataSourceRegister Mar 13, 2016
@srowen
Copy link
Member

srowen commented Mar 13, 2016

OK. Have you checked to see if there are any other similar typos to correct while you're at it?

@SparkQA
Copy link

SparkQA commented Mar 13, 2016

Test build #53037 has finished for PR 11686 at commit 22ec56b.

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

@rxin
Copy link
Contributor

rxin commented Mar 14, 2016

Thanks - merging in master & branch-1.6.

As @srowen said, it'd be great to do more of these in a single batch in the future.

@asfgit asfgit closed this in f3daa09 Mar 14, 2016
asfgit pushed a commit that referenced this pull request Mar 14, 2016
## What changes were proposed in this pull request?
fix typo in DataSourceRegister

## How was this patch tested?

found when going through latest code

Author: Jacky Li <[email protected]>

Closes #11686 from jackylk/patch-12.

(cherry picked from commit f3daa09)
Signed-off-by: Reynold Xin <[email protected]>
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request Mar 17, 2016
## What changes were proposed in this pull request?
fix typo in DataSourceRegister

## How was this patch tested?

found when going through latest code

Author: Jacky Li <[email protected]>

Closes apache#11686 from jackylk/patch-12.
roygao94 pushed a commit to roygao94/spark that referenced this pull request Mar 22, 2016
## What changes were proposed in this pull request?
fix typo in DataSourceRegister

## How was this patch tested?

found when going through latest code

Author: Jacky Li <[email protected]>

Closes apache#11686 from jackylk/patch-12.
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.

4 participants