Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

please pull this #25

Merged
merged 2 commits into from Jun 29, 2012

Conversation

Projects
None yet
2 participants
@jjYBdx4IL
Copy link
Contributor

jjYBdx4IL commented Jun 27, 2012

see commit messages for details.

@jjYBdx4IL

This comment has been minimized.

Copy link
Contributor Author

jjYBdx4IL commented Jun 27, 2012

"no SQL generation" means "bad SQL generation", ie. needs further changes I have currently no plan on working on them.

@jjYBdx4IL

This comment has been minimized.

Copy link
Contributor Author

jjYBdx4IL commented Jun 27, 2012

Someone interested may use sybase ase dev edition for testing/development.

installer: https://gist.github.com/3003417

Maybe one should add that fact (and that SQL generation for ase stored procs does not work yet) to the docs.

@lukaseder

This comment has been minimized.

Copy link
Member

lukaseder commented Jun 29, 2012

Thanks for this contribution. I will have a closer look at your first commit. Currently, there was no support for Sybase ASE stored procedures as there seems not to be a lot of traction for using jOOQ with Sybase:
http://blog.jooq.org/2012/04/12/which-database-are-you-using-jooq-with/

Complete support will be tracked as #1507:
https://sourceforge.net/apps/trac/jooq/ticket/1507

The second commit seems to fix #1498 (a.k.a. GitHub issue #24):
https://sourceforge.net/apps/trac/jooq/ticket/1498

I will test this with JDK 7 / JDBC 4.1 shortly.

lukaseder added a commit that referenced this pull request Jun 29, 2012

@lukaseder lukaseder merged commit a128acc into jOOQ:master Jun 29, 2012

@lukaseder

This comment has been minimized.

Copy link
Member

lukaseder commented Jun 29, 2012

I think I will not be able to support Sybase ASE stored procedures in jOOQ 2.4.0, hence, I'll comment out that section of the code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.