Sybase SCOPE_IDENTITY(), some tests fixed, other skipped #95

Closed
wants to merge 3 commits into
from

Conversation

Projects
None yet
2 participants
Contributor

vjt commented Nov 6, 2012

Hi,

Sybase doesn't support SCOPE_IDENTITY(), so I've made TDS-Version dependant the choice of two SQL strings that return the last identity column in an Ident field.

This pull request is meant as a discussion of the above strategy.

Thanks,

~Marcello

vjt added some commits Nov 6, 2012

@vjt vjt Fix Sybase schema due to obscure syntax errors
An UTF8 character in a comment make Sybase choke with:

TinyTds::Error: Incorrect syntax near '@@@V0_INT91@@@V1_INT91'

Didn't investigate much, as the characters were located in comments
used only to keep the schema files up to date.
53a0131
@vjt vjt Add abstraction for different IDENTITY functions 51402c8
@vjt vjt Disable some tests for ASE, fix some others f180edb
Collaborator

metaskills commented Nov 6, 2012

Nice, I'll take a look at this soon! Ping me if I forget.

Contributor

vjt commented Nov 16, 2012

Ping! It's Friday, a wonderful day to work on Open Source code! 😉

@metaskills metaskills added a commit that referenced this pull request Nov 17, 2012

@metaskills metaskills Small cleanup for pull request #95 work. 57442f9
Collaborator

metaskills commented Nov 17, 2012

How about Saturday :)

So I got this in and id a small amount of persnickety cleanup on my part in commit 57442f9. I'm gonna work on doing another release right now. Again, great work!

metaskills closed this Nov 17, 2012

Contributor

vjt commented Nov 22, 2012

Thanks for the merge :-) /me happy

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment