diff options
| author | erebus1 <erebus@mail.ua> | 2016-06-05 12:56:25 +0300 |
|---|---|---|
| committer | erebus1 <erebus@mail.ua> | 2016-06-05 12:56:25 +0300 |
| commit | a39774e3f568aa964c1d99da2ff2d4654a291bf7 (patch) | |
| tree | 3ceb1c177149ab37b665c5d8810676017ca5f39c /MANIFEST.in | |
| parent | 991346d5bbc29479c6c500c3f2b64fd6cc2e9a39 (diff) | |
| download | sqlalchemy-pr/283.tar.gz | |
Raise error on empty in_pr/283
I think it would be good idea to add error on empy in_ usage in version 1.1.*
Because current implementation is not fully correct:
Firstly, it's not optimised, because of sequential scan
Secondly, in current implementation null in () will return null, when in sql it should return False, try this:
select null in (select 1 from some_table where false); -> false but not null
I understant that this can break some legacy code, but current implementation is obscure and quite expensive
Diffstat (limited to 'MANIFEST.in')
0 files changed, 0 insertions, 0 deletions
