Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations.

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations.
-+-
 Reporter:  FunkyBob |Owner:  Dan Tao
 Type:  New feature  |   Status:  closed
Component:  contrib.postgres |  Version:  master
 Severity:  Normal   |   Resolution:  fixed
 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 Mariusz Felisiak ):

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


Comment:

 In [changeset:"85ac838d9e6975130b5c55299e9d7d1222a8e289" 85ac838d]:
 {{{
 #!CommitTicketReference repository=""
 revision="85ac838d9e6975130b5c55299e9d7d1222a8e289"
 Fixed #21039 -- Added AddIndexConcurrently/RemoveIndexConcurrently
 operations for PostgreSQL.

 Thanks to Simon Charettes for review.

 Co-Authored-By: Daniel Tao 
 }}}

-- 
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/066.8e74a8f1cd5f03b3b46de2270fd37ec2%40djangoproject.com.

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations. (was: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations)

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations.
-+-
 Reporter:  FunkyBob |Owner:  Dan Tao
 Type:  New feature  |   Status:  assigned
Component:  contrib.postgres |  Version:  master
 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 felixxm):

 * needs_better_patch:  1 => 0
 * stage:  Accepted => Ready for checkin
 * component:  Migrations => contrib.postgres


-- 
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/066.2b94228734f106497ca75c59e3149fec%40djangoproject.com.

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+
 Reporter:  FunkyBob |Owner:  Dan Tao
 Type:  New feature  |   Status:  assigned
Component:  Migrations   |  Version:  master
 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 felixxm):

 * needs_better_patch:  0 => 1
 * stage:  Ready for checkin => 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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/066.b850daa2732c33673e4ef88195ef2431%40djangoproject.com.

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+-
 Reporter:  FunkyBob |Owner:  Dan Tao
 Type:  New feature  |   Status:  assigned
Component:  Migrations   |  Version:  master
 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 Simon Charette):

 * stage:  Accepted => Ready for checkin


-- 
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/066.b65010d242e4eacc877b34178e931e99%40djangoproject.com.

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+
 Reporter:  FunkyBob |Owner:  Dan Tao
 Type:  New feature  |   Status:  assigned
Component:  Migrations   |  Version:  master
 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 Mads Jensen):

 * needs_better_patch:  1 => 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/066.d5ae88ed4b4740bb19ef756977c1a81c%40djangoproject.com.

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+
 Reporter:  FunkyBob |Owner:  Dan Tao
 Type:  New feature  |   Status:  assigned
Component:  Migrations   |  Version:  master
 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 Mads Jensen):

 * needs_better_patch:  0 => 1


Comment:

 Since there are still things that need work, I'm marking this as WIP.

-- 
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/066.a18b8d2337b4ab275e71a90a390b7179%40djangoproject.com.

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+
 Reporter:  FunkyBob |Owner:  Dan Tao
 Type:  New feature  |   Status:  assigned
Component:  Migrations   |  Version:  master
 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 felixxm):

 * needs_better_patch:  1 => 0


Comment:

 [https://github.com/django/django/pull/11594 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/066.829e88edab5006a24d7ea2ae39dfdac6%40djangoproject.com.

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+
 Reporter:  FunkyBob |Owner:  Dan Tao
 Type:  New feature  |   Status:  assigned
Component:  Migrations   |  Version:  master
 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


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

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+-
 Reporter:  FunkyBob |Owner:  Dan Tao
 Type:  New feature  |   Status:  assigned
Component:  Migrations   |  Version:  master
 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 Asif Saifuddin Auvi):

 * stage:  Accepted => Ready for checkin


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

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+
 Reporter:  FunkyBob |Owner:  Dan Tao
 Type:  New feature  |   Status:  assigned
Component:  Migrations   |  Version:  master
 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 Dan Tao):

 * needs_docs:  1 => 0


Comment:

 I've
 
[https://github.com/django/django/pull/10901/commits/b98a0db0f0d33852a74c0ce53d44ddbd1af74d9b
 added documentation], so I'm unsetting the **Needs documentation** flag.
 Of course I am happy to add more docs or update what I've written based on
 feedback.

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

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+
 Reporter:  FunkyBob |Owner:  Dan Tao
 Type:  New feature  |   Status:  assigned
Component:  Migrations   |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  1
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+
Changes (by Dan Tao):

 * owner:  (none) => Dan Tao
 * status:  new => assigned


Comment:

 Per [https://docs.djangoproject.com/en/dev/internals/contributing/writing-
 code/submitting-patches/#claiming-tickets the guidelines on claiming
 tickets], I'm assigning this to myself.

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

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+
 Reporter:  FunkyBob |Owner:  (none)
 Type:  New feature  |   Status:  new
Component:  Migrations   |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  1
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+
Changes (by Tim Graham):

 * needs_docs:  0 => 1


Comment:

 Please add documentation. See the
 [https://docs.djangoproject.com/en/dev/internals/contributing/writing-code
 /submitting-patches/#patch-review-checklist patch review checklist].

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

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+
 Reporter:  FunkyBob |Owner:  (none)
 Type:  New feature  |   Status:  new
Component:  Migrations   |  Version:  master
 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 Dan Tao):

 * cc: Dan Tao (added)
 * has_patch:  0 => 1


Comment:

 I've opened a pull request to add the `CreateIndexConcurrently` operation
 proposed in my previous comment:

 https://github.com/django/django/pull/10901

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

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+
 Reporter:  FunkyBob |Owner:  (none)
 Type:  New feature  |   Status:  new
