subject:"Re\: \[Django\] #27685\: Allow autoreloader to use watchman"

Re: [Django] #27685: Allow autoreloader to use watchman

2019-05-14 Thread Django

#27685: Allow autoreloader to use watchman
-+-
 Reporter:  Aymeric Augustin |Owner:  Tom
 Type:   |  Forbes
  Cleanup/optimization   |   Status:  closed
Component:  Utilities|  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
-+-

Comment (by Mariusz Felisiak ):

 In [changeset:"43f54e136e9282f5c0bfcf1169d0d59b3c365add" 43f54e1]:
 {{{
 #!CommitTicketReference repository=""
 revision="43f54e136e9282f5c0bfcf1169d0d59b3c365add"
 Refs #27685 -- Logged unexpected Watchman autoreloader errors.
 }}}

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

Re: [Django] #27685: Allow autoreloader to use watchman

2019-05-01 Thread Django

#27685: Allow autoreloader to use watchman
-+-
 Reporter:  Aymeric Augustin |Owner:  Tom
 Type:   |  Forbes
  Cleanup/optimization   |   Status:  closed
Component:  Utilities|  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
-+-

Comment (by Ülgen Sarıkavak):

 Is it possible to enable "watchman unavailable" errors back? I understand
 that watchman is not the default option and printing watchman related
 errors as default not a good experience but when i try to install/use
 watchman and something goes wrong, there is no way to what is happening.
 Maybe something like `--force-watchman` or `--reloader=watchman` can do
 the work.

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

Re: [Django] #27685: Allow autoreloader to use watchman

2019-02-25 Thread Django

#27685: Allow autoreloader to use watchman
-+-
 Reporter:  Aymeric Augustin |Owner:  Tom
 Type:   |  Forbes
  Cleanup/optimization   |   Status:  closed
Component:  Utilities|  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
-+-

Comment (by Tim Graham ):

 In [changeset:"65ef5f467ba84c26392a157de1622d805401ec7d" 65ef5f46]:
 {{{
 #!CommitTicketReference repository=""
 revision="65ef5f467ba84c26392a157de1622d805401ec7d"
 Refs #27685 -- Removed "watchman unavailable" message.
 }}}

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

Re: [Django] #27685: Allow autoreloader to use watchman

2019-02-25 Thread Django

#27685: Allow autoreloader to use watchman
-+-
 Reporter:  Aymeric Augustin |Owner:  Tom
 Type:   |  Forbes
  Cleanup/optimization   |   Status:  closed
Component:  Utilities|  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
-+-

Comment (by Tim Graham ):

 In [changeset:"7331dd8a989465c3deed0db4e37ebbdf9d60bb71" 7331dd8a]:
 {{{
 #!CommitTicketReference repository=""
 revision="7331dd8a989465c3deed0db4e37ebbdf9d60bb71"
 [2.2.x] Refs #27685 -- Removed "watchman unavailable" message.

 Backport of 65ef5f467ba84c26392a157de1622d805401ec7d from master
 }}}

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

Re: [Django] #27685: Allow autoreloader to use watchman

2019-02-25 Thread Django

#27685: Allow autoreloader to use watchman
-+-
 Reporter:  Aymeric Augustin |Owner:  Tom
 Type:   |  Forbes
  Cleanup/optimization   |   Status:  closed
Component:  Utilities|  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
-+-

Comment (by Tim Graham ):

 In [changeset:"65ef5f467ba84c26392a157de1622d805401ec7d" 65ef5f46]:
 {{{
 #!CommitTicketReference repository=""
 revision="65ef5f467ba84c26392a157de1622d805401ec7d"
 Refs #27685 -- Removed "watchman unavailable" message.
 }}}

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

Re: [Django] #27685: Allow autoreloader to use watchman

2019-02-25 Thread Django

#27685: Allow autoreloader to use watchman
-+-
 Reporter:  Aymeric Augustin |Owner:  Tom
 Type:   |  Forbes
  Cleanup/optimization   |   Status:  closed
Component:  Utilities|  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
-+-

Comment (by Tim Graham ):

 In [changeset:"7331dd8a989465c3deed0db4e37ebbdf9d60bb71" 7331dd8a]:
 {{{
 #!CommitTicketReference repository=""
 revision="7331dd8a989465c3deed0db4e37ebbdf9d60bb71"
 [2.2.x] Refs #27685 -- Removed "watchman unavailable" message.

 Backport of 65ef5f467ba84c26392a157de1622d805401ec7d from master
 }}}

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

