Mike Stall
Mike Stall
Same issue here
@aaronhoffman - I'd like to followup with you on this. Are you still interested?
That looks like a problem with the MS build system and nuget?
This looks like a bug in Nuget/msbuild. Do we have a contact on that team?
We should solve this with some rule in the converter manager that's not Table specific. Maybe a JArray -> T[] rule?
Although a counter-point to consider: If we read T[], then the underlying table call can do a select to only pull down properties on T. Whereas if we read a...
We could solve this with a JArray --> T[] rule. This approach also lets T be a poco (and not need to derive from ITableEntity). Table could override specifically to...
> protected internal static async Task GetObject(HttpClient httpClient, string message, string uri, string content, CancellationToken cancellationToken, ConnectorLogger logger = null, [CallerMemberName] string callingMethod = "") can we remove (coming from...
My concern about current PR is that it _duplicates_ the code. I'm we can solve this in a way that avoids duplicating.
This will also flush out other concerns. We extract a connectorName from the uri: https://github.com/microsoft/Power-Fx/blob/8d2a19061d747f0baaefc35b6ee7c3bbf42c7691/src/libraries/Microsoft.PowerFx.Connectors/Tabular/CdpTableResolver.cs#L96 we should avoid this because: - Fx.Connectors (client code) shouldn't be having per-connector logic. CDP...