Component:  Migrations   |  Version:  master
 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 Simon Charette):

 > The first option seems simpler to me, at least for now, unless there is
 compelling evidence that another database backend will support concurrent
 index creation in the near future.

 I agree with your conclusions; there's little benefit to add a
 `concurrent` option to `CreateIndex` for now. If any supported database
 adds support for this option in the future it should be easy enough to
 deprecate `contrib.postgres.CreateIndexConcurrently`.

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

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+
 Reporter:  FunkyBob |Owner:  (none)
 Type:  New feature  |   Status:  new
Component:  Migrations   |  Version:  master
 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 Dan Tao):

 Now that #25833 is resolved, it seems to me there are (at least) two
 possible ways to approach this:

 1. Add a new `CreateIndexConcurrently` operation in the
 django.contrib.postgres.operations module.
 2. Add a new `concurrent` argument to the constructor for the base
 `AddIndex` operation, and conditionally support this argument via feature
 detection based on a new attribute (maybe something like
 `supports_concurrent_index_creation`) on the base schema editor class.
 Database backends that don't support concurrent index creation (i.e.
 everything but PostgreSQL at present) would presumably raise
 `NotSupportedError` or something along those lines.

 The first option seems simpler to me, at least for now, unless there is
 compelling evidence that another database backend will support concurrent
 index creation in the near future.

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

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+
 Reporter:  FunkyBob |Owner:  (none)
 Type:  New feature  |   Status:  new
Component:  Migrations   |  Version:  master
 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 Tim Graham):

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


Comment:

 With #25833 resolved, it's not clear to me how this is a duplicate of that
 issue. #28448 is a new ticket requesting support for concurrently indexes.
 I'm reopening this ticket and closing that new ticket as a duplicate of
 this one.

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

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+-
 Reporter:  FunkyBob |Owner:
 Type:  New feature  |   Status:  closed
Component:  Migrations   |  Version:  master
 Severity:  Normal   |   Resolution:  duplicate
 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 aaugustin):

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


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

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+
 Reporter:  FunkyBob |Owner:
 Type:  New feature  |   Status:  new
Component:  Migrations   |  Version:  master
 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 shaib):

 #25833 is more general, but it would allow this to be written easily as
 user code. I'd even consider closing this ticket as a duplicate of #25833.

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

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+
 Reporter:  FunkyBob |Owner:
 Type:  New feature  |   Status:  new
Component:  Migrations   |  Version:  master
 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 erikvanzijst):

 * cc: erik.van.zijst, at, gmail (added)


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

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+
 Reporter:  FunkyBob |Owner:
 Type:  New feature  |   Status:  new
Component:  Migrations   |  Version:  master
 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 erikvanzijst):

 > In the meanwhile I suggest you create your own subclass of
 Alter(Index|Unique) operations and override database_forward to issue the
 desired SQL.

 `CREATE INDEX CONCURRENTLY` doesn't run inside a transaction. Can
 `Operation` subclasses execute SQL outside the current transaction?

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

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+
 Reporter:  FunkyBob |Owner:
 Type:  New feature  |   Status:  new
Component:  Migrations   |  Version:  master
 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 charettes):

 @BlindHunter, concurrent index creation has many
 [http://www.postgresql.org/docs/current/static/sql-createindex.html#SQL-
 CREATEINDEX-CONCURRENTLY documented caveats] hence why it's not enabled by
 default.

 I don't think we should expect newcomers to Django/PostgreSQL to make sure
 all their index have been correctly created even if their migration ran
 fine. That's the reason we're planing to make it an ''opt-in'' feature and
 not the default behavior, just like it's not the default in PostgreSQL
 itself.

 In the meanwhile I suggest you create your own subclass of
 `Alter(Index|Unique)` operations and override `database_forward` to issue
 the desired SQL.

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

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+
 Reporter:  FunkyBob |Owner:
 Type:  New feature  |   Status:  new
Component:  Migrations   |  Version:  master
 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 BlindHunter):

 Why don't we want to always add CONCURRENTLY statement ? It's a bit slower
 but allows us to execute zero downtime updates without dirty hacks like
 create "fake" migration which adds indexes and add them in management
 commands with "create index concurrently".

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

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+
 Reporter:  FunkyBob |Owner:
 Type:  New feature  |   Status:  new
Component:  Migrations   |  Version:  master
 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 andrewgodwin):

 * status:  assigned => new
 * owner:  jcatalan =>


Comment:

 This won't make it into 1.7, as it's too big of a change.

-- 
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/066.f3426ce91af005c8b0f9c7a14fde514b%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+
 Reporter:  FunkyBob |Owner:  jcatalan
 Type:  New feature  |   Status:  assigned
Component:  Migrations   |  Version:  master
 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 timo):

 * 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/066.11398a70a253a136d246c42040eca1ae%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations

[Django]

#21039: Support Postgres "CREATE INDEX CONCURRENTLY" in migrations
-+--
 Reporter:  FunkyBob |Owner:  jcatalan
 Type:  New feature  |   Status:  assigned
Component:  Migrations   |  Version:  master
 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
-+--
Changes (by jcatalan):

 * status:  new => assigned
 * needs_docs:   => 0
 * owner:   => jcatalan
 * needs_tests:   => 0
 * needs_better_patch:   => 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 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/066.5ef856717e1fcf0a573af25dc905b90f%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.