Skip to content

Conversation

@yhuai
Copy link
Contributor

@yhuai yhuai commented May 1, 2015

@yhuai
Copy link
Contributor Author

yhuai commented May 1, 2015

@rxin @marmbrus

Copy link
Member

Choose a reason for hiding this comment

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

+1 looks like the right thing to do according to the docs:
http://docs.oracle.com/javase/7/docs/api/java/sql/Driver.html#getParentLogger()

@rxin
Copy link
Contributor

rxin commented May 1, 2015

LGTM.

@SparkQA
Copy link

SparkQA commented May 1, 2015

Test build #31611 has finished for PR 5847 at commit 68433a2.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • "public class " + className + extendsText + " implements java.io.Serializable
    • class DataFrameStatFunctions(object):

@marmbrus
Copy link
Contributor

marmbrus commented May 1, 2015

Thanks for the quick fix. Merged to master.

@asfgit asfgit closed this in 41c6a44 May 1, 2015
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request May 28, 2015
JIRA: https://issues.apache.org/jira/browse/SPARK-7312

Author: Yin Huai <[email protected]>

Closes apache#5847 from yhuai/jdbcJava6 and squashes the following commits:

68433a2 [Yin Huai] compile with Java 6
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request Jun 12, 2015
JIRA: https://issues.apache.org/jira/browse/SPARK-7312

Author: Yin Huai <[email protected]>

Closes apache#5847 from yhuai/jdbcJava6 and squashes the following commits:

68433a2 [Yin Huai] compile with Java 6
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
JIRA: https://issues.apache.org/jira/browse/SPARK-7312

Author: Yin Huai <[email protected]>

Closes apache#5847 from yhuai/jdbcJava6 and squashes the following commits:

68433a2 [Yin Huai] compile with Java 6
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.

5 participants