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

Changed the UpdateAll method, added an extra '#' symbol to the table … #84

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ulise28
Copy link

@ulise28 ulise28 commented May 13, 2016

…created/dropped when performing the action. This would ensure that SQL server would treat the table as temporary, and no Permissions issues would appear on the UpdateAll action.

The problem we encountered was when the application user had no rights to create/drop table in the main database. We followed the generated queries and realized that SQL Server does not treat a table named like temp_AAAA_635987400336237839 as temporary, but by simply adding a '#' character the permissions issue is gone. The push request is only related to this problem

…created/dropped when performing the action. This would ensure that SQL server would treat the table as temporary, and no Permissions issues would appear on the UpdateAll action.
@brunosantosfarfetch
Copy link

Do you have any ideia, when this correction will be merged to "main"? I have this exact problem

@ulise28
Copy link
Author

ulise28 commented Jun 6, 2016

No , still no news from the author, for now I'm using my version of the library and keeping an eye on any updates here :)

@brunorenn
Copy link

I have the same problem. That Would be great feature!

@RudeySH
Copy link

RudeySH commented Mar 2, 2019

I'm maintaining a fork of EFUtilities here: https://github.com/RudeySH/EntityFramework.Utilities . Among many other improvements, it contains @ulise28 's changes to make the tables temporary.

NuGet package available here: https://www.nuget.org/packages/RudeySH.EFUtilities/

@brunorenn
Copy link

@RudeySH thanks !

@ulise28
Copy link
Author

ulise28 commented Mar 7, 2019

Thanks!!! I will also try to update to your version

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