Skip to content

Commit

Permalink
[scripts] Include only the binding-specific jars and conf directory in
Browse files Browse the repository at this point in the history
the classpath, rather than all of them.
  • Loading branch information
Govind Kamat committed Jun 20, 2015
1 parent be6c4d9 commit 2559ce6
Showing 1 changed file with 6 additions and 7 deletions.
13 changes: 6 additions & 7 deletions bin/ycsb
Original file line number Diff line number Diff line change
Expand Up @@ -91,13 +91,8 @@ def usage():
def find_jars(dir, database):
jars = []
for (dirpath, dirnames, filenames) in os.walk(dir):
if dirpath.endswith(os.path.join(database + "-binding", "conf")):
jars.append(dirpath)
for filename in filenames:
if filename.endswith(".jar") and \
(filename.startswith("core") or \
filename.startswith(database.split("-")[0]) or \
not "binding" in filename):
if filename.endswith(".jar"):
jars.append(os.path.join(dirpath, filename))
return jars

Expand Down Expand Up @@ -134,7 +129,11 @@ def main():
db_classname = DATABASES[args.database]
command = COMMANDS[args.command]["command"]
main_classname = COMMANDS[args.command]["main"]
classpath = os.pathsep.join(find_jars(ycsb_home, args.database))
db_dir = os.path.join(ycsb_home, args.database + "-binding")
cp = [ os.path.join(db_dir, "conf") ]
cp.extend(find_jars(os.path.join(ycsb_home, "lib"), args.database))
cp.extend(find_jars(os.path.join(db_dir, "lib"), args.database))
classpath = os.pathsep.join(cp)
if args.classpath:
classpath = os.pathsep.join([args.classpath, classpath])

Expand Down

0 comments on commit 2559ce6

Please sign in to comment.