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


---