Partial support for non-2012 sql

[fsprojects]

Partial support for non-2012 sql

[rojepp]

I've been experimenting with converting to set fmtonly on and it seems hard to get this to work without knowing the types of the parameters. I get all sorts of errors. Maybe this isn't possible after all... I'll think about it some more.

[dmitry-a-morozov]

I got the same impression trying combination of general ADO.NET schema collections and ExecuteReader with CommandBehaviour.SchemaOnly.

[rojepp]

It is not an easy problem. The possible types that would satisfy running the query with SchemaOnly are few, however. Seeing as it is only compile time, maybe it would be possible to try some known variants before failing out?

Think about a query like this:

select top (@top) 
from tbl t
where t.ColName = @param

Given that we know the name of each param, but not the type, We could try sending int for @top, parse the error, then try '', parse the error again. If the error now says @param is wrong, then do the same for that. 0 works for numerics, '' works for strings, dates. With those two, most types are covered. We can add more as we see the need.

It would be awesome to remove the temp table restriction..

[dmitry-a-morozov]

I'm afraid that we'll end up writing some sort of SQL parser. I would rather provide solution that works well under certain limitations that something that "kinda works" but has wider applicability. Anyway, I'll keep thinking about it. We can save it version or 1.5 or 2.0.

[dsevastianov]

I agree with @dmitry-a-morozov, writing sql parser is way over our original intent.

[rojepp]

I'm on the same page. I agree to keep it open though.
On Oct 27, 2013 11:49 PM, "dsevastianov" [email protected] wrote:

I agree with @dmitry-a-morozov https://github.com/dmitry-a-morozov,
writing sql parser is way over our original intent.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/21#issuecomment-27181683
.

[rojepp]

This can be closed?

[dsevastianov]

I'd like to play with this first, right now minimal version is 12. We should have partial supporting I think.

[rojepp]

My guess is that Sql 2014 will have better support, and the importance of this will diminish in time.
Attempting this might prove to be sisyfosian. :)

[dsevastianov]

I guess you're right, closing