> 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 > >