Re: [Django] #22879: Database errors coming from cursor iterator are not converted to Django database errors

[Django]

#22879: Database errors coming from cursor iterator are not converted to Django
database errors
-+-
 Reporter:  denisenkom@… |Owner:  nobody
 Type:  Bug  |   Status:  closed
Component:  Database layer   |  Version:  1.6
  (models, ORM)  |   Resolution:  fixed
 Severity:  Normal   | Triage Stage:  Ready for
 Keywords:   |  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Anssi Kääriäinen ):

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


Comment:

 In [changeset:"d4bc0bf435402f32d1e31aeffd280276349283d8"]:
 {{{
 #!CommitTicketReference repository=""
 revision="d4bc0bf435402f32d1e31aeffd280276349283d8"
 Fixed #22879 -- Intercept errors coming from cursor.__next__()
 }}}

--
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/078.b82464850ee6eccc6b39c5569a56aa8f%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22879: Database errors coming from cursor iterator are not converted to Django database errors

[Django]

#22879: Database errors coming from cursor iterator are not converted to Django
database errors
-+-
 Reporter:  denisenkom@… |Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  Database layer   |  Version:  1.6
  (models, ORM)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Ready for
 Keywords:   |  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by timgraham):

 * stage:  Accepted => Ready for checkin


Comment:

 According to the PR, there is already such a test in Django's test suite.

--
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/078.c7bf9a3a6727fecb5d4e4f0ee5c92603%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22879: Database errors coming from cursor iterator are not converted to Django database errors

[Django]

#22879: Database errors coming from cursor iterator are not converted to Django
database errors
-+-
 Reporter:  denisenkom@… |Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  Database layer   |  Version:  1.6
  (models, ORM)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:   |  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  0
  Needs tests:  1|UI/UX:  0
Easy pickings:  0|
-+-
Changes (by timgraham):

 * needs_better_patch:  1 => 0


Comment:

 Patch is now simpler. How about including a regression test for MSSQL
 Server even if it passes on all the bundled backends?

--
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/078.6178a1bfe1e105f14de4a7a49e870703%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22879: Database errors coming from cursor iterator are not converted to Django database errors

[Django]

#22879: Database errors coming from cursor iterator are not converted to Django
database errors
-+-
 Reporter:  denisenkom@… |Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  Database layer   |  Version:  1.6
  (models, ORM)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:   |  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  1
  Needs tests:  1|UI/UX:  0
Easy pickings:  0|
-+-
Changes (by tchaumeny):

 * needs_better_patch:  0 => 1
 * needs_tests:  0 => 1


Comment:

 As discussed on https://github.com/django/django/pull/2839, the patch
 could be simplified a bit.

--
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/078.f85cf3d5287eb7184afc30182f4ef370%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22879: Database errors coming from cursor iterator are not converted to Django database errors

[Django]

#22879: Database errors coming from cursor iterator are not converted to Django
database errors
-+-
 Reporter:  denisenkom@… |Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  Database layer   |  Version:  1.6
  (models, ORM)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:   |  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  0
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-
Changes (by aaugustin):

 * type:  Uncategorized => Bug


--
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/078.83e8fe78a5688b09654cf4d8bede1436%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22879: Database errors coming from cursor iterator are not converted to Django database errors

[Django]

#22879: Database errors coming from cursor iterator are not converted to Django
database errors
-+-
 Reporter:  denisenkom@… |Owner:  nobody
 Type:  Uncategorized|   Status:  new
Component:  Database layer   |  Version:  1.6
  (models, ORM)  |   Resolution:
 Severity:  Normal   | Triage Stage:  Accepted
 Keywords:   |  Needs documentation:  0
Has patch:  1|  Patch needs improvement:  0
  Needs tests:  0|UI/UX:  0
Easy pickings:  0|
-+-
Changes (by aaugustin):

 * stage:  Unreviewed => Accepted


--
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/078.4e8ecff98284f2613cf7b97d64174e27%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #22879: Database errors coming from cursor iterator are not converted to Django database errors

[Django]

#22879: Database errors coming from cursor iterator are not converted to Django
database errors
-+-
 Reporter:  denisenkom@… |Owner:  nobody
 Type:  Uncategorized|   Status:  new
Component:  Database layer   |  Version:  1.6
  (models, ORM)  |   Resolution:
 Severity:  Normal   | Triage Stage:
 Keywords:   |  Unreviewed
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by timo):

 * needs_docs:   => 0
 * has_patch:  0 => 1
 * component:  Uncategorized => Database layer (models, ORM)
 * needs_tests:   => 0
 * needs_better_patch:   => 0


Comment:

 [https://github.com/django/django/pull/2839 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 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/078.2016e51c4ef6bbe919c98d4ad5002577%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

[Django] #22879: Database errors coming from cursor iterator are not converted to Django database errors

[Django]

#22879: Database errors coming from cursor iterator are not converted to Django
database errors
---+
 Reporter:  denisenkom@…   |  Owner:  nobody
 Type:  Uncategorized  | Status:  new
Component:  Uncategorized  |Version:  1.6
 Severity:  Normal |   Keywords:
 Triage Stage:  Unreviewed |  Has patch:  0
Easy pickings:  0  |  UI/UX:  0
---+
 Here is one possible situation:

 {{{
 Traceback (most recent call last):
   File "/opt/python-2.7/lib/python2.7/threading.py", line 552, in
 __bootstrap_inner
 self.run()
   File "/opt/python-2.7/lib/python2.7/threading.py", line 505, in run
 self.__target(*self.__args, **self.__kwargs)
   File "/home/denisenk/django-
 sqlserver/env/src/django/tests/select_for_update/tests.py", line 245, in
 raw
 connection.ops.for_update_sql(nowait=True)
   File "/home/denisenk/django-
 sqlserver/env/src/django/django/db/models/query.py", line 1427, in
 __iter__
 for values in query:
   File "/home/denisenk/django-sqlserver/env/lib/python2.7/site-
 packages/six.py", line 473, in next
 return type(self).__next__(self)
   File "/home/denisenk/django-sqlserver/env/src/pytds/pytds/__init__.py",
 line 703, in __next__
 row = self.fetchone()
   File "/home/denisenk/django-sqlserver/env/src/pytds/pytds/__init__.py",
 line 676, in fetchone
 row = self._session.fetchone()
   File "/home/denisenk/django-sqlserver/env/src/pytds/pytds/tds.py", line
 3429, in fetchone
 if not self.next_row():
   File "/home/denisenk/django-sqlserver/env/src/pytds/pytds/tds.py", line
 3443, in next_row
 self.process_end(marker)
   File "/home/denisenk/django-sqlserver/env/src/pytds/pytds/tds.py", line
 2654, in process_end
 self.raise_db_exception()
   File "/home/denisenk/django-sqlserver/env/src/pytds/pytds/tds.py", line
 2443, in raise_db_exception
 raise ex
 OperationalError: Lock request time out period exceeded.
 }}}

-- 
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/063.634d8795896f3deb65c3dc454343d3bd%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.