Re: Review Request 47324: Update leveldb to 1.18.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47324/#review145970 --- Closing in favor of [51053](https://reviews.apache.org/r/51053/) - Tomasz Janiszewski On Aug. 12, 2016, 4:58 p.m., Tomasz Janiszewski wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47324/ > --- > > (Updated Aug. 12, 2016, 4:58 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 e5cf7d00921c73e37b4bdaaa43874ab3a1d2f1fe > 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 08e47b6c93be51b0ea86f099f76e9d2a4a6ffd78 > LICENSE 7946d1d46174ff30459a33b3d0b44ab62672af0a > 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: Update leveldb to 1.18.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47324/ --- (Updated Aug. 12, 2016, 4:58 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 (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 e5cf7d00921c73e37b4bdaaa43874ab3a1d2f1fe 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 08e47b6c93be51b0ea86f099f76e9d2a4a6ffd78 LICENSE 7946d1d46174ff30459a33b3d0b44ab62672af0a 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: Update leveldb to 1.18.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47324/ --- (Updated Aug. 12, 2016, 4:58 p.m.) Review request for mesos, Bing Li, Benjamin Mahler, Zhiwei Chen, and haosdent huang. Changes --- Rebase 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 (updated) - 3rdparty/cmake/Versions.cmake e5cf7d00921c73e37b4bdaaa43874ab3a1d2f1fe 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 08e47b6c93be51b0ea86f099f76e9d2a4a6ffd78 LICENSE 7946d1d46174ff30459a33b3d0b44ab62672af0a 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: Update leveldb to 1.18.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47324/#review133415 --- Ship it! Tested on s390x, SLES12SP1. - Bing Li On May 14, 2016, 3:01 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, 3:01 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: Update leveldb to 1.18.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47324/#review133327 --- Ship it! I tested on ppc64le with ssl and libevent enabled, it passed. - Zhiwei Chen On May 14, 2016, 11:01 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, 11:01 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: Update leveldb to 1.18.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47324/#review133278 --- Patch looks great! Reviews applied: [47324] Passed command: export OS='ubuntu:14.04' CONFIGURATION='--verbose' COMPILER='gcc' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker_build.sh - Mesos ReviewBot On May 14, 2016, 3:01 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, 3:01 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: Update leveldb to 1.18.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47324/ --- (Updated May 14, 2016, 3:01 p.m.) Review request for mesos, Bing Li, Benjamin Mahler, Zhiwei Chen, and haosdent huang. Summary (updated) - Update 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 (updated) - 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: Update leveldb.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47324/#review133264 --- Patch looks great! Reviews applied: [47324] Passed command: export OS='ubuntu:14.04' CONFIGURATION='--verbose' COMPILER='gcc' ENVIRONMENT='GLOG_v=1 MESOS_VERBOSE=1'; ./support/docker_build.sh - Mesos ReviewBot On May 14, 2016, 7:33 a.m., Tomasz Janiszewski wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47324/ > --- > > (Updated May 14, 2016, 7:33 a.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 > --- > > Update leveldb. > > > 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: Update leveldb.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47324/#review133263 --- I suggest to change Summary and Description to `Upgrade leveldb to 1.18 to suport s390x.` - haosdent huang On May 14, 2016, 7:33 a.m., Tomasz Janiszewski wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47324/ > --- > > (Updated May 14, 2016, 7:33 a.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 > --- > > Update leveldb. > > > 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: Update leveldb.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47324/ --- (Updated May 14, 2016, 7:33 a.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 --- Update leveldb. Diffs (updated) - 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: Update leveldb.
> On May 13, 2016, 4:38 a.m., Mesos ReviewBot wrote: > > Bad patch! > > > > Reviews applied: [47324] > > > > Failed command: ./support/apply-review.sh -n -r 47324 > > > > Error: > > 2016-05-13 04:38:10 URL:https://reviews.apache.org/r/47324/diff/raw/ > > [4739/4739] -> "47324.patch" [1] > > error: missing binary patch data for '3rdparty/leveldb-1.18.tar.gz' > > error: binary patch does not apply to '3rdparty/leveldb-1.18.tar.gz' > > error: 3rdparty/leveldb-1.18.tar.gz: patch does not apply > > > > Full log: https://builds.apache.org/job/mesos-reviewbot/13038/console > > Zhiwei Chen wrote: > The rbtools does not support binary patch, please apply this patch to > your rbtools. https://reviews.reviewboard.org/r/7571/diff/1#index_header > > Or use git diff --binary to generate a patch file and upload it through > reviewboard Web UI. > > I will help you test on ppc64le when you fix this. > > haosdent huang wrote: > Thank you so much for your kindly help. @zhiwei. Sorry to forgot remind > @janisz before. I think we may document this at the `submitting a patch`. :-) Thanks for help! I think documentation for patching binary files is not necessary, it's not performed often and hopefully fix will be merged soon. - Tomasz --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47324/#review133030 --- On May 13, 2016, 7:55 p.m., Tomasz Janiszewski wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47324/ > --- > > (Updated May 13, 2016, 7:55 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 > --- > > Update leveldb. > > > Diffs > - > > 3rdparty/cmake/Versions.cmake 86c51edb3aa2daf6451459aaf18278f09b91b000 > 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: Update leveldb.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47324/ --- (Updated May 13, 2016, 7:55 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 (updated) --- Update leveldb. Diffs (updated) - 3rdparty/cmake/Versions.cmake 86c51edb3aa2daf6451459aaf18278f09b91b000 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: Update leveldb.
> On May 13, 2016, 4:38 a.m., Mesos ReviewBot wrote: > > Bad patch! > > > > Reviews applied: [47324] > > > > Failed command: ./support/apply-review.sh -n -r 47324 > > > > Error: > > 2016-05-13 04:38:10 URL:https://reviews.apache.org/r/47324/diff/raw/ > > [4739/4739] -> "47324.patch" [1] > > error: missing binary patch data for '3rdparty/leveldb-1.18.tar.gz' > > error: binary patch does not apply to '3rdparty/leveldb-1.18.tar.gz' > > error: 3rdparty/leveldb-1.18.tar.gz: patch does not apply > > > > Full log: https://builds.apache.org/job/mesos-reviewbot/13038/console > > Zhiwei Chen wrote: > The rbtools does not support binary patch, please apply this patch to > your rbtools. https://reviews.reviewboard.org/r/7571/diff/1#index_header > > Or use git diff --binary to generate a patch file and upload it through > reviewboard Web UI. > > I will help you test on ppc64le when you fix this. Thank you so much for your kindly help. @zhiwei. Sorry to forgot remind @janisz before. I think we may document this at the `submitting a patch`. :-) - haosdent --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47324/#review133030 --- On May 12, 2016, 8:46 p.m., Tomasz Janiszewski wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47324/ > --- > > (Updated May 12, 2016, 8: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 > --- > > Update leveldb to 1.18. > Remove patch that is already included in latest leveldb release. > > > Diffs > - > > 3rdparty/cmake/Versions.cmake 86c51edb3aa2daf6451459aaf18278f09b91b000 > 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: Update leveldb.
> On May 13, 2016, 12:38 p.m., Mesos ReviewBot wrote: > > Bad patch! > > > > Reviews applied: [47324] > > > > Failed command: ./support/apply-review.sh -n -r 47324 > > > > Error: > > 2016-05-13 04:38:10 URL:https://reviews.apache.org/r/47324/diff/raw/ > > [4739/4739] -> "47324.patch" [1] > > error: missing binary patch data for '3rdparty/leveldb-1.18.tar.gz' > > error: binary patch does not apply to '3rdparty/leveldb-1.18.tar.gz' > > error: 3rdparty/leveldb-1.18.tar.gz: patch does not apply > > > > Full log: https://builds.apache.org/job/mesos-reviewbot/13038/console The rbtools does not support binary patch, please apply this patch to your rbtools. https://reviews.reviewboard.org/r/7571/diff/1#index_header Or use git diff --binary to generate a patch file and upload it through reviewboard Web UI. I will help you test on ppc64le when you fix this. - Zhiwei --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47324/#review133030 --- On May 13, 2016, 4:46 a.m., Tomasz Janiszewski wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47324/ > --- > > (Updated May 13, 2016, 4:46 a.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 > --- > > Update leveldb to 1.18. > Remove patch that is already included in latest leveldb release. > > > Diffs > - > > 3rdparty/cmake/Versions.cmake 86c51edb3aa2daf6451459aaf18278f09b91b000 > 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: Update leveldb.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47324/#review133030 --- Bad patch! Reviews applied: [47324] Failed command: ./support/apply-review.sh -n -r 47324 Error: 2016-05-13 04:38:10 URL:https://reviews.apache.org/r/47324/diff/raw/ [4739/4739] -> "47324.patch" [1] error: missing binary patch data for '3rdparty/leveldb-1.18.tar.gz' error: binary patch does not apply to '3rdparty/leveldb-1.18.tar.gz' error: 3rdparty/leveldb-1.18.tar.gz: patch does not apply Full log: https://builds.apache.org/job/mesos-reviewbot/13038/console - Mesos ReviewBot On May 12, 2016, 8:46 p.m., Tomasz Janiszewski wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/47324/ > --- > > (Updated May 12, 2016, 8: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 > --- > > Update leveldb to 1.18. > Remove patch that is already included in latest leveldb release. > > > Diffs > - > > 3rdparty/cmake/Versions.cmake 86c51edb3aa2daf6451459aaf18278f09b91b000 > 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 > >
Review Request 47324: Update leveldb.
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/47324/ --- Review request for mesos, Bing Li, Ben Mahler, Zhiwei Chen, and haosdent huang. Bugs: MESOS-970 https://issues.apache.org/jira/browse/MESOS-970 Repository: mesos Description --- Update leveldb to 1.18. Remove patch that is already included in latest leveldb release. Diffs - 3rdparty/cmake/Versions.cmake 86c51edb3aa2daf6451459aaf18278f09b91b000 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
