Re: [Django] #34125: Limiting QuerySet crashes on union() with a single non-empty query

2022-10-29 Thread Django 
#34125: Limiting QuerySet crashes on union() with a single non-empty query
-+-
 Reporter:  Stefan Hammer|Owner:  Simon
 |  Charette
 Type:  Bug  |   Status:  closed
Component:  Database layer   |  Version:  dev
  (models, ORM)  |
 Severity:  Release blocker  |   Resolution:  fixed
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by GitHub ):

 * status:  assigned => closed
 * resolution:   => fixed


Comment:

 In [changeset:"c2cc80756b8949cdd87b88bbfdfee698ced441e0" c2cc807]:
 {{{
 #!CommitTicketReference repository=""
 revision="c2cc80756b8949cdd87b88bbfdfee698ced441e0"
 Fixed #34125 -- Fixed sliced QuerySet.union() crash on a single non-empty
 queryset.

 The bug existed since sliced query union was added but was elevated to
 query union slices by moving the .exists() optimization to the compiler
 in 3d734c09ff0138441dfe0a59010435871d17950f.

 Thanks Stefan Hammer for the report.
 }}}

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/01070184229eb987-6e783a2c-599b-4e27-aad3-c3ebe0370122-00%40eu-central-1.amazonses.com.

Re: [Django] #34125: Limiting QuerySet crashes on union() with a single non-empty query

2022-10-28 Thread Django 
#34125: Limiting QuerySet crashes on union() with a single non-empty query
-+-
 Reporter:  Stefan Hammer|Owner:  Simon
 |  Charette
 Type:  Bug  |   Status:  assigned
Component:  Database layer   |  Version:  dev
  (models, ORM)  |
 Severity:  Release blocker  |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Simon Charette):

 The patch in its current form should address the reported issue and also
 cover other use cases that were not covered when dealing with slicing
 combined queries that are sliced themselves.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/01070184219a58a2-c31a86ba-3267-4cf6-b605-cc706a60c7a4-00%40eu-central-1.amazonses.com.

Re: [Django] #34125: Limiting QuerySet crashes on union() with a single non-empty query

2022-10-27 Thread Django 
#34125: Limiting QuerySet crashes on union() with a single non-empty query
-+-
 Reporter:  Stefan Hammer|Owner:  Simon
 |  Charette
 Type:  Bug  |   Status:  assigned
Component:  Database layer   |  Version:  dev
  (models, ORM)  |
 Severity:  Release blocker  |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Simon Charette):

 * has_patch:  0 => 1


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/010701841d01c911-e520483e-3c70-4365-b84a-d16b19228fa0-00%40eu-central-1.amazonses.com.

Re: [Django] #34125: Limiting QuerySet crashes on union() with a single non-empty query

2022-10-27 Thread Django 
#34125: Limiting QuerySet crashes on union() with a single non-empty query
-+-
 Reporter:  Stefan Hammer|Owner:  Simon
 |  Charette
 Type:  Bug  |   Status:  assigned
Component:  Database layer   |  Version:  dev
  (models, ORM)  |
 Severity:  Release blocker  |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Mariusz Felisiak):

 * Attachment "test-34125.diff" added.

 Regression test.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/010701841cb3cd26-af0111ea-efa7-4e52-8407-1d8200e1fa36-00%40eu-central-1.amazonses.com.

Re: [Django] #34125: Limiting QuerySet crashes on union() with a single non-empty query

2022-10-27 Thread Django 
#34125: Limiting QuerySet crashes on union() with a single non-empty query
-+-
 Reporter:  Stefan Hammer|Owner:  Simon
 |  Charette
 Type:  Bug  |   Status:  assigned
Component:  Database layer   |  Version:  dev
  (models, ORM)  |
 Severity:  Release blocker  |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Simon Charette):

 * owner:  nobody => Simon Charette
 * status:  new => assigned


-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/010701841cb38b86-e957be9d-0d3f-4ef4-8e9e-ed76daff983c-00%40eu-central-1.amazonses.com.

Re: [Django] #34125: Limiting QuerySet crashes on union() with a single non-empty query

2022-10-27 Thread Django 
#34125: Limiting QuerySet crashes on union() with a single non-empty query
-+-
 Reporter:  Stefan Hammer|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  Database layer   |  Version:  dev
  (models, ORM)  |
 Severity:  Release blocker  |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Mariusz Felisiak):

 * cc: Simon Charette (added)
 * severity:  Normal => Release blocker
 * stage:  Unreviewed => Accepted


Comment:

 Thanks for the report! I attached a regression test which generates the
 following SQL on PostgreSQL:
 {{{#!sql
   (SELECT "queries_number"."id",
   "queries_number"."num",
   "queries_number"."other_num",
   "queries_number"."another_num"
FROM "queries_number"
WHERE "queries_number"."num" <= 0
LIMIT 1)
 LIMIT 1;
 }}}



 Regression in 3d734c09ff0138441dfe0a59010435871d17950f.
 Reproduced at de2c2127b66e77a034c01c81753c5c08e651a5b4.

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/010701841cb2ef66-961e4611-ea7a-4cb8-a9eb-f4cf011fae09-00%40eu-central-1.amazonses.com.

Re: [Django] #34125: Limiting QuerySet crashes on union() with a single non-empty query (was: Limitting QuerySet crashes on union() with a single non-empty query)

2022-10-27 Thread Django 
#34125: Limiting QuerySet crashes on union() with a single non-empty query
-+-
 Reporter:  Stefan Hammer|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  Database layer   |  Version:  dev
  (models, ORM)  |
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:
 |  Unreviewed
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

-- 
Ticket URL: 
Django 
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to django-updates+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/010701841b738197-89b646f0-36d4-4c4f-897c-dbeea5db2649-00%40eu-central-1.amazonses.com.