Skip to content

Conversation

@maropu
Copy link
Member

@maropu maropu commented Jan 5, 2016

We can handle posgresql-specific enum types as strings in jdbc.
So, we should just add tests and close the corresponding JIRA ticket.

@SparkQA
Copy link

SparkQA commented Jan 5, 2016

Test build #48755 has finished for PR 10596 at commit 6460aca.

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

@maropu maropu force-pushed the AddTestsInIntegration branch from 6460aca to a24c798 Compare January 14, 2016 06:26
@maropu
Copy link
Member Author

maropu commented Jan 14, 2016

@liancheng @yhuai ping

@SparkQA
Copy link

SparkQA commented Jan 14, 2016

Test build #49386 has finished for PR 10596 at commit a24c798.

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

@maropu
Copy link
Member Author

maropu commented Jan 21, 2016

@liancheng @yhuai ping

@liancheng
Copy link
Contributor

Sorry for being late. This LGTM, could you please rebase it? Then I can merge it. Thanks!

@maropu maropu force-pushed the AddTestsInIntegration branch from a24c798 to dbc6829 Compare January 26, 2016 02:23
@maropu
Copy link
Member Author

maropu commented Jan 26, 2016

@liancheng Rebased. We need to wait for the test done.

@SparkQA
Copy link

SparkQA commented Jan 26, 2016

Test build #50062 has finished for PR 10596 at commit dbc6829.

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

@maropu
Copy link
Member Author

maropu commented Jan 26, 2016

@liancheng Okay and ready to merge.

@maropu
Copy link
Member Author

maropu commented Jan 28, 2016

@liancheng ping

@liancheng
Copy link
Contributor

Thank, I'm merging this!

@asfgit asfgit closed this in abae889 Jan 28, 2016
@maropu maropu deleted the AddTestsInIntegration branch July 5, 2017 11:46
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