On Mon, 18 Sep 2017 at 22:30 Yaniv Kaul <ykaul@redhat.com> wrote:

I see few advantages and they all drawn from the assumption the initial patchset is always some sort of work in progress in really most of the cases:

1. It doesn't invoke automation and waste resources. First the developer should run it and be passed the checkstyle/pep/other errors locally.

2. Default reviewers feature hopefully will put the reviewers in place automatically so it not hidden.

3. After the patch is bit more mature it is worth publishing to get more reviews. Half baked or controversial patches may be costly to review. After they are published the reviewer can expect higher quality and can estimate better the effort in review

IMHO we don't use this practice enough.