-
Notifications
You must be signed in to change notification settings - Fork 934
Ingres9 sequences #439
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
Ingres9 sequences #439
Conversation
…variable limits (NH-3281)
Closing this. I've submitted a cleaned version as PR #529. |
@oskarb I have got PR#529 and it builds ok. I cannot access the TeamCity details on why they are failing to build, but I cannot imagine how adding a couple of changes to the Ingres9 Dialect can make the builds fail for unrelated databases. I am unsure how to proceed. |
Have you executed all the tests? It's a common test that's failing. ( I'm
away from computer right now and don't remember the name.)
Den 4 dec. 2016 2:35 em skrev "Nic Gamble" <notifications@github.com>:
@oskarb <https://github.com/oskarb> I have got PR#529 and it builds ok. I
cannot access the TeamCity details on why they are failing to build, but I
cannot imagine how adding a couple of changes to the Ingres9 Dialect can
make the builds fail for unrelated databases. I am unsure how to proceed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#439 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAdbL_INGJnk8vCwlf9lqfin4AXSXG7fks5rEs-rgaJpZM4Fcwr0>
.
|
@oskarb Yes, there were plenty of failures, mostly all unable to connect to a MSSQL database. I am currently on a laptop with no DB's installed, so possibly cannot see the wood for the trees.. There is no steps in the ShowMenuBuild.bat that will setup for Ingres. Would that need to be done? |
Support for Sequence use within Ingres