Re: [Django] #27685: Allow autoreloader to use watchman

2019-01-13 Thread Django

#27685: Allow autoreloader to use watchman
-+-
 Reporter:  Aymeric Augustin |Owner:  Tom
 Type:   |  Forbes
  Cleanup/optimization   |   Status:  closed
Component:  Utilities|  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 Tim Graham ):

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


Comment:

 In [changeset:"c8720e7696ca41f3262d5369365cc1bd72a216ca" c8720e76]:
 {{{
 #!CommitTicketReference repository=""
 revision="c8720e7696ca41f3262d5369365cc1bd72a216ca"
 Fixed #27685 -- Added watchman support to the autoreloader.

 Removed support for pyinotify (refs #9722).
 }}}

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

Re: [Django] #27685: Allow autoreloader to use watchman

2019-01-12 Thread Django

#27685: Allow autoreloader to use watchman
-+-
 Reporter:  Aymeric Augustin |Owner:  Tom
 Type:   |  Forbes
  Cleanup/optimization   |   Status:  assigned
Component:  Utilities|  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 Tim Graham):

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

Re: [Django] #27685: Allow autoreloader to use watchman

2018-11-18 Thread Django

#27685: Allow autoreloader to use watchman
-+-
 Reporter:  Aymeric Augustin |Owner:  Tom
 Type:   |  Forbes
  Cleanup/optimization   |   Status:  assigned
Component:  Utilities|  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 Tom Forbes):

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

Re: [Django] #27685: Allow autoreloader to use watchman

2018-11-15 Thread Django

#27685: Allow autoreloader to use watchman
-+-
 Reporter:  Aymeric Augustin |Owner:  Tom
 Type:   |  Forbes
  Cleanup/optimization   |   Status:  assigned
Component:  Utilities|  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


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

Re: [Django] #27685: Allow autoreloader to use watchman

2018-11-05 Thread Django

* needs_better_patch: 1 => 0

Comment:

I've updated the PR considerably since last time. I've removed the entire
pyinotify mess and replaced it with a concrete, working (at least on my
machine!) watchman implementation.

I've also worked out a pretty decent way of testing the reloaders. If we
structure the inner loop as a generator we can run individual 'ticks' in
tests using `next()`. The first tick will set up the files to be watched.
Then we modify them in some way and then trigger another tick. This should
then pick up the modifications we have made and do the right thing.

I've used a metaclass to share 8 integration tests across the watchman and
stat reloaders. Individually each class passes but when run as a whole
module the watchman ones fail (perhaps due to the metaclass?), so I need
to look into that.

The watchman service is pretty good at cleaning itself up, if we can get
it installed on a CI worker I think it would be safe to just run in the
background indefinitely?

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

Re: [Django] #27685: Allow autoreloader to use watchman

2018-10-15 Thread Django

#27685: Allow autoreloader to use watchman
-+-
 Reporter:  Aymeric Augustin |Owner:  Tom
 Type:   |  Forbes
  Cleanup/optimization   |   Status:  assigned
Component:  Utilities|  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


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

Re: [Django] #27685: Allow autoreloader to use watchman

2018-10-07 Thread Django

#27685: Allow autoreloader to use watchman
-+-
 Reporter:  Aymeric Augustin |Owner:  Tom
 Type:   |  Forbes
  Cleanup/optimization   |   Status:  assigned
Component:  Utilities|  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 Tom Forbes):

 * needs_better_patch:  1 => 0


Comment:

 I've completed what I think is the bulk of the work for the first stage of
 this, refactoring the autoreloader to allow us to add support for Watchman
 in the future. After making the autoreloader more modular I've expanded
 the tests (including ones for the inotify reloader) and fleshed out the
 API for triggering arbitrary watches.

 I think with this we can easily close #25624, #25791 and #28602 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/067.df997a4fc1d88b0cb2500d730b855f3f%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27685: Allow autoreloader to use watchman

2018-02-12 Thread Django

#27685: Allow autoreloader to use watchman
-+-
 Reporter:  Aymeric Augustin |Owner:  Tom
 Type:   |  Forbes
  Cleanup/optimization   |   Status:  assigned
Component:  Utilities|  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 Tom Forbes):

 * owner:  nobody => Tom Forbes
 * status:  new => assigned


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

Re: [Django] #27685: Allow autoreloader to use watchman

2017-12-09 Thread Django

