You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@bigtop.apache.org by youngwookim <gi...@git.apache.org> on 2017/12/08 03:41:34 UTC
[GitHub] bigtop pull request #316: BIGTOP-2957: Upgrade YCSB to 0.12.0
GitHub user youngwookim opened a pull request:
https://github.com/apache/bigtop/pull/316
BIGTOP-2957: Upgrade YCSB to 0.12.0
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/youngwookim/bigtop BIGTOP-2957
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/bigtop/pull/316.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #316
----
commit 56b553058eafc8e09ff93703a65eab163ada6c60
Author: Youngwoo Kim <yw...@apache.org>
Date: 2017-12-08T02:46:29Z
BIGTOP-2957: Upgrade YCSB to 0.12.0
----
---
[GitHub] bigtop pull request #316: BIGTOP-2957: Upgrade YCSB to 0.12.0
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/bigtop/pull/316
---
[GitHub] bigtop pull request #316: BIGTOP-2957: Upgrade YCSB to 0.12.0
Posted by evans-ye <gi...@git.apache.org>.
Github user evans-ye commented on a diff in the pull request:
https://github.com/apache/bigtop/pull/316#discussion_r157007619
--- Diff: bigtop-packages/src/common/ycsb/install_ycsb.sh ---
@@ -74,6 +74,25 @@ for var in PREFIX BUILD_DIR ; do
done
LIB_DIR=${LIB_DIR:-/usr/lib/ycsb}
+BIN_DIR=${BIN_DIR:-$PREFIX/usr/bin}
install -d -m 0755 $PREFIX/$LIB_DIR
+install -d -m 0755 ${BIN_DIR}
+
(cd ${BUILD_DIR} && tar -cf - .)|(cd $PREFIX/${LIB_DIR} && tar -xf -)
+rm -rf $PREFIX/$LIB_DIR/bin/*.bat
+
+wrapper=$BIN_DIR/ycsb
+cat >>$wrapper <<EOF
+#!/bin/sh
--- End diff --
I remember once before we've agreed to use /bin/bash ;)
---