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

Support UPDATE with ORDER BY #179

Merged
merged 1 commit into from Aug 19, 2017
Merged

Support UPDATE with ORDER BY #179

merged 1 commit into from Aug 19, 2017

Conversation

@adaamz
Copy link
Contributor

adaamz commented Aug 8, 2017

  • bug fix? no
  • new feature? yes
  • BC break? no
  • doc PR: later if accepted

Hi,
this PR adds support for UPDATE with ORDER BY.
This is good when you have sequnce column and you want to increment tail values in this column.
SQL for this action:

UPDATE myTable SET sort_order = sort_order + 1 WHERE sort_order > 100 ORDER BY sort_order DESC

Current approach is to write own query, as stated above.

WIP: I will add test and doc for this.

@adaamz

This comment has been minimized.

Copy link
Contributor Author

adaamz commented Aug 8, 2017

@dg

This comment has been minimized.

Copy link
Member

dg commented Aug 19, 2017

Thanks

@dg dg merged commit e8817ec into nette:master Aug 19, 2017
0 of 2 checks passed
0 of 2 checks passed
continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build failed
Details
dg added a commit that referenced this pull request Aug 19, 2017
dg added a commit that referenced this pull request Aug 19, 2017
@adaamz adaamz deleted the adaamz:patch-1 branch Aug 19, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.