Commit message (Collapse) | Author | Age | Files | Lines | ||
---|---|---|---|---|---|---|
... | ||||||
* | | Merge pull request #330 from sgrif/sg-binary-node-hash-equality | Sean Griffin | 2014-10-24 | 2 | -1/+27 | |
|\ \ | |/ |/| | Binary nodes should not generate the same hash as nodes of other classes | |||||
| * | Binary nodes should not generate the same hash as nodes of other classes | Sean Griffin | 2014-10-24 | 2 | -1/+27 | |
|/ | ||||||
* | Merge pull request #326 from claudiob/reuse-maybe-visit | Rafael Mendonça França | 2014-10-23 | 1 | -3/+3 | |
|\ | | | | | Completes 08e6491 in reusing `maybe_visit` | |||||
| * | Completes 08e6491 in reusing `maybe_visit` | claudiob | 2014-10-23 | 1 | -3/+3 | |
|/ | | | | | | | | | :sweat: I don't know why the tests did not fail, but to keep the same syntax as before, `collector =` is required. Maybe `visit` changes `collector` in-place, so the result is the same, but since I'm not sure about the side effects, I think this PR is needed to. Sorry! :sweat: | |||||
* | Merge pull request #325 from claudiob/reuse-maybe-visit-method | Rafael Mendonça França | 2014-10-23 | 2 | -18/+4 | |
|\ | | | | | Reuse `maybe_visit` method | |||||
| * | Reuse `maybe_visit` method | claudiob | 2014-10-23 | 2 | -18/+4 | |
|/ | | | | | This commit simply removes duplicated code by reusing the existing `maybe_visit` method. | |||||
* | Merge pull request #324 from vipulnsward/cte | Matthew Draper | 2014-10-24 | 1 | -1/+1 | |
|\ | | | | | `Expresssions` => `Expressions` | |||||
| * | `Expresssions` => `Expressions` [ci skip] | Vipul A M | 2014-10-24 | 1 | -1/+1 | |
|/ | ||||||
* | Merge pull request #318 from tamird/match-escape | Matthew Draper | 2014-09-26 | 6 | -20/+60 | |
|\ | | | | | {Matches,DoesNotMatch} support the ESCAPE clause | |||||
| * | {Matches,DoesNotMatch} support the ESCAPE clause | Tamir Duberstein | 2014-09-25 | 6 | -20/+60 | |
|/ | ||||||
* | move the dispatch table to be per-instance | Aaron Patterson | 2014-09-22 | 8 | -26/+38 | |
| | | | | | | | visitors are not shared among threads, so any mutations to the cache should be OK. The cache is also pre-populated on construction, but we should pull that out so we can share the cache among visitors in the future. | |||||
* | connections should cache these values, so remove the cache in arel | Aaron Patterson | 2014-09-22 | 1 | -4/+3 | |
| | ||||||
* | Merge pull request #317 from dimko/patch-2 | Rafael Mendonça França | 2014-09-17 | 1 | -1/+1 | |
|\ | | | | | Fix set visiting spec | |||||
| * | Fix set visiting spec | Dimko | 2014-09-17 | 1 | -1/+1 | |
|/ | ||||||
* | Merge pull request #167 from pparidans/informix-v9-support | Rafael Mendonça França | 2014-09-17 | 2 | -7/+7 | |
|\ | | | | | Informix versions < 10 use 'FIRST' keyword instead of 'LIMIT' | |||||
| * | Informix versions < 10 use 'FIRST' keyword instead of 'LIMIT' | Pierre Paridans | 2014-09-16 | 2 | -7/+7 | |
| | | | | | | | | Still supported in versions 10+ | |||||
* | | Merge pull request #137 from dimko/visit_set | Rafael Mendonça França | 2014-09-17 | 5 | -0/+16 | |
|\ \ | |/ |/| | Added a visitor for Set objects | |||||
| * | added a visitor for Set objects | Dimko | 2012-09-18 | 5 | -0/+16 | |
| | | ||||||
* | | Merge pull request #316 from vipulnsward/remove-alias | Matthew Draper | 2014-09-17 | 1 | -1/+1 | |
|\ \ | | | | | | | Remove unused @alias, being referenced in hashing. | |||||
| * | | Remove unused @alias, being referenced in hashing. | Vipul A M | 2014-09-16 | 1 | -1/+1 | |
|/ / | ||||||
* | | Build on ruby-head | Matthew Draper | 2014-09-17 | 1 | -1/+3 | |
| | | ||||||
* | | Merge pull request #315 from tamird/129 | Matthew Draper | 2014-09-17 | 2 | -1/+11 | |
|\ \ | | | | | | | Wrap group_by sql with a subquery for limiting rows in Oracle. | |||||
| * | | Wrap group_by sql with a subquery for limiting rows in Oracle. | Chris Dinger | 2014-09-16 | 2 | -1/+11 | |
|/ / | | | | | | | Fixes #128 | |||||
* | | Merge pull request #311 from tamird/master | Rafael Mendonça França | 2014-09-16 | 3 | -18/+12 | |
|\ \ | | | | | | | `Extract#as` should not mutate the receiver | |||||
| * | | `Extract#as` should not mutate the receiver | Tamir Duberstein | 2014-09-13 | 3 | -18/+12 | |
|/ / | | | | | | | Fixes https://github.com/rails/rails/issues/16913 | |||||
* | | Merge pull request #306 from mxhold/master | Rafael Mendonça França | 2014-09-12 | 2 | -0/+30 | |
|\ \ | | | | | | | Add SelectManager#distinct_on to set/unset Arel::Nodes::DistinctOn quantifier | |||||
| * | | Add SelectManager#distinct_on to set/unset Arel::Nodes::DistinctOn quantifier | Max Holder | 2014-08-30 | 2 | -0/+30 | |
| | | | ||||||
* | | | Merge pull request #304 from bigxiang/fix_lt_and_lteq_error | Rafael Mendonça França | 2014-09-12 | 2 | -6/+50 | |
|\ \ \ | | | | | | | | | Fix lt & lteq don't accept most of values. | |||||
| * | | | Change specs to cover lt, lteq, gt and gteq. | bigxiang | 2014-08-31 | 1 | -4/+42 | |
| | | | | ||||||
| * | | | Fix lt & lteq don't accept most of values. | bigxiang | 2014-08-23 | 2 | -6/+12 | |
| | | | | ||||||
* | | | | Merge pull request #277 from sergey-alekseev/wrap-nested-gropings-only-once | Rafael Mendonça França | 2014-09-12 | 2 | -2/+13 | |
|\ \ \ \ | |_|/ / |/| | | | Wrap nested Nodes::Grouping in brackets only once | |||||
| * | | | Wrap nested Nodes::Grouping in brackets only once | Sergey Alekseev | 2014-05-26 | 2 | -2/+13 | |
| | | | | ||||||
* | | | | Merge pull request #300 from jpcody/master | Matthew Draper | 2014-08-26 | 3 | -19/+17 | |
|\ \ \ \ | |_|/ / |/| | | | Allow for alias omission in aggregate expressions | |||||
| * | | | Remove default aliases from aggregate functions | Joshua Cody | 2014-07-31 | 4 | -58/+20 | |
| | | | | ||||||
| * | | | Allow for alias omission in aggregate expressions | Joshua Cody | 2014-07-26 | 2 | -8/+44 | |
| | | | | ||||||
* | | | | Merge pull request #303 from ksss/typo | Rafael Mendonça França | 2014-08-21 | 1 | -1/+1 | |
|\ \ \ \ | | | | | | | | | | | Fix typo `gt` means `>` | |||||
| * | | | | Fix typo `gt` means `>` | ksss | 2014-08-21 | 1 | -1/+1 | |
|/ / / / | ||||||
* / / / | Prepare to 6.0.0.beta1 | Rafael Mendonça França | 2014-08-17 | 3 | -13/+13 | |
|/ / / | ||||||
* | | | Merge pull request #291 from vipulnsward/rename-test | Rafael Mendonça França | 2014-06-30 | 1 | -52/+53 | |
|\ \ \ | | | | | | | | | Cleanup SelectManager Test | |||||
| * | | | - Stop passing redundant `:engine => Table.engine` to `Table#new` | Vipul A M | 2014-06-15 | 1 | -51/+52 | |
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Remove redundant `ast` variable creation - Rename similar join tests name to what they actually test - Move "joins itself" test to "joins" describe instead of defining describe block twice - Move update manager statement tests in logical order - Move `project` tests in logical order | |||||
| * | | | Rename ambiguous test name to what it actually tests | Vipul A M | 2014-06-15 | 1 | -1/+1 | |
| | | | | ||||||
* | | | | Merge pull request #163 from atombender/master | Aaron Patterson | 2014-06-26 | 5 | -7/+83 | |
|\ \ \ \ | | | | | | | | | | | Support for PARTITION BY in window | |||||
| * | | | | Add test for window + partition + order. | Alexander Staubo | 2014-06-22 | 1 | -0/+11 | |
| | | | | | ||||||
| * | | | | Windowing: Calling #rows or #range should assign framing only once. | Alexander Staubo | 2014-06-22 | 2 | -2/+12 | |
| | | | | | ||||||
| * | | | | Windowing support for PARTITION BY clause. | Alexander Staubo | 2014-06-20 | 5 | -5/+60 | |
|/ / / / | ||||||
* | | | | Merge pull request #292 from mekishizufu/dot_visitor_api | Rafael Mendonça França | 2014-06-15 | 5 | -21/+30 | |
|\ \ \ \ | | | | | | | | | | | Modify Visitors::Dot's API to use collectors | |||||
| * | | | | Modify Visitors::Dot's API to use collectors | Jiri Pospisil | 2014-06-14 | 3 | -7/+9 | |
| | | | | | ||||||
| * | | | | Collectors: Introduce PlainString and make use of it in SQLString | Jiri Pospisil | 2014-06-14 | 2 | -14/+21 | |
| |/ / / | ||||||
* | | | | Merge pull request #293 from vipulnsward/cleanup-select-manager | Rafael Mendonça França | 2014-06-15 | 1 | -3/+8 | |
|\ \ \ \ | |/ / / |/| | | | Cleanup Insert Manager | |||||
| * | | | - Test noop when passing empty list to `insert` | Vipul A M | 2014-06-15 | 1 | -3/+8 | |
|/ / / | | | | | | | | | | | | | - Rename into test to indicate it accepts table and chains on it - Rename "combo" test to what it actually tests |