Re: Review Request 42068: Porting Mesos on ppc64le.
> On Jan. 17, 2016, 10:37 a.m., Artem Harutyunyan wrote: > > We avoid making changes in 3rdparty and the Mesos code within the same > > patch. Could you please break this patch into 2? > > Qian Zhang wrote: > Did you mean splitting the commit between mesos and libprocess? I saw we > have a check in the pre-commit (see the link below) which ensures we should > not have the code changes for Mesos and libprocess in the same patch, that's > why I have two commit: this one and https://reviews.apache.org/r/42069/, > please let me know if you have further comments. > https://github.com/apache/mesos/blob/0.26.0/support/hooks/pre-commit#L29 There's isn't a pre-commit hook for everything in 3rdparty, but it would be good to separate: 3rdparty/leveldb.patch 3rdparty/zookeeper-3.4.5.patch --- src/linux/fs.cpp - Joseph --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42068/#review114910 --- On Jan. 17, 2016, 5:51 p.m., Qian Zhang wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42068/ > --- > > (Updated Jan. 17, 2016, 5:51 p.m.) > > > Review request for mesos and Ben Mahler. > > > Bugs: MESOS-4312 > https://issues.apache.org/jira/browse/MESOS-4312 > > > Repository: mesos > > > Description > --- > > Porting Mesos on ppc64le. > > > Diffs > - > > 3rdparty/leveldb.patch ad8c19b9caa856ff85978ba832d48df11b3a83f0 > 3rdparty/zookeeper-3.4.5.patch 3ca180d0c81f5de521ada7fb6c1c248a871ab2da > src/linux/fs.cpp bfcf97186cd1b0696a9537c4a332083def6b462e > > Diff: https://reviews.apache.org/r/42068/diff/ > > > Testing > --- > > make check (OSX 10.10.5, Ubuntu 14.04.3 LTS ppc64le) > > > Thanks, > > Qian Zhang > >
Re: Review Request 42068: Porting Mesos on ppc64le.
> On Jan. 18, 2016, 2:37 a.m., Artem Harutyunyan wrote: > > We avoid making changes in 3rdparty and the Mesos code within the same > > patch. Could you please break this patch into 2? Did you mean splitting the commit between mesos and libprocess? I saw we have a check in the pre-commit (see the link below) which ensures we should not have the code changes for Mesos and libprocess in the same patch, that's why I have two commit: this one and https://reviews.apache.org/r/42069/, please let me know if you have further comments. https://github.com/apache/mesos/blob/0.26.0/support/hooks/pre-commit#L29 - Qian --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42068/#review114910 --- On Jan. 18, 2016, 9:51 a.m., Qian Zhang wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42068/ > --- > > (Updated Jan. 18, 2016, 9:51 a.m.) > > > Review request for mesos and Ben Mahler. > > > Bugs: MESOS-4312 > https://issues.apache.org/jira/browse/MESOS-4312 > > > Repository: mesos > > > Description > --- > > Porting Mesos on ppc64le. > > > Diffs > - > > 3rdparty/leveldb.patch ad8c19b9caa856ff85978ba832d48df11b3a83f0 > 3rdparty/zookeeper-3.4.5.patch 3ca180d0c81f5de521ada7fb6c1c248a871ab2da > src/linux/fs.cpp bfcf97186cd1b0696a9537c4a332083def6b462e > > Diff: https://reviews.apache.org/r/42068/diff/ > > > Testing > --- > > make check (OSX 10.10.5, Ubuntu 14.04.3 LTS ppc64le) > > > Thanks, > > Qian Zhang > >
Re: Review Request 42068: Porting Mesos on ppc64le.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42068/#review114927 --- Ship it! Ship It! - Zhiwei Chen On Jan. 18, 2016, 9:51 a.m., Qian Zhang wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42068/ > --- > > (Updated Jan. 18, 2016, 9:51 a.m.) > > > Review request for mesos and Ben Mahler. > > > Bugs: MESOS-4312 > https://issues.apache.org/jira/browse/MESOS-4312 > > > Repository: mesos > > > Description > --- > > Porting Mesos on ppc64le. > > > Diffs > - > > 3rdparty/leveldb.patch ad8c19b9caa856ff85978ba832d48df11b3a83f0 > 3rdparty/zookeeper-3.4.5.patch 3ca180d0c81f5de521ada7fb6c1c248a871ab2da > src/linux/fs.cpp bfcf97186cd1b0696a9537c4a332083def6b462e > > Diff: https://reviews.apache.org/r/42068/diff/ > > > Testing > --- > > make check (OSX 10.10.5, Ubuntu 14.04.3 LTS ppc64le) > > > Thanks, > > Qian Zhang > >
Re: Review Request 42068: Porting Mesos on ppc64le.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42068/#review114910 --- We avoid making changes in 3rdparty and the Mesos code within the same patch. Could you please break this patch into 2? - Artem Harutyunyan On Jan. 8, 2016, 3:02 a.m., Qian Zhang wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42068/ > --- > > (Updated Jan. 8, 2016, 3:02 a.m.) > > > Review request for mesos. > > > Bugs: MESOS-4312 > https://issues.apache.org/jira/browse/MESOS-4312 > > > Repository: mesos > > > Description > --- > > Porting Mesos on ppc64le. > > > Diffs > - > > 3rdparty/leveldb.patch ad8c19b9caa856ff85978ba832d48df11b3a83f0 > 3rdparty/zookeeper-3.4.5.patch 3ca180d0c81f5de521ada7fb6c1c248a871ab2da > src/linux/fs.cpp bfcf97186cd1b0696a9537c4a332083def6b462e > > Diff: https://reviews.apache.org/r/42068/diff/ > > > Testing > --- > > make check (OSX 10.10.5, Ubuntu 14.04.3 LTS ppc64le) > > > Thanks, > > Qian Zhang > >
Re: Review Request 42068: Porting Mesos on ppc64le.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42068/#review113460 --- Bad review! Reviews applied: [] Error: No reviewers specified. Please find a reviewer by asking on JIRA or the mailing list. - Mesos ReviewBot On Jan. 8, 2016, 11:02 a.m., Qian Zhang wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/42068/ > --- > > (Updated Jan. 8, 2016, 11:02 a.m.) > > > Review request for mesos. > > > Bugs: MESOS-4312 > https://issues.apache.org/jira/browse/MESOS-4312 > > > Repository: mesos > > > Description > --- > > Porting Mesos on ppc64le. > > > Diffs > - > > 3rdparty/leveldb.patch ad8c19b9caa856ff85978ba832d48df11b3a83f0 > 3rdparty/zookeeper-3.4.5.patch 3ca180d0c81f5de521ada7fb6c1c248a871ab2da > src/linux/fs.cpp bfcf97186cd1b0696a9537c4a332083def6b462e > > Diff: https://reviews.apache.org/r/42068/diff/ > > > Testing > --- > > make check (OSX 10.10.5, Ubuntu 14.04.3 LTS ppc64le) > > > Thanks, > > Qian Zhang > >
Review Request 42068: Porting Mesos on ppc64le.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42068/ --- Review request for mesos. Bugs: MESOS-4312 https://issues.apache.org/jira/browse/MESOS-4312 Repository: mesos Description --- Porting Mesos on ppc64le. Diffs - 3rdparty/leveldb.patch ad8c19b9caa856ff85978ba832d48df11b3a83f0 3rdparty/zookeeper-3.4.5.patch 3ca180d0c81f5de521ada7fb6c1c248a871ab2da src/linux/fs.cpp bfcf97186cd1b0696a9537c4a332083def6b462e Diff: https://reviews.apache.org/r/42068/diff/ Testing --- make check (OSX 10.10.5, Ubuntu 14.04.3 LTS ppc64le) Thanks, Qian Zhang