Re: [Django] #30448: close_if_unusable_or_obsolete should skip connections in atomic block for autocommit check

[Django]

#30448: close_if_unusable_or_obsolete should skip connections in atomic block 
for
autocommit check
-+-
 Reporter:  Daniel Hahler|Owner:  (none)
 Type:  Bug  |   Status:  new
Component:  Database layer   |  Version:  2.2
  (models, ORM)  |
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Mariusz Felisiak):

 * owner:  Daniel Hahler => (none)
 * status:  assigned => new

-- 
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/0107018e1d1a308c-84aabfa3-5f70-4d2b-932d-f8d255019047-00%40eu-central-1.amazonses.com.

Re: [Django] #30448: close_if_unusable_or_obsolete should skip connections in atomic block for autocommit check

[Django]

#30448: close_if_unusable_or_obsolete should skip connections in atomic block 
for
autocommit check
-+-
 Reporter:  Daniel Hahler|Owner:  Daniel
 |  Hahler
 Type:  Bug  |   Status:  assigned
Component:  Database layer   |  Version:  2.2
  (models, ORM)  |
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Jacob Walls):

 * owner:  nobody => Daniel Hahler
 * status:  new => assigned
 * has_patch:  0 => 1
 * needs_tests:  0 => 1


Comment:

 [https://github.com/django/django/pull/11769 PR]

-- 
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/065.117b47e297bfba08a2f7b3f3a3ed56ac%40djangoproject.com.

Re: [Django] #30448: close_if_unusable_or_obsolete should skip connections in atomic block for autocommit check

[Django]

#30448: close_if_unusable_or_obsolete should skip connections in atomic block 
for
autocommit check
-+-
 Reporter:  Daniel Hahler|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  Database layer   |  Version:  2.2
  (models, ORM)  |
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Daniel Hahler):

 Created https://github.com/django/django/pull/11769.

 > but I agree that not considering self.in_atomic_block in the check
 against settings_dict["AUTOCOMMIT"] is way too naive to determine the
 connection is unusable or obsolete.

 Not sure I understand that correctly: do you think the patch makes sense,
 or is still to naive?

-- 
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/065.f8a959dccad03442d8d1a908b8643518%40djangoproject.com.

Re: [Django] #30448: close_if_unusable_or_obsolete should skip connections in atomic block for autocommit check

[Django]

#30448: close_if_unusable_or_obsolete should skip connections in atomic block 
for
autocommit check
-+-
 Reporter:  Daniel Hahler|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  Database layer   |  Version:  2.2
  (models, ORM)  |
 Severity:  Normal   |   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):

 * stage:  Unreviewed => Accepted


Comment:

 `close_if_unusable_or_obsolete` is certainly an internal API but I agree
 that not considering `self.in_atomic_block` in the check against
 `settings_dict["AUTOCOMMIT"]` is way too naive to determine the connection
 is unusable or obsolete.

-- 
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 post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.413ba948a95cdb1629b286c99c617fd2%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.