#27685: Allow autoreloader to use watchman
--+
 Reporter:  Aymeric Augustin  |Owner:  nobody
 Type:  Cleanup/optimization  |   Status:  new
Component:  Utilities |  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 Martin):

 * needs_better_patch:  0 => 1


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

Re: [Django] #27685: Allow autoreloader to use watchman

2017-07-29 Thread Django

Comment (by Tom):

Hey Aymeric, thanks for the response!

1. I believe so. Currently I'm envisioning the watchdog backend first, as
it seems pretty easy to add. That supports watching individual directories
without a glob, so we'd need to do some manual work to find out if any
file changes match out glob expressions (which doesn't sound too
difficult?). Services like watchman take a directory and a glob pattern so
it does that for us. I can't think of any other more generic method than a
(direcrtory, glob) tuple?

2. When the autoreloader is ready to watch for events a
`autoreload_started` signal is fired, with the autoreloader instance as
it's sender. It knows when to send this signal by waiting for `apps.ready`
in a while loop, which is a bit iffy IMO. Any handlers (like the one in
`i18n`) are triggered and can use `sender.watch(...)` to subscribe to
anything they want - in i18n's case it's .mo files in various places.
When any file is changed a `file_changed` signal is sent with the path,
and each handler can do whatever it pleases with that path. If any handler
returns True then the server will not restart due to this change. I like
this approach because it's just simple signals, and you don't care about
any settings while registering them. If the debug server is not used (or
autoreload is turned off) then no signals will be sent. Simple!

3. I believe the `autoreload` module is private, but there are a few
usages on GitHub:

https://github.com/search?l=Python=django.utils.autoreload=Code=%E2%9C%93

A lot of these are similar to these:

https://github.com/JamesTing/ConnectNodes/blob/ba4f6a82416a81d31e51c00d5472ee48098388d8/
后台/reload.py
https://github.com/ch3ll0v3k/Doc-

Browser/blob/ea31f3c3ce82621f77311bcf536455cb16f137a3/pyinstaller/build/lib.linux-i686-2.7/PyInstaller/loader/rthooks/pyi_rth_django.py

https://github.com/tyru/homedir/blob/98166ffee204db6083bed644d6cb438d274e2635/bin.d/google_appengine/google/appengine/_internal/django/core/management/commands/runserver.py#L82

So to keep some semblance of compatibility we should:
Implement a `code_changed` function that just runs the current
implementation (for people using uwsgi in development?)
Add a restart_with_reloader method (seems to be for pyinstaller compat)
Add a main() method.

None of these seem insurmountable IMO.

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

Re: [Django] #27685: Allow autoreloader to use watchman

2017-07-29 Thread Django

Comment (by Aymeric Augustin):

Tom: thanks for picking up the flag! I'm super happy to see this move
forwards.

I read the description of your PR; it makes a lot of sense. Having a base
class and several implementation (possibly depending on additional
software) is an excellent starting point. It would be good to merge
regardless of what comes next. I'll try to take a closer look in the
coming days. Ping me if you haven't heard of me by mid-August.

My three questions are:

- is the directory / glob pattern configuration method sufficiently
generic to work with all backends we envision?
- how does the signal-based design work? (I'm usually not a fan of
signals, but surprisingly, I have rather positive feelings about this use
case. I just want to make sure that's really the best solution.)
- how much backwards incompatibility does this patch incur? (Larger
backwards incompatibility need larger improvements to justify them.)

PS - In addition to the mailing list threads I linked above, I think the
best summary I wrote is on the
[https://code.djangoproject.com/wiki/SummerOfCode2017#Improvedautoreloader27685
GSOC Ideas page].

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

Re: [Django] #27685: Allow autoreloader to use watchman

2017-07-28 Thread Django

#27685: Allow autoreloader to use watchman
--+
 Reporter:  Aymeric Augustin  |Owner:  nobody
 Type:  Cleanup/optimization  |   Status:  new
Component:  Utilities |  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 Tom):

 * has_patch:  0 => 1


Comment:

 I have a potential start of a patch here:
 https://github.com/django/django/pull/8819

 I think the first step is to refactor/re-write the autoreloader, and the
 next step is to add support for interesting things like watchman. The MR
 linked above is the start of the refactor.

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

Re: [Django] #27685: Allow autoreloader to use watchman

2017-07-23 Thread Django

#27685: Allow autoreloader to use watchman
--+
 Reporter:  Aymeric Augustin  |Owner:  nobody
 Type:  Cleanup/optimization  |   Status:  new
Component:  Utilities |  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 Tom):

 * cc: Tom (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/067.883d51624b3539b3930cbd6734020c82%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27685: Allow autoreloader to use watchman

2017-07-23 Thread Django

Comment (by Tom):

For reference, flask has quite a nice reloading implementation that uses
Watchdog:
https://github.com/pallets/werkzeug/blob/master/werkzeug/_reloader.py

Django obviously cannot bundle watchman as a Watchdog, so it will have to
use a 'dumb' stat based implementation as it's base. While this is the
lowest common denominator it is OK for tiny projects, we could detect when
the number of watched files exceeds X and display a helpful message, like
'install the `watchdog` package for fun and profit'. This seems to be the
easiest way forward to improve the reloader in general, the current
implementation is in great need of some improvements.

I really like Watchman though (https://facebook.github.io/watchman/), a
frontend frameworks that I use utilizes it. It has a socket API and runs
as a service, so if it's available (there is an official API package) we
could add this as a supported reloader.

The reloading code in general is a bit of a black box, but it shouldn't
be. If a re-implementation is on the cards then maybe it could be
pluggable by third party apps that want to do something when files change
during development. One interesting use case could be triggering a page
reload when template files 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/067.9196fd7d91e40635f5fa3d24539c047f%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27685: Allow autoreloader to use watchman

2017-01-05 Thread Django

#27685: Allow autoreloader to use watchman
--+
 Reporter:  Aymeric Augustin  |Owner:  nobody
 Type:  Cleanup/optimization  |   Status:  new
Component:  Utilities |  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 Ed Morley):

 * cc: emorley@… (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/067.e2619abdb1f7c7fbaac7c81ab1002ca4%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27685: Allow autoreloader to use watchman

2017-01-05 Thread Django

#27685: Allow autoreloader to use watchman
--+
 Reporter:  Aymeric Augustin  |Owner:  nobody
 Type:  Cleanup/optimization  |   Status:  new
Component:  Utilities |  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
--+
Description changed by Aymeric Augustin:

Old description:

> I believe we should stop maintaining an autoreloader as soon as possible.
> Django’s autoreloader is annoyingly slow, highly inefficient, moderately
> well designed, and a gigantic pain to maintain. I’m more scared of
> `django.utils.autoreload` than of `django.db.models.related` before it
> was cleaned up.
>
> I wish one day someone will take the time to write a good autoreloading
> dev server based on watchman. This would solve the problem discussed here
> because watchman watches all files in the current directory. The correct
> way to do this is to throw away the current design and start from
> scratch.
>
> Watchman is smart enough to wait until you’ve finished a git operation to
> trigger a reload. Once such polished tech has become available, trying to
> compete with a thread that checks the mtime of all known files every
> second isn’t funny anymore. In fact it’s just sad.
>
> from [https://groups.google.com/d/msg/django-
> developers/HB0S4cF0DO4/YepyGyJZFAAJ django-developers thread]
>
> The [https://groups.google.com/d/topic/django-
> developers/voXNIDdDcpU/discussion Future of the development server's
> auto-reloading thread] has some ideas about this.

New description:

 I believe we should stop maintaining an autoreloader as soon as possible.
 Django’s autoreloader is annoyingly slow, highly inefficient, moderately
 well designed, and a gigantic pain to maintain. I’m more scared of
 `django.utils.autoreload` than of `django.db.models.related` before it was
 cleaned up.

 I wish one day someone will take the time to write a good autoreloading
 dev server based on watchman. This would solve the problem discussed here
 because watchman watches all files in the current directory. The correct
 way to do this is to throw away the current design and start from scratch.

 Watchman is smart enough to wait until you’ve finished a git operation to
 trigger a reload. Once such polished tech has become available, trying to
 compete with a thread that checks the mtime of all known files every
 second isn’t funny anymore. In fact it’s just sad.

 from [https://groups.google.com/d/msg/django-
 developers/HB0S4cF0DO4/YepyGyJZFAAJ django-developers thread]

 The [https://groups.google.com/d/topic/django-
 developers/voXNIDdDcpU/discussion Future of the development server's auto-
 reloading thread] has some ideas about this.

 EDIT -- That description was initially enclosed in a  tag on the
 mailing list. While I stand by the facts, the tone is more ironic that
 what I'd write in a bug report. Different context.

--

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

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

Re: [Django] #27685: Allow autoreloader to use watchman

22 matches

Site Navigation

Mail list logo

Footer information