Re: Review Request 47324: Upgrade leveldb to 1.18.
> On May 14, 2016, 10:11 a.m., haosdent huang wrote: > > I suggest to change Summary and Description to `Upgrade leveldb to 1.18 to > > suport s390x.` > > Tomasz Janiszewski wrote: > Fixed. Actually I need it to fix replica log provlems on ARM Cool! And thhe description have a 72 characters limit, otherwise could not pass jenkins. Please update to ``` Leveldb in modern version is required to support s390x. It's also required to replace default byte-wise comparator with varint comparator in `src/log/leveldb.cpp`. ``` - haosdent --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47324/#review133263 --- On May 14, 2016, 1:46 p.m., Tomasz Janiszewski wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47324/ > --- > > (Updated May 14, 2016, 1:46 p.m.) > > > Review request for mesos, Bing Li, Benjamin Mahler, Zhiwei Chen, and haosdent > huang. > > > Bugs: MESOS-970 > https://issues.apache.org/jira/browse/MESOS-970 > > > Repository: mesos > > > Description > --- > > Leveldb in modern version is required to support s390x. > It's also required to replace default byte-wise comparator with varint > comparator in `src/log/leveldb.cpp` > > > Diffs > - > > 3rdparty/cmake/Versions.cmake 86c51edb3aa2daf6451459aaf18278f09b91b000 > 3rdparty/leveldb-1.18.patch PRE-CREATION > 3rdparty/leveldb-1.18.tar.gz PRE-CREATION > 3rdparty/leveldb-1.4.patch b899f0141d633b1ffb2321e573395256fc893b16 > 3rdparty/leveldb-1.4.tar.gz 2ddbc0c2e02054406ff0ea43ddc10d14979de8d8 > 3rdparty/versions.am 7dcd6bf914de3213755ec9d4e701a190750424e9 > LICENSE eb39f6d69a165f59c00e8bb0ba9e15be8c958a5b > src/python/native_common/ext_modules.py.in > 2d4a45efa224b32f80ace4542a00062c5ccb06d5 > > Diff: https://reviews.apache.org/r/47324/diff/ > > > Testing > --- > > Tested on Ubuntu amd_64 need to test on PPC > > > Thanks, > > Tomasz Janiszewski > >
Re: Review Request 47324: Upgrade leveldb to 1.18.
> On May 14, 2016, 10:11 a.m., haosdent huang wrote: > > I suggest to change Summary and Description to `Upgrade leveldb to 1.18 to > > suport s390x.` Fixed. Actually I need it to fix replica log provlems on ARM - Tomasz --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47324/#review133263 --- On May 14, 2016, 1:46 p.m., Tomasz Janiszewski wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47324/ > --- > > (Updated May 14, 2016, 1:46 p.m.) > > > Review request for mesos, Bing Li, Benjamin Mahler, Zhiwei Chen, and haosdent > huang. > > > Bugs: MESOS-970 > https://issues.apache.org/jira/browse/MESOS-970 > > > Repository: mesos > > > Description > --- > > Leveldb in modern version is required to support s390x. > It's also required to replace default byte-wise comparator with varint > comparator in `src/log/leveldb.cpp` > > > Diffs > - > > 3rdparty/cmake/Versions.cmake 86c51edb3aa2daf6451459aaf18278f09b91b000 > 3rdparty/leveldb-1.18.patch PRE-CREATION > 3rdparty/leveldb-1.18.tar.gz PRE-CREATION > 3rdparty/leveldb-1.4.patch b899f0141d633b1ffb2321e573395256fc893b16 > 3rdparty/leveldb-1.4.tar.gz 2ddbc0c2e02054406ff0ea43ddc10d14979de8d8 > 3rdparty/versions.am 7dcd6bf914de3213755ec9d4e701a190750424e9 > LICENSE eb39f6d69a165f59c00e8bb0ba9e15be8c958a5b > src/python/native_common/ext_modules.py.in > 2d4a45efa224b32f80ace4542a00062c5ccb06d5 > > Diff: https://reviews.apache.org/r/47324/diff/ > > > Testing > --- > > Tested on Ubuntu amd_64 need to test on PPC > > > Thanks, > > Tomasz Janiszewski > >
Re: Review Request 47324: Upgrade leveldb to 1.18.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47324/ --- (Updated May 14, 2016, 1:46 p.m.) Review request for mesos, Bing Li, Benjamin Mahler, Zhiwei Chen, and haosdent huang. Changes --- Change summary and description Summary (updated) - Upgrade leveldb to 1.18. Bugs: MESOS-970 https://issues.apache.org/jira/browse/MESOS-970 Repository: mesos Description (updated) --- Leveldb in modern version is required to support s390x. It's also required to replace default byte-wise comparator with varint comparator in `src/log/leveldb.cpp` Diffs - 3rdparty/cmake/Versions.cmake 86c51edb3aa2daf6451459aaf18278f09b91b000 3rdparty/leveldb-1.18.patch PRE-CREATION 3rdparty/leveldb-1.18.tar.gz PRE-CREATION 3rdparty/leveldb-1.4.patch b899f0141d633b1ffb2321e573395256fc893b16 3rdparty/leveldb-1.4.tar.gz 2ddbc0c2e02054406ff0ea43ddc10d14979de8d8 3rdparty/versions.am 7dcd6bf914de3213755ec9d4e701a190750424e9 LICENSE eb39f6d69a165f59c00e8bb0ba9e15be8c958a5b src/python/native_common/ext_modules.py.in 2d4a45efa224b32f80ace4542a00062c5ccb06d5 Diff: https://reviews.apache.org/r/47324/diff/ Testing --- Tested on Ubuntu amd_64 need to test on PPC Thanks, Tomasz Janiszewski
