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

[as400] Fixed ordering when limit and offset present #789

Closed
wants to merge 1 commit into from

Conversation

rsov
Copy link
Contributor

@rsov rsov commented May 14, 2017

Re-wrote the query in replace_limit_offset! method in db2 adapter. SQL is simple to follow and it handles ordering

@kares
Copy link
Member

kares commented May 15, 2017

thanks, will this work in other DB2 systems besides AS400? (I mean the 'regular' DB2 one can install)
could you please verify by running the suite ?

@kares kares added the db2 label May 15, 2017
@rsov
Copy link
Contributor Author

rsov commented May 15, 2017

I tried to connect to the db2 side of our as400 but could not get it to work. But I'll give it a try once we figure out why its not working

@kares
Copy link
Member

kares commented Jun 22, 2017

ping ... if you're really into having this in for next 1.3 release. isn't planned yet but its been a month or so.

@rsov
Copy link
Contributor Author

rsov commented Jun 28, 2017

@kares Sorry, I could not connect to DB2, only AS400. The tests that were originally failing are not passing but I am not sure what else it will effect

@rsov
Copy link
Contributor Author

rsov commented Jun 28, 2017

@kares Turns out there is no way I can test this. Its a product that needs purchasing and setting up. Currently we are running JDBC on AS400, because could not get the DB2 connection to work

@kares
Copy link
Member

kares commented Jun 28, 2017

@rsov that's weird I had it installed just fine on a (virtual) Linux box. there should be a developer edition.
guess one needs to register though ... https://www-01.ibm.com/marketing/iwm/iwm/web/preLogin.do?source=swg-db2dtrial

@rsov
Copy link
Contributor Author

rsov commented Jun 28, 2017

@kares I did not know about this product. Thank you. Will look into it in a few days

@kares kares removed the needs-input label Sep 24, 2017
@kares
Copy link
Member

kares commented Sep 24, 2017

did a quick run, seems that tests run fine and actually 2 errors went away, thus put this in 15d7d1f

@kares kares closed this Sep 24, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants