search

brianfrankcooper / YCSB

Yahoo! Cloud Serving Benchmark
Apache License 2.0
4.94k stars 2.24k forks source link

ycsb Python wrapper vs. ycsb.sh #1645

Open lfrancke opened 1 year ago

lfrancke commented 1 year ago

There are two scripts (well, three if you count the .bat) version: ycsb (Python) and ycsb.sh.

The "main" readme refers to the .sh version but some/most (I have only checked a few) of the module readmes refer to the Python script.

Can someone enlighten us as to the history? Both seem to be maintained a bit. This issue https://github.com/brianfrankcooper/YCSB/issues/1030 refers to the Python one as "old" but it has not been removed.

@busbey, can you maybe shed some light here?

Thank you very much!

busbey commented 1 year ago

the python launcher should be considered deprecated. the OS native scripts should be preferred.

we'll eventually remove the python launcher but I don't want to do that while we still have documentation referring to it.

busbey commented 1 year ago

an open question is wether we still need the windows specific bat file given how mature WSL is now. (or that we really ought to be publishing container images)

soenkeliebau commented 1 year ago

We have wrapped ycsb in a (really simple) container for our purposes. Happy to contribute this, but I'd almost feel bad about the PR because its so simple :)

lfrancke commented 1 year ago

Thank you very much for the quick response. The question has been answered and I'd be happy to close the issue but we can also leave it open as a reminder to maybe get rid of the old python wrapper at some point then?