> On Jan. 20, 2017, 6:24 a.m., haosdent huang wrote: > > support/apply-reviews.py, lines 120-121 > > <https://reviews.apache.org/r/55732/diff/1/?file=1609817#file1609817line120> > > > > Is this change necessary?
>From what I observed, in `--3way` mode, git apply seems to drop the `.patch` >file by itself, so this line would error out becasue the file does not exist. Given that the goal here is to ensure the file is deleted, I think the change is okay. - Zhitao ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/55732/#review162389 ----------------------------------------------------------- On Jan. 19, 2017, 10:49 p.m., Zhitao Li wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/55732/ > ----------------------------------------------------------- > > (Updated Jan. 19, 2017, 10:49 p.m.) > > > Review request for mesos, Jason Lai, Joseph Wu, and Michael Park. > > > Bugs: MESOS-4119 > https://issues.apache.org/jira/browse/MESOS-4119 > > > Repository: mesos > > > Description > ------- > > This can help on some patches which includes conflicts that can be > resolved by 3 way merge. > > > Diffs > ----- > > support/apply-reviews.py c77b4c2b2d7a3d5e74e225403e71a84e23a9a1e7 > > Diff: https://reviews.apache.org/r/55732/diff/ > > > Testing > ------- > > With this fix, I was able to apply the chain in r/52534, which has a conflict > otherwise. > > Bash log: > ``` > $ python ./support/apply-reviews.py -r 52534 -c > 2017-01-19 14:45:26 URL:https://reviews.apache.org/r/51027/diff/raw/ > [9830/9830] -> "51027.patch" [1] > [3way fba53e2] Track allocation candidates to bound allocator. > Author: Jacob Janco <[email protected]> > 2 files changed, 97 insertions(+), 54 deletions(-) > 2017-01-19 14:45:28 URL:https://reviews.apache.org/r/52534/diff/raw/ > [2950/2950] -> "52534.patch" [1] > error: patch failed: src/master/allocator/mesos/hierarchical.hpp:224 > Falling back to three-way merge... > Applied patch to 'src/master/allocator/mesos/hierarchical.hpp' cleanly. > [3way 261018a] Dispatch filter expiration twice. > Author: Jacob Janco <[email protected]> > 2 files changed, 32 insertions(+), 8 deletions(-) > ``` > > > Thanks, > > Zhitao Li > >
