temp tvp types #302
temp tvp types #302
Conversation
# Conflicts: # RELEASE_NOTES.md # src/SqlClient/DesignTime.fs
# Conflicts: # build.sh # src/SqlClient.Tests/app.config # src/SqlClient.Tests/packages.config # src/SqlClient/ISqlCommand.fs # src/SqlClient/packages.config
|
@davidoptima Thanks! |
|
Hi guys, Is this included in 1.8.6 ? |
|
I see it is, but this doesn't fix call a stored procedure that uses temp tables. Right? |
|
@erlis such procedures will always require using dynamic "exec sp_execultesql" along with "with result sets ..." to describe the shape of result set explicitly. Though, I haven't tried to issue call to a procedure not having that defined but leveraging the temp table declaration with this new feature in a custom provided SqlCommand, I suspect this might work. |
|
What about modifying the ProgrammabilityProvider so we can pass the "with result sets..." to the stored procs that need that information? I'm thinking something like providing a map with StoreProcName, ResultSet for those that fail. About this:
What if the proc uses 2 or more temp tables? This is a big restriction because temp tables are used (at least in my stored procs) for performance optimizations. Let me know what do you think about my idea and I can fork and work on this change... Thanks! |
I generally do that inside the stored procedure itself, AFAIU it doesn't preclude usage of several temp tables, but maybe I'm mistaken, could you share a sample SQL snippet + TP declaration to check the issue further? I might not have faced the issue because I'm using dynamic sql statements pretty much any time temp tables are involved in my case so far but I start to see that you can have static sql and temp tables and no place to use The way you propose to solve the issue is reasonable, let's gather more feedback in #308. |
No description provided.