Skip to content

Conversation

@trystanleftwich
Copy link

… sql, for 1.6

@AmplabJenkins
Copy link

Can one of the admins verify this patch?

@hvanhovell
Copy link
Contributor

@trystanleftwich could you please add a test for this?

@trystanleftwich
Copy link
Author

I'm not sure what the best way to test the code would be, I've made a stab at it but any suggestions for it would be good.

@rxin
Copy link
Contributor

rxin commented Jun 15, 2016

Thanks for the pull request. I'm going through a list of pull requests to cut them down since the sheer number is breaking some of the tooling we have. Due to lack of activity on this pull request, I'm going to push a commit to close it. Feel free to reopen it or create a new one. We can also continue the discussion on the JIRA ticket.

@asfgit asfgit closed this in 1a33f2e Jun 15, 2016
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