[Django] #29444: Allow modification of RETURNING values in django.contrib.postgres

classic Classic list List threaded Threaded
32 messages Options
12
Reply | Threaded
Open this post in threaded view
|

[Django] #29444: Allow modification of RETURNING values in django.contrib.postgres

Django
#29444: Allow modification of RETURNING values in django.contrib.postgres
-------------------------------------+-------------------------------------
               Reporter:  Johannes   |          Owner:  (none)
  Hoppe                              |
                   Type:  New        |         Status:  new
  feature                            |
              Component:             |        Version:  2.0
  contrib.postgres                   |       Keywords:  django, db,
               Severity:  Normal     |  returning, default, model, field
           Triage Stage:             |      Has patch:  0
  Unreviewed                         |
    Needs documentation:  0          |    Needs tests:  0
Patch needs improvement:  0          |  Easy pickings:  0
                  UI/UX:  0          |
-------------------------------------+-------------------------------------
 Dependency for #27452

 The SQL `RETURNING` statement is currently used for inserts of new
 objects. Mainly it is responsible to return the primary key in a single
 query.

 That's great but currently in the postgres database backend this value is
 hard coded to only the primary key. I would like to propose to make this
 configurable.

 There are plenty of uses cases for such a feature ultimately even enabling
 database defaults, but I would suggest to keep it very simple for now.

 I would suggest to simply allow model fields to specify if they should be
 included in the returning statement or not. The default being not (except
 PK) to maintain the current behavior.

 The implementation as well as the related code lines have been already
 discussed here:
 https://github.com/django/django/pull/7525#issuecomment-354269077

 I highly suggest to keep this a Postgres ONLY feature for now. That way we
 can see what sticks and consider extending this functionality into
 `django.db` in the future after we have seen how it's being used and
 gathered feedback.

--
Ticket URL: <https://code.djangoproject.com/ticket/29444>
Django <https://code.djangoproject.com/>
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 [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/052.664c756aedf4e5d67e8f5771c5b96d87%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #29444: Allow modification of RETURNING values in django.contrib.postgres

Django
#29444: Allow modification of RETURNING values in django.contrib.postgres
-------------------------------------+-------------------------------------
     Reporter:  Johannes Hoppe       |                    Owner:  (none)
         Type:  New feature          |                   Status:  new
    Component:  contrib.postgres     |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  django, db,          |             Triage Stage:
  returning, default, model, field   |  Unreviewed
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Johannes Hoppe):

 * version:  2.0 => master


--
Ticket URL: <https://code.djangoproject.com/ticket/29444#comment:1>
Django <https://code.djangoproject.com/>
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 [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.96aee39622de83089e9eec5bface59a7%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #29444: Allow modification of RETURNING values in django.contrib.postgres

Django
In reply to this post by Django
#29444: Allow modification of RETURNING values in django.contrib.postgres
-------------------------------------+-------------------------------------
     Reporter:  Johannes Hoppe       |                    Owner:  Johannes
                                     |  Hoppe
         Type:  New feature          |                   Status:  assigned
    Component:  contrib.postgres     |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  django, db,          |             Triage Stage:
  returning, default, model, field   |  Unreviewed
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Johannes Hoppe):

 * owner:  (none) => Johannes Hoppe
 * status:  new => assigned


--
Ticket URL: <https://code.djangoproject.com/ticket/29444#comment:2>
Django <https://code.djangoproject.com/>
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 [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.a935d4becc1050f7fd2ba540f4852de0%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #29444: Allow modification of RETURNING values in django.contrib.postgres

Django
In reply to this post by Django
#29444: Allow modification of RETURNING values in django.contrib.postgres
-------------------------------------+-------------------------------------
     Reporter:  Johannes Hoppe       |                    Owner:  Johannes
                                     |  Hoppe
         Type:  New feature          |                   Status:  assigned
    Component:  contrib.postgres     |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  django, db,          |             Triage Stage:
  returning, default, model, field   |  Unreviewed
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Johannes Hoppe):

 * has_patch:  0 => 1


--
Ticket URL: <https://code.djangoproject.com/ticket/29444#comment:3>
Django <https://code.djangoproject.com/>
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 [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.83ce5db7d20a5c66612a3e173b8da6a2%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #29444: Allow modification of RETURNING values in django.contrib.postgres

Django
In reply to this post by Django
#29444: Allow modification of RETURNING values in django.contrib.postgres
-------------------------------------+-------------------------------------
     Reporter:  Johannes Hoppe       |                    Owner:  Johannes
                                     |  Hoppe
         Type:  New feature          |                   Status:  assigned
    Component:  contrib.postgres     |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  django, db,          |             Triage Stage:  Accepted
  returning, default, model, field   |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  1
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Johannes Hoppe):

 * needs_better_patch:  0 => 1
 * stage:  Unreviewed => Accepted


--
Ticket URL: <https://code.djangoproject.com/ticket/29444#comment:4>
Django <https://code.djangoproject.com/>
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 [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.98d06a0008d837d49408df5889dbb096%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #29444: Allow modification of RETURNING values in django.contrib.postgres

Django
In reply to this post by Django
#29444: Allow modification of RETURNING values in django.contrib.postgres
-------------------------------------+-------------------------------------
     Reporter:  Johannes Hoppe       |                    Owner:  Johannes
                                     |  Hoppe
         Type:  New feature          |                   Status:  assigned
    Component:  contrib.postgres     |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  django, db,          |             Triage Stage:  Accepted
  returning, default, model, field   |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Johannes Hoppe):

 * needs_better_patch:  1 => 0


--
Ticket URL: <https://code.djangoproject.com/ticket/29444#comment:5>
Django <https://code.djangoproject.com/>
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 [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.8dad8fe0d64557039ca8520cd237e03e%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #29444: Allow modification of RETURNING values in django.contrib.postgres

Django
In reply to this post by Django
#29444: Allow modification of RETURNING values in django.contrib.postgres
-------------------------------------+-------------------------------------
     Reporter:  Johannes Hoppe       |                    Owner:  Johannes
                                     |  Hoppe
         Type:  New feature          |                   Status:  assigned
    Component:  contrib.postgres     |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  django, db,          |             Triage Stage:  Accepted
  returning, default, model, field   |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  1
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Tim Graham):

 * needs_better_patch:  0 => 1


--
Ticket URL: <https://code.djangoproject.com/ticket/29444#comment:6>
Django <https://code.djangoproject.com/>
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 [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.92c08e0c84332c308156045c9a363d73%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #29444: Allow modification of RETURNING values in django.contrib.postgres

Django
In reply to this post by Django
#29444: Allow modification of RETURNING values in django.contrib.postgres
-------------------------------------+-------------------------------------
     Reporter:  Johannes Hoppe       |                    Owner:  Johannes
         Type:                       |  Hoppe
  Cleanup/optimization               |                   Status:  assigned
    Component:  contrib.postgres     |                  Version:  master
     Severity:  Normal               |               Resolution:
     Keywords:  django, db,          |             Triage Stage:  Accepted
  returning, default, model, field   |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Johannes Hoppe):

 * needs_better_patch:  1 => 0
 * type:  New feature => Cleanup/optimization


--
Ticket URL: <https://code.djangoproject.com/ticket/29444#comment:7>
Django <https://code.djangoproject.com/>
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 [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.b172db5986f0c123e5970e2ab0ed480b%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #29444: Allow fields to be part of the RETURNING clause during INSERT (was: Allow modification of RETURNING values in django.contrib.postgres)

Django
In reply to this post by Django
#29444: Allow fields to be part of the RETURNING clause during INSERT
-------------------------------------+-------------------------------------
     Reporter:  Johannes Hoppe       |                    Owner:  Johannes
         Type:                       |  Hoppe
  Cleanup/optimization               |                   Status:  assigned
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:
     Keywords:  django, db,          |             Triage Stage:  Accepted
  returning, default, model, field   |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  1
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Tim Graham):

 * needs_better_patch:  0 => 1
 * component:  contrib.postgres => Database layer (models, ORM)


Comment:

 A number of Oracle test failures remain.

--
Ticket URL: <https://code.djangoproject.com/ticket/29444#comment:8>
Django <https://code.djangoproject.com/>
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 [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.ce0a34529ece916478e4b6ba0263177e%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #29444: Allow fields to be part of the RETURNING clause during INSERT

Django
In reply to this post by Django
#29444: Allow fields to be part of the RETURNING clause during INSERT
-------------------------------------+-------------------------------------
     Reporter:  Johannes Hoppe       |                    Owner:  Johannes
         Type:                       |  Hoppe
  Cleanup/optimization               |                   Status:  assigned
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:
     Keywords:  django, db,          |             Triage Stage:  Accepted
  returning, default, model, field   |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Johannes Hoppe):

 * needs_better_patch:  1 => 0


--
Ticket URL: <https://code.djangoproject.com/ticket/29444#comment:9>
Django <https://code.djangoproject.com/>
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 [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.f13ee8639d8ab1fa0e827f83423e243e%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #29444: Allow fields to be part of the RETURNING clause during INSERT

Django
In reply to this post by Django
#29444: Allow fields to be part of the RETURNING clause during INSERT
-------------------------------------+-------------------------------------
     Reporter:  Johannes Hoppe       |                    Owner:  Johannes
         Type:                       |  Hoppe
  Cleanup/optimization               |                   Status:  assigned
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:
     Keywords:  django, db,          |             Triage Stage:  Accepted
  returning, default, model, field   |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Johannes Hoppe):

 * needs_better_patch:  1 => 0


Comment:

 I fixed a couple of things. Nothing fails for me now locally that doesn't
 fail on master. Can you trigger it again?

--
Ticket URL: <https://code.djangoproject.com/ticket/29444#comment:9>
Django <https://code.djangoproject.com/>
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 [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.cf5ec0770e46ec0105350abe6287e8e9%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #29444: Allow fields to be part of the RETURNING clause during INSERT

Django
In reply to this post by Django
#29444: Allow fields to be part of the RETURNING clause during INSERT
-------------------------------------+-------------------------------------
     Reporter:  Johannes Hoppe       |                    Owner:  Johannes
         Type:                       |  Hoppe
  Cleanup/optimization               |                   Status:  assigned
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:
     Keywords:  django, db,          |             Triage Stage:  Accepted
  returning, default, model, field   |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Tim Graham <timograham@…>):

 In [changeset:"b131f9c79f3284f7220c65ceb9125ba4278dc0a4" b131f9c7]:
 {{{
 #!CommitTicketReference repository=""
 revision="b131f9c79f3284f7220c65ceb9125ba4278dc0a4"
 Refs #29444 -- Renamed DatabaseFeatures.can_return_id* to be generic for
 other columns.
 }}}

--
Ticket URL: <https://code.djangoproject.com/ticket/29444#comment:10>
Django <https://code.djangoproject.com/>
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 [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.f068f144fdfbf5a12683f7fb69e5bd39%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #29444: Allow fields to be part of the RETURNING clause during INSERT

Django
In reply to this post by Django
#29444: Allow fields to be part of the RETURNING clause during INSERT
-------------------------------------+-------------------------------------
     Reporter:  Johannes Hoppe       |                    Owner:  Johannes
         Type:                       |  Hoppe
  Cleanup/optimization               |                   Status:  assigned
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:
     Keywords:  django, db,          |             Triage Stage:  Accepted
  returning, default, model, field   |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  1
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Tim Graham):

 * needs_better_patch:  0 => 1


Comment:

 You should be able to trigger the Oracle build yourself as described on
 wiki:Jenkins. I ran the tests locally and put the failures in a PR
 comment.

--
Ticket URL: <https://code.djangoproject.com/ticket/29444#comment:11>
Django <https://code.djangoproject.com/>
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 [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.d4648fe79149b404eb369d30b07cf182%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #29444: Allow fields to be part of the RETURNING clause during INSERT

Django
In reply to this post by Django
#29444: Allow fields to be part of the RETURNING clause during INSERT
-------------------------------------+-------------------------------------
     Reporter:  Johannes Hoppe       |                    Owner:  Johannes
         Type:                       |  Hoppe
  Cleanup/optimization               |                   Status:  assigned
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:
     Keywords:  django, db,          |             Triage Stage:  Accepted
  returning, default, model, field   |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Johannes Hoppe):

 * needs_better_patch:  1 => 0


--
Ticket URL: <https://code.djangoproject.com/ticket/29444#comment:12>
Django <https://code.djangoproject.com/>
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 [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.3cd6a4309e1eec3bde668ede89db3fcf%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #29444: Allow fields to be part of the RETURNING clause during INSERT

Django
In reply to this post by Django
#29444: Allow fields to be part of the RETURNING clause during INSERT
-------------------------------------+-------------------------------------
     Reporter:  Johannes Hoppe       |                    Owner:  Johannes
         Type:                       |  Hoppe
  Cleanup/optimization               |                   Status:  assigned
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:
     Keywords:  django, db,          |             Triage Stage:  Accepted
  returning, default, model, field   |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  1                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Carlton Gibson):

 * needs_tests:  0 => 1


--
Ticket URL: <https://code.djangoproject.com/ticket/29444#comment:13>
Django <https://code.djangoproject.com/>
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 [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.13bfbab20bd6a24295a596eb60155d5a%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #29444: Allow fields to be part of the RETURNING clause during INSERT

Django
In reply to this post by Django
#29444: Allow fields to be part of the RETURNING clause during INSERT
-------------------------------------+-------------------------------------
     Reporter:  Johannes Hoppe       |                    Owner:  Johannes
         Type:                       |  Hoppe
  Cleanup/optimization               |                   Status:  assigned
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:
     Keywords:  django, db,          |             Triage Stage:  Accepted
  returning, default, model, field   |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Johannes Hoppe):

 * needs_tests:  1 => 0


--
Ticket URL: <https://code.djangoproject.com/ticket/29444#comment:14>
Django <https://code.djangoproject.com/>
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 [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.6fd34d3d9512cccce082f1ab04f0cb3f%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #29444: Allow fields to be part of the RETURNING clause during INSERT

Django
In reply to this post by Django
#29444: Allow fields to be part of the RETURNING clause during INSERT
-------------------------------------+-------------------------------------
     Reporter:  Johannes Hoppe       |                    Owner:  Johannes
         Type:                       |  Hoppe
  Cleanup/optimization               |                   Status:  assigned
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:
     Keywords:  django, db,          |             Triage Stage:  Accepted
  returning, default, model, field   |
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  1
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by felixxm):

 * needs_better_patch:  0 => 1
 * has_patch:  1 => 0


--
Ticket URL: <https://code.djangoproject.com/ticket/29444#comment:15>
Django <https://code.djangoproject.com/>
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 [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.46c5bb97bd1a9df5de80b52be40c72ce%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #29444: Allow fields to be part of the RETURNING clause during INSERT

Django
In reply to this post by Django
#29444: Allow fields to be part of the RETURNING clause during INSERT
-------------------------------------+-------------------------------------
     Reporter:  Johannes Hoppe       |                    Owner:  Johannes
         Type:                       |  Hoppe
  Cleanup/optimization               |                   Status:  assigned
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:
     Keywords:  django, db,          |             Triage Stage:  Accepted
  returning, default, model, field   |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  1
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by felixxm):

 * needs_better_patch:  0 => 1


--
Ticket URL: <https://code.djangoproject.com/ticket/29444#comment:15>
Django <https://code.djangoproject.com/>
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 [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.422d4280b527874a817593ef514b422a%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #29444: Allow fields to be part of the RETURNING clause during INSERT

Django
In reply to this post by Django
#29444: Allow fields to be part of the RETURNING clause during INSERT
-------------------------------------+-------------------------------------
     Reporter:  Johannes Hoppe       |                    Owner:  Johannes
         Type:                       |  Hoppe
  Cleanup/optimization               |                   Status:  assigned
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:
     Keywords:  django, db,          |             Triage Stage:  Accepted
  returning, default, model, field   |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Johannes Hoppe):

 * needs_better_patch:  1 => 0


--
Ticket URL: <https://code.djangoproject.com/ticket/29444#comment:16>
Django <https://code.djangoproject.com/>
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 [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.88bd3770ed3540700adf9880af9e3204%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #29444: Allow fields to be part of the RETURNING clause during INSERT

Django
In reply to this post by Django
#29444: Allow fields to be part of the RETURNING clause during INSERT
-------------------------------------+-------------------------------------
     Reporter:  Johannes Hoppe       |                    Owner:  Johannes
         Type:                       |  Hoppe
  Cleanup/optimization               |                   Status:  assigned
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:
     Keywords:  django, db,          |             Triage Stage:  Accepted
  returning, default, model, field   |
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Mariusz Felisiak <felisiak.mariusz@…>):

 In [changeset:"bc91f27a86090b4c688b56cd4e37f95eebe6e969" bc91f27a]:
 {{{
 #!CommitTicketReference repository=""
 revision="bc91f27a86090b4c688b56cd4e37f95eebe6e969"
 Refs #29444 -- Added support for fetching a returned non-integer insert
 values on Oracle.

 This is currently not actively used, since the ORM will ask the
 SQL compiler to only return auto fields.
 }}}

--
Ticket URL: <https://code.djangoproject.com/ticket/29444#comment:17>
Django <https://code.djangoproject.com/>
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 [hidden email].
To post to this group, send email to [hidden email].
To view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/067.5c9a8e0dd3eb34b1cd51380dd1c1632a%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
12