-
Notifications
You must be signed in to change notification settings - Fork 78
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Accept uuid types Iss30 #55
Conversation
@@ -0,0 +1,55 @@ | |||
context("Convenience Functions") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The generic functions are now mostly (if not entirely) tested by the DBitest package. I'd be happy to review pull requests for new tests there.
@krlmlr Thanks for the feedback. I went through and made those modifications. |
@@ -280,6 +280,8 @@ class PqResult : boost::noncopyable { | |||
case 1184: // TIMESTAMPTZOID | |||
case 1186: // INTERVAL | |||
case 1266: // TIMETZOID | |||
case 3802: // JSONB |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change seems unrelated to the original issue. I would be better to open a new PR for this, but I don't mind including it if you could provide a test.
Thanks! The dbDisconnect() should better be in on.exit(), but this doesn't matter too much for the tests. |
@krlmlr Thank you for the feedback and help. I really appreciate it. |
If everyone is ok with uuid types being character classes in R then this is ready to merge.
Fixes #30.