> Not necessarily. Why not provide both, a prebuilt benchmark and an ability > to run from code? >
Sure, we can do that. My point was to cover the most frequently asked use case which is the ability to run compiled benchmarks. > Also, please keep in mind that any binary we include into release, must > either come form maven or be built as part of the release itself. Is there any reason why we don’t host ignite-yardstick on maven at the moment? — Denis > On Dec 19, 2016, at 9:55 AM, Dmitriy Setrakyan <dsetrak...@apache.org> wrote: > > On Mon, Dec 19, 2016 at 9:46 AM, Denis Magda <dma...@apache.org> wrote: > >> Then it will not much differ from the flow the users have to follow >> presently. The only difference is that the sources and instructions will be >> available in an Apache Ignite release as well. But the users will still >> need to build the benchmarks manually, right? They already can do the same >> referring to Yardstick. >> > > Not necessarily. Why not provide both, a prebuilt benchmark and an ability > to run from code? > > Also, please keep in mind that any binary we include into release, must > either come form maven or be built as part of the release itself. > > >> >> — >> Denis >> >>> On Dec 19, 2016, at 8:53 AM, Dmitriy Setrakyan <dsetrak...@apache.org> >> wrote: >>> >>> I would be against using libs/optional or libs/ folder for anything >>> benchmark related. I am also against adding any yardstick libraries >> without >>> providing code. >>> >>> In my view, if the community wants to include benchmarks in releases, >> then >>> we should add a "benchmarks" folder, which provides everything benchmark >>> related, from code to all the dependent libraries, and documentation >>> instructions. >>> >>> D. >>> >>> On Mon, Dec 19, 2016 at 8:11 AM, Denis Magda <dma...@apache.org> wrote: >>> >>>> Actually, “libs/optional” is already a kind of extra for me. Why do we >>>> need this new folder if “libs/optional” semantic works well? >>>> >>>> Is there anyone else who is concerned about “libs/optional”? If there’re >>>> not, I would agree on this and get down to the implementation. >>>> >>>> — >>>> Denis >>>> >>>>> On Dec 19, 2016, at 1:10 AM, Sergey Kozlov <skoz...@gridgain.com> >> wrote: >>>>> >>>>> Hi >>>>> >>>>> What's about to introduce the new root folder called 'extras' with >>>>> subfolder 'ignite-yardstick' and put there yardstick binaries? >>>>> >>>>> >>>>> On Sun, Dec 18, 2016 at 10:02 PM, Denis Magda <dma...@apache.org> >> wrote: >>>>> >>>>>> Dmitriy, >>>>>> >>>>>> Please have a look at IGNITE-4212 description ( >>>> https://issues.apache.org/ >>>>>> jira/browse/IGNITE-4212). >>>>>> >>>>>> The whole purpose of the ticket is to automate benchmarks execution >> for >>>>>> the end user for a specific Ignite release. Now he/she needs to go >>>> through >>>>>> a number of steps like build, configure, run strictly following >> lengthy >>>>>> Yardstick guidance. >>>>>> >>>>>> Ideally, once a specific release is downloaded it should be possible >> to >>>>>> run a concrete benchmark with a ready-to-use script. The script needs >>>>>> benchmarks' lib which makes sense to put under “libs/optional” folder. >>>>>> >>>>>> If someone wants to modify the source of an existed benchmark or add a >>>> new >>>>>> one then he/she needs to follow existed Yardstick guidance. So, no >> need >>>> to >>>>>> release benchmarks’s sources as a part of Ignite release. >>>>>> >>>>>> — >>>>>> Denis >>>>>> >>>>>>> On Dec 18, 2016, at 7:08 AM, Dmitriy Setrakyan < >> dsetrak...@apache.org> >>>>>> wrote: >>>>>>> >>>>>>> On Sun, Dec 18, 2016 at 2:53 AM, Oleg Ostanin <oosta...@gridgain.com >>> >>>>>> wrote: >>>>>>> >>>>>>>> Dmitriy, ignite-yardstick allows user to run plenty of useful >>>> Yardstick >>>>>>>> benchmarks, which can be used to check Ignite performance. >>>>>>>> >>>>>>> >>>>>>> In that case, why would it be under the "libs" folder at all? Do we >>>>>> really >>>>>>> need to include benchmarks into Ignite? If yes, then I would create a >>>>>>> benchmarks folder under "examples" and add all the benchmarks there. >>>>>>> >>>>>>> >>>>>>>> >>>>>>>> On Fri, Dec 16, 2016 at 11:49 PM, Dmitriy Setrakyan < >>>>>> dsetrak...@apache.org >>>>>>>>> >>>>>>>> wrote: >>>>>>>> >>>>>>>>> Oleg, what does ignite-yardstick module do? >>>>>>>>> >>>>>>>>> On Fri, Dec 16, 2016 at 12:37 AM, Oleg Ostanin < >>>> oosta...@gridgain.com> >>>>>>>>> wrote: >>>>>>>>> >>>>>>>>>> Hello Igniters! >>>>>>>>>> I'm working on ticket IGNITE-4212 "Ignite Benchmarking >>>> Simplification >>>>>>>> and >>>>>>>>>> Automation" and I'd like to ask your opinion about >> ignite-yardstick: >>>>>>>>> where >>>>>>>>>> do you think is the most appropriate place to put a compiled >>>>>>>>>> ignite-yardstick module in the apache-ignite binary assembly? We >> can >>>>>>>> put >>>>>>>>> it >>>>>>>>>> in the libs/optional along with an others optional libraries, or >> we >>>>>> can >>>>>>>>>> create a new directory named "tools" in the root directory and put >>>>>>>>>> "ignite-yardstick" in it, or we can find another solution. >>>>>>>>>> >>>>>>>>>> Best regards >>>>>>>>>> Oleg >>>>>>>>>> >>>>>>>>> >>>>>>>> >>>>>> >>>>>> >>>>> >>>>> >>>>> -- >>>>> Sergey Kozlov >>>>> GridGain Systems >>>>> www.gridgain.com >>>> >>>> >> >>