Skip to content
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

Add From() binding for UPSERTs #4

Open
0x6e6562 opened this issue Feb 19, 2014 · 0 comments
Open

Add From() binding for UPSERTs #4

0x6e6562 opened this issue Feb 19, 2014 · 0 comments

Comments

@0x6e6562
Copy link
Member

e72de39 added the Apply(...ColumnBindings) for generating UPSERTs using ColumnBindings. This is probably messy, since you need to use the To() method to create the binding, which accepts a destination pointer. Whilst this works, I think it would be clearer to offer an explicit non-pointer method such as From(). If we were to do this, we'd probably need to create a specialized type of ColumnBinding to avoid overloading the legal arguments for To() and From().

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

No branches or pull requests

1 participant