Merge | Netfx SMI Classes #3326
Conversation
…ject, some very light cleanup
… project, Removing obsolete code
…ame to SmiXetterAccessMap.cs
…erAction.netfx.cs in common project
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #3326 +/- ##
==========================================
- Coverage 67.74% 65.10% -2.64%
==========================================
Files 298 298
Lines 65536 65469 -67
==========================================
- Hits 44397 42624 -1773
- Misses 21139 22845 +1706
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
I'm pretty sure that everything here will be deleted when the context connection PR set (#2996 and its follow-ups) is finished. Every path here is already unused because we block context connections in the connection string. Do you still want to merge the files? I completely agree with removing the event sink code from SqlCommand. I think the original SQLCLR methodology had a set of custom event sinks and a custom SmiLink derivative in the SqlAccess assembly, so it's used to support that. |
nooooooooooooooooooooooooooooooooooooooooooooooooooooooo 😭 |
| // SqlDbTypes as rows | ||
| // bool, byte, bytes, chars, strng, int16, int32, int64, singl, doubl, sqldec, date, guid, varmd, Xetr, time, dtost | ||
| /*BigInt*/ { _, _, _, _, _, _, _, X, _, _, _, _, _, _, _, _, _, }, | ||
| /*Binary*/ { _, _, X, _, _, _, _, _, _, _, _, _, _, _, _, _, _, }, |
There was a problem hiding this comment.
What's crazier - a lot of this is just going to disappear in a later PR :)
| @@ -405,7 +405,7 @@ internal override void ParameterAvailable(SmiParameterMetaData metaData, SmiType | |||
|
|
|||
| private SmiContext _smiRequestContext; // context that _smiRequest came from | |||
| private CommandEventSink _smiEventSink; | |||
| private SmiEventSink_DeferedProcessing _outParamEventSink; | |||
| private SmiEventSink_DeferredProcessing _outParamEventSink; | |||
There was a problem hiding this comment.
Heh, I changed this on another branch of mine as well. Shouldn't conflict.
| /// <summary> | ||
| /// Formal encoding of SMI's metadata-to-ITypedSetter/-from-ITypedGetter validity rules | ||
| /// </summary> | ||
| internal partial class SmiXetterAccessMap |
There was a problem hiding this comment.
Afaik it's [G|S]etter - so this has both getters and setters
Description: This PR is a bit bigger than usual, but it's mostly cut/paste of classes from the netfx project to the common project. There are a few additional changes going on in this PR though that are worth calling out:
SmiContextFactoryto just make sure it's following some style guidelinesSmiEventStream.Disposeobsolete (since it is notated to be obsolete and only throws)SqlCommand, but there are no classes that inherit from it. All the virtual code paths overridden in it throw, so any codepath that uses it will always throw. I suspect it is ultimately not used. I will investigate if this entire codepath can be deleted.TriggerActionenum definitions derive directly fromEMDEventTypeenum definitions, which seems completely pointless to me. So, I rolled theEMDEventTypevalues into theTriggerActionenum and deletedEMDEventType.SmiXetterAccessMapclasses were combined together with#if NETFRAMEWORK.SmiEventSink_DeferedProcessingtoSmiEventSink_DeferredProcessing(typo in Deferred)Testing: Project builds locally, will wait for CI to call it good.