[Django] #27676: MariaDB 10.2 supports defaults for text columns

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

[Django] #27676: MariaDB 10.2 supports defaults for text columns

Django
#27676: MariaDB 10.2 supports defaults for text columns
-------------------------------------+-------------------------------------
               Reporter:  Adam       |          Owner:  Adam Chainz
  Chainz                             |
                   Type:  Bug        |         Status:  assigned
              Component:  Database   |        Version:  master
  layer (models, ORM)                |
               Severity:  Normal     |       Keywords:
           Triage Stage:             |      Has patch:  0
  Unreviewed                         |
    Needs documentation:  0          |    Needs tests:  0
Patch needs improvement:  0          |  Easy pickings:  0
                  UI/UX:  0          |
-------------------------------------+-------------------------------------
 Django has workaround code for MySQL's deficiency not supporting DEFAULT
 for TEXT, BLOB, and JSON columns, but MariaDB 10.2 fixes this ( see
 https://mariadb.com/kb/en/mariadb/what-is-mariadb-102/: "BLOB and TEXT
 fields can now have a DEFAULT value." ).

 Django should allow DEFAULT for these columns when on MariaDB 10.2+.

--
Ticket URL: <https://code.djangoproject.com/ticket/27676>
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/053.b57037bda8a23e8ffcb933b21e5d728b%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27676: MariaDB 10.2 supports defaults for text columns

Django
#27676: MariaDB 10.2 supports defaults for text columns
-------------------------------------+-------------------------------------
     Reporter:  Adam Chainz          |                    Owner:  Adam
                                     |  Chainz
         Type:  New feature          |                   Status:  assigned
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:
                                     |  Someday/Maybe
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Tim Graham):

 * type:  Bug => New feature
 * stage:  Unreviewed => Someday/Maybe


Comment:

 First some consensus is needed about whether Django
 [https://groups.google.com/d/topic/django-
 developers/31j1dNOZ7zc/discussion should officially support MariaDB].

--
Ticket URL: <https://code.djangoproject.com/ticket/27676#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/068.7bf012522484be5ae93ec5c25f1890b3%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27676: MariaDB 10.2 supports defaults for text columns

Django
In reply to this post by Django
#27676: MariaDB 10.2 supports defaults for text columns
-------------------------------------+-------------------------------------
     Reporter:  Adam Chainz          |                    Owner:  Adam
                                     |  Chainz
         Type:  New feature          |                   Status:  assigned
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:
                                     |  Someday/Maybe
    Has patch:  0                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by Adam Chainz):

 Thanks, I missed that thread, commented

--
Ticket URL: <https://code.djangoproject.com/ticket/27676#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/068.576087d3d0171d5fc4c51ecfb3549083%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27676: MariaDB 10.2 supports defaults for text columns

Django
In reply to this post by Django
#27676: MariaDB 10.2 supports defaults for text columns
-------------------------------------+-------------------------------------
     Reporter:  Adam Chainz          |                    Owner:  Adam
                                     |  Chainz
         Type:  New feature          |                   Status:  assigned
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Ready for
                                     |  checkin
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  0
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------
Changes (by Tim Graham):

 * has_patch:  0 => 1
 * stage:  Someday/Maybe => Ready for checkin


Comment:

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

--
Ticket URL: <https://code.djangoproject.com/ticket/27676#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/068.5565d13d3dd7516271f12be1648849e5%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27676: MariaDB 10.2 supports defaults for text columns

Django
In reply to this post by Django
#27676: MariaDB 10.2 supports defaults for text columns
-------------------------------------+-------------------------------------
     Reporter:  Adam (Chainz)        |                    Owner:  Adam
  Johnson                            |  (Chainz) Johnson
         Type:  New feature          |                   Status:  assigned
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    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
 * stage:  Ready for checkin => Accepted


Comment:

 On the PR, Adam said, "I was thinking of forming a better plan around
 MariaDB first."

--
Ticket URL: <https://code.djangoproject.com/ticket/27676#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/068.29e3b070bc1dcfcb8594deda782870a3%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27676: MariaDB 10.2 supports defaults for text columns

Django
In reply to this post by Django
#27676: MariaDB 10.2 supports defaults for text columns
-------------------------------------+-------------------------------------
     Reporter:  Adam (Chainz)        |                    Owner:  Adam
  Johnson                            |  (Chainz) Johnson
         Type:  New feature          |                   Status:  assigned
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               Resolution:
     Keywords:                       |             Triage Stage:  Accepted
    Has patch:  1                    |      Needs documentation:  0
  Needs tests:  0                    |  Patch needs improvement:  1
Easy pickings:  0                    |                    UI/UX:  0
-------------------------------------+-------------------------------------

Comment (by felixxm):

 Django 3.0 will officially support MariaDB 10.1+. I think it's time to
 change this feature flag. Adam, can you resubmit your patch?

--
Ticket URL: <https://code.djangoproject.com/ticket/27676#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/068.47a7ed85c85da379afdc30d00e24ece1%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27676: MariaDB 10.2 supports defaults for text columns

Django
In reply to this post by Django
#27676: MariaDB 10.2 supports defaults for text columns
-------------------------------------+-------------------------------------
     Reporter:  Adam (Chainz)        |                    Owner:  Adam
  Johnson                            |  (Chainz) Johnson
         Type:  New feature          |                   Status:  assigned
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               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 Adam (Chainz) Johnson):

 * needs_better_patch:  1 => 0


--
Ticket URL: <https://code.djangoproject.com/ticket/27676#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 view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/068.bf984cafa7bb76073bb5af3c71ec1871%40djangoproject.com.
Reply | Threaded
Open this post in threaded view
|

Re: [Django] #27676: MariaDB 10.2 supports defaults for text columns

Django
In reply to this post by Django
#27676: MariaDB 10.2 supports defaults for text columns
-------------------------------------+-------------------------------------
     Reporter:  Adam (Chainz)        |                    Owner:  Adam
  Johnson                            |  (Chainz) Johnson
         Type:  New feature          |                   Status:  closed
    Component:  Database layer       |                  Version:  master
  (models, ORM)                      |
     Severity:  Normal               |               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 Mariusz Felisiak <felisiak.mariusz@…>):

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


Comment:

 In [changeset:"7da6a28a447dc0db2a2c6ef31894094eb968f408" 7da6a28a]:
 {{{
 #!CommitTicketReference repository=""
 revision="7da6a28a447dc0db2a2c6ef31894094eb968f408"
 Fixed #27676 -- Allowed BLOB/TEXT defaults on MariaDB 10.2.1+.
 }}}

--
Ticket URL: <https://code.djangoproject.com/ticket/27676#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 view this discussion on the web visit https://groups.google.com/d/msgid/django-updates/068.3edd1fba0aaa37df3a88a17ec2e499b1%40djangoproject.com.