[infinispan-dev] RadargunTPCC benchmark review

Begin forwarded message: > From: Sebastiano Peluso <peluso(a)gsd.inesc-id.pt&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 14 October 2011 10:33:36 GMT+01:00 > To: Mircea Markus <mircea.markus(a)jboss.com&gt; > Cc: Paolo Romano <romano(a)inesc-id.pt&gt;, Manik Surtani <msurtani(a)redhat.com&gt; > > Hi Mircea, > > OK, don't worry! :-) > > Cheers > > Sebastiano > > Il 13/10/11 19:31, Mircea Markus ha scritto: >> Hi Sebastiano, right now I'm busy remaking locking/transaction support in 5.1. This should keep me busy for the next 3 weeks or so - after that the plan is to look at tpcc, integrate it and produce performance numbers to go with it. >> Sorry for not taking this sooner. >> Cheers, >> Mircea >> On 11 Oct 2011, at 09:49, Sebastiano Peluso wrote: >>> Hi Mircea, >>> >>> have you had the opportunity to take a look at the Radargun's tpcc branch in https://github.com/sebastianopeluso/radargun/tree/tpcc ? >>> >>> Can I issue anyway a pull request in Github? >>> >>> Thank you. >>> >>> Cheers >>> >>> Sebastiano >>> >>> Il 29/08/11 11:49, Sebastiano Peluso ha scritto: >>>> Hi Mircea, >>>> >>>> I've finalized the implementation of TPC-C code (New Order, Payment and Order Status transactions) within Radargun. >>>> You can find the result as Radargun's tpcc branch at this link: >>>> https://github.com/sebastianopeluso/radargun/tree/tpcc >>>> >>>> Can you take a look at the code? I wait for your feedback before issuing a pull request in Github. >>>> >>>> Thank you. >>>> >>>> Cheers, >>>> >>>> Sebastiano >>>> >>>> Il 12/07/11 11:20, Sebastiano Peluso ha scritto: >>>>> Hi Mircea, >>>>> >>>>> I've just forked Radargun's master version and I will start the integration (following your observations/comments in the last mail) in the next week . I hope to finalize the work by the end of July. >>>>> When I will have finished I will issue a pull request. >>>>> >>>>> Thank you for your helpfulness. >>>>> >>>>> Sebastiano >>>>> >>>>> >>>>> Il 11/07/11 13:28, Mircea Markus ha scritto: >>>>>> Thanks Sebastiano. Once you are finished, can you please issue a pull request for Radargun and we can take it from there? Please let me know if there'a anything I can help you with, e.g. github integration etc. >>>>>> >>>>>> On 8 Jul 2011, at 15:31, Sebastiano Peluso wrote: >>>>>> >>>>>>> Hi Mircea, >>>>>>> >>>>>>> thank you for your reply. My answers to your questions/comments are inline. >>>>>>> >>>>>>> Il 08/07/11 11:33, Mircea Markus ha scritto: >>>>>>>> Hi Sebastiano, >>>>>>>> >>>>>>>> Very interesting stuff! >>>>>>>> I think this is a nice approach of benchmarking transactions with a controlled degree contention. This is a feature currently missing in Radargun, but very important especially considering the stuff you guys are doing and the locking improvements we have scheduled for 5.1 release. >>>>>>>> Do you consider contributing this back to Radargun? >>>>>>> YES. >>>>>>>> >>>>>>>> Here are some observations I have after looking at the code: >>>>>>>> - The code should not be in the PutGetStressor, as this is used for benchmarking a web-session-like access pattern which we don't want to replace. It might make more sense to be placed in a stage of its own, e.g. org.radargun.stages.TpccBenchmarkStage >>>>>>> Ok. Your observation is correct. In this way the code is more modular and we can have both the web session and the tpcc benchmarks. >>>>>>>> - "isPrimary" argument of TPCCTerminal.choiceTransaction is true if the node is a jgroups coordinator. This has some limitations: >>>>>>>> - other data grid providers than Infinispan might not have an "coordinator" concept, and this might be difficult to infer >>>>>>>> - ATM only the coordinator does writes (on multiple threads). IMO this is too restricting as it doesn't induce deadlock situations that might appear when data is written by multiple nodes. Wouldn't it make more sense to spread the writes over all the nodes? In order to overcome the dependency on jgroup's coordinator, you can use instead the AbstractDistStage.slaveIndex. This is a Radargun-level view of the benchmark cluster, independent of the benchmarked product >>>>>>> This is partially correct. The "isPrimary" condition is evaluated only if the Primary-Backup replication mechanism is enabled because the code has been written in order to test this replication mechanism too. In this case only the primary node can perform write transactions. >>>>>>> But if 2-Phase-Commit is enabled (lines 72-83 in TPCCTerminal), transactions (read and write) are spread over all nodes. This is always the case, because we don't have Primary-Backup in official Infinispan releases. >>>>>>>> - PutGetStressor compares Strings based on the == operator, which is incorrect. e.g. e.getClass().getName() == "org.infinispan.CacheException" >>>>>>> Sorry, this is a mistake :-) . >>>>>>>> - As an design suggestion, I think it would make sense to have an TpccTransaction interface with an executeTransaction(CacheWrapper cacheWrapper) methods, with specialized implementations: NewOrderTransaction, PayementTransaction, OrderStatusTransaction. Basically to move all the code TPCCTerminal.executeTransaction into these three classes. Such an approach would reduce coupling between the transactions and most importantly would offer a simple way of adding new tpcc transactions to the system, by implementing the TpccTransaction interface. >>>>>>> I agree. I can create one class for each tpcc transaction and assign each "case" block (referring to the current implementation of the executeTransaction method) to the corresponding class. >>>>>>>> - class javadoc are the onese generater by IDEA >>>>>>>> - adding .idea directory in github might cause troubles for people using IDEA and forking your repo; I think the dir should be removed >>>>>>>> >>>>>>>> >>>>>>>> Another observation is about the way the project is managed in github: it is not a fork of Infinispan, but it was downloaded and committed as a totally different project. This means that all the nice review and merge feature that github and git offer cannot be used, but the merge needs to be made "by hand", comparing the source trees etc. I think it would make more sense to fork[1] from radargun's master[2] and apply all these nice feature on a fork. What do you think? >>>>>>> Ok. I'll remove .idea directory. >>>>>>> In addition, I'll insert tpcc implementation in a forked version of radargun's master in order to benefit from merge features provided by git/github. >>>>>>>> >>>>>>>> Cheers, >>>>>>>> Mircea >>>>>>> >>>>>>> Thank you again. >>>>>>> >>>>>>> Cheers, >>>>>>> >>>>>>> Sebastiano >>>>>>>> >>>>>>>> [1] http://help.github.com/fork-a-repo/ >>>>>>>> [2] https://github.com/radargun/radargun >>>>>>>> >>>>>>>> >>>>>>>> On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: >>>>>>>>> Hi all, >>>>>>>>> >>>>>>>>> I have added a new subsection >>>>>>>>> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >>>>>>>>> "Shared software resources" section on the Cloud-TM Wiki, in order to >>>>>>>>> share information about the RadargunTPCC benchmark. >>>>>>>>> >>>>>>>>> RadargunTPCC benchmark is an implementation of the Transaction Profiles >>>>>>>>> defined in the TPC Benchmark C specification [1] within Radargun >>>>>>>>> Benchamark [2], and it has been designed with the purpose of executing >>>>>>>>> the TPC-C benchmark against Infinispan. >>>>>>>>> >>>>>>>>> The source code is available at this link: >>>>>>>>> https://github.com/cloudtm/RadargunTPCC >>>>>>>>> >>>>>>>>> Regards, >>>>>>>>> Sebastiano Peluso >>>>>>>>> >>>>>>>>> >>>>>>>>> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >>>>>>>>> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >>>>>>>>> >>>>>>>>> ------------------------------------------------------------------------------ >>>>>>>>> Simplify data backup and recovery for your virtual environment with vRanger. >>>>>>>>> Installation's a snap, and flexible recovery options mean your data is safe, >>>>>>>>> secure and there when you need it. Data protection magic? >>>>>>>>> Nope - It's vRanger. Get your free trial download today. >>>>>>>>> http://p.sf.net/sfu/quest-sfdev2dev >>>>>>>>> _______________________________________________ >>>>>>>>> Cloudtm-discussion mailing list >>>>>>>>> Cloudtm-discussion(a)lists.sourceforge.net >>>>>>>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>>>>>>> >>>>>>>> >>>>>>>> ------------------------------------------------------------------------------ >>>>>>>> All of the data generated in your IT infrastructure is seriously valuable. >>>>>>>> Why? It contains a definitive record of application performance, security >>>>>>>> threats, fraudulent activity, and more. Splunk takes this data and makes >>>>>>>> sense of it. IT sense. And common sense. >>>>>>>> >>>>>>>> http://p.sf.net/sfu/splunk-d2d-c2 >>>>>>>> >>>>>>>> _______________________________________________ >>>>>>>> Cloudtm-discussion mailing list >>>>>>>> >>>>>>>> Cloudtm-discussion(a)lists.sourceforge.net >>>>>>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>>>>>> >>>>>>> ------------------------------------------------------------------------------ >>>>>>> All of the data generated in your IT infrastructure is seriously valuable. >>>>>>> Why? It contains a definitive record of application performance, security >>>>>>> threats, fraudulent activity, and more. Splunk takes this data and makes >>>>>>> sense of it. IT sense. And common sense. >>>>>>> http://p.sf.net/sfu/splunk-d2d-c2________________________________________... >>>>>>> Cloudtm-discussion mailing list >>>>>>> Cloudtm-discussion(a)lists.sourceforge.net >>>>>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>>>>> >>>>>> >>>>>> ------------------------------------------------------------------------------ >>>>>> All of the data generated in your IT infrastructure is seriously valuable. >>>>>> Why? It contains a definitive record of application performance, security >>>>>> threats, fraudulent activity, and more. Splunk takes this data and makes >>>>>> sense of it. IT sense. And common sense. >>>>>> >>>>>> http://p.sf.net/sfu/splunk-d2d-c2 >>>>>> >>>>>> _______________________________________________ >>>>>> Cloudtm-discussion mailing list >>>>>> >>>>>> Cloudtm-discussion(a)lists.sourceforge.net >>>>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>>>> >>>>> >>>>> >>>>> ------------------------------------------------------------------------------ >>>>> All of the data generated in your IT infrastructure is seriously valuable. >>>>> Why? It contains a definitive record of application performance, security >>>>> threats, fraudulent activity, and more. Splunk takes this data and makes >>>>> sense of it. IT sense. And common sense. >>>>> >>>>> http://p.sf.net/sfu/splunk-d2d-c2 >>>>> >>>>> >>>>> _______________________________________________ >>>>> Cloudtm-discussion mailing list >>>>> >>>>> Cloudtm-discussion(a)lists.sourceforge.net >>>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>>> >>> >> > Begin forwarded message: > From: Mircea Markus <mircea.markus(a)jboss.com&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 13 October 2011 19:31:00 GMT+01:00 > To: Sebastiano Peluso <peluso(a)gsd.inesc-id.pt&gt; > Cc: Paolo Romano <romano(a)inesc-id.pt&gt;, Manik Surtani <msurtani(a)redhat.com&gt; > > Hi Sebastiano, right now I'm busy remaking locking/transaction support in 5.1. This should keep me busy for the next 3 weeks or so - after that the plan is to look at tpcc, integrate it and produce performance numbers to go with it. > Sorry for not taking this sooner. > Cheers, > Mircea > On 11 Oct 2011, at 09:49, Sebastiano Peluso wrote: >> Hi Mircea, >> >> have you had the opportunity to take a look at the Radargun's tpcc branch in https://github.com/sebastianopeluso/radargun/tree/tpcc ? >> >> Can I issue anyway a pull request in Github? >> >> Thank you. >> >> Cheers >> >> Sebastiano >> >> Il 29/08/11 11:49, Sebastiano Peluso ha scritto: >>> Hi Mircea, >>> >>> I've finalized the implementation of TPC-C code (New Order, Payment and Order Status transactions) within Radargun. >>> You can find the result as Radargun's tpcc branch at this link: >>> https://github.com/sebastianopeluso/radargun/tree/tpcc >>> >>> Can you take a look at the code? I wait for your feedback before issuing a pull request in Github. >>> >>> Thank you. >>> >>> Cheers, >>> >>> Sebastiano >>> >>> Il 12/07/11 11:20, Sebastiano Peluso ha scritto: >>>> Hi Mircea, >>>> >>>> I've just forked Radargun's master version and I will start the integration (following your observations/comments in the last mail) in the next week . I hope to finalize the work by the end of July. >>>> When I will have finished I will issue a pull request. >>>> >>>> Thank you for your helpfulness. >>>> >>>> Sebastiano >>>> >>>> >>>> Il 11/07/11 13:28, Mircea Markus ha scritto: >>>>> Thanks Sebastiano. Once you are finished, can you please issue a pull request for Radargun and we can take it from there? Please let me know if there'a anything I can help you with, e.g. github integration etc. >>>>> >>>>> On 8 Jul 2011, at 15:31, Sebastiano Peluso wrote: >>>>> >>>>>> Hi Mircea, >>>>>> >>>>>> thank you for your reply. My answers to your questions/comments are inline. >>>>>> >>>>>> Il 08/07/11 11:33, Mircea Markus ha scritto: >>>>>>> Hi Sebastiano, >>>>>>> >>>>>>> Very interesting stuff! >>>>>>> I think this is a nice approach of benchmarking transactions with a controlled degree contention. This is a feature currently missing in Radargun, but very important especially considering the stuff you guys are doing and the locking improvements we have scheduled for 5.1 release. >>>>>>> Do you consider contributing this back to Radargun? >>>>>> YES. >>>>>>> >>>>>>> Here are some observations I have after looking at the code: >>>>>>> - The code should not be in the PutGetStressor, as this is used for benchmarking a web-session-like access pattern which we don't want to replace. It might make more sense to be placed in a stage of its own, e.g. org.radargun.stages.TpccBenchmarkStage >>>>>> Ok. Your observation is correct. In this way the code is more modular and we can have both the web session and the tpcc benchmarks. >>>>>>> - "isPrimary" argument of TPCCTerminal.choiceTransaction is true if the node is a jgroups coordinator. This has some limitations: >>>>>>> - other data grid providers than Infinispan might not have an "coordinator" concept, and this might be difficult to infer >>>>>>> - ATM only the coordinator does writes (on multiple threads). IMO this is too restricting as it doesn't induce deadlock situations that might appear when data is written by multiple nodes. Wouldn't it make more sense to spread the writes over all the nodes? In order to overcome the dependency on jgroup's coordinator, you can use instead the AbstractDistStage.slaveIndex. This is a Radargun-level view of the benchmark cluster, independent of the benchmarked product >>>>>> This is partially correct. The "isPrimary" condition is evaluated only if the Primary-Backup replication mechanism is enabled because the code has been written in order to test this replication mechanism too. In this case only the primary node can perform write transactions. >>>>>> But if 2-Phase-Commit is enabled (lines 72-83 in TPCCTerminal), transactions (read and write) are spread over all nodes. This is always the case, because we don't have Primary-Backup in official Infinispan releases. >>>>>>> - PutGetStressor compares Strings based on the == operator, which is incorrect. e.g. e.getClass().getName() == "org.infinispan.CacheException" >>>>>> Sorry, this is a mistake :-) . >>>>>>> - As an design suggestion, I think it would make sense to have an TpccTransaction interface with an executeTransaction(CacheWrapper cacheWrapper) methods, with specialized implementations: NewOrderTransaction, PayementTransaction, OrderStatusTransaction. Basically to move all the code TPCCTerminal.executeTransaction into these three classes. Such an approach would reduce coupling between the transactions and most importantly would offer a simple way of adding new tpcc transactions to the system, by implementing the TpccTransaction interface. >>>>>> I agree. I can create one class for each tpcc transaction and assign each "case" block (referring to the current implementation of the executeTransaction method) to the corresponding class. >>>>>>> - class javadoc are the onese generater by IDEA >>>>>>> - adding .idea directory in github might cause troubles for people using IDEA and forking your repo; I think the dir should be removed >>>>>>> >>>>>>> >>>>>>> Another observation is about the way the project is managed in github: it is not a fork of Infinispan, but it was downloaded and committed as a totally different project. This means that all the nice review and merge feature that github and git offer cannot be used, but the merge needs to be made "by hand", comparing the source trees etc. I think it would make more sense to fork[1] from radargun's master[2] and apply all these nice feature on a fork. What do you think? >>>>>> Ok. I'll remove .idea directory. >>>>>> In addition, I'll insert tpcc implementation in a forked version of radargun's master in order to benefit from merge features provided by git/github. >>>>>>> >>>>>>> Cheers, >>>>>>> Mircea >>>>>> >>>>>> Thank you again. >>>>>> >>>>>> Cheers, >>>>>> >>>>>> Sebastiano >>>>>>> >>>>>>> [1] http://help.github.com/fork-a-repo/ >>>>>>> [2] https://github.com/radargun/radargun >>>>>>> >>>>>>> >>>>>>> On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: >>>>>>>> Hi all, >>>>>>>> >>>>>>>> I have added a new subsection >>>>>>>> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >>>>>>>> "Shared software resources" section on the Cloud-TM Wiki, in order to >>>>>>>> share information about the RadargunTPCC benchmark. >>>>>>>> >>>>>>>> RadargunTPCC benchmark is an implementation of the Transaction Profiles >>>>>>>> defined in the TPC Benchmark C specification [1] within Radargun >>>>>>>> Benchamark [2], and it has been designed with the purpose of executing >>>>>>>> the TPC-C benchmark against Infinispan. >>>>>>>> >>>>>>>> The source code is available at this link: >>>>>>>> https://github.com/cloudtm/RadargunTPCC >>>>>>>> >>>>>>>> Regards, >>>>>>>> Sebastiano Peluso >>>>>>>> >>>>>>>> >>>>>>>> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >>>>>>>> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >>>>>>>> >>>>>>>> ------------------------------------------------------------------------------ >>>>>>>> Simplify data backup and recovery for your virtual environment with vRanger. >>>>>>>> Installation's a snap, and flexible recovery options mean your data is safe, >>>>>>>> secure and there when you need it. Data protection magic? >>>>>>>> Nope - It's vRanger. Get your free trial download today. >>>>>>>> http://p.sf.net/sfu/quest-sfdev2dev >>>>>>>> _______________________________________________ >>>>>>>> Cloudtm-discussion mailing list >>>>>>>> Cloudtm-discussion(a)lists.sourceforge.net >>>>>>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>>>>>> >>>>>>> >>>>>>> ------------------------------------------------------------------------------ >>>>>>> All of the data generated in your IT infrastructure is seriously valuable. >>>>>>> Why? It contains a definitive record of application performance, security >>>>>>> threats, fraudulent activity, and more. Splunk takes this data and makes >>>>>>> sense of it. IT sense. And common sense. >>>>>>> >>>>>>> http://p.sf.net/sfu/splunk-d2d-c2 >>>>>>> >>>>>>> _______________________________________________ >>>>>>> Cloudtm-discussion mailing list >>>>>>> >>>>>>> Cloudtm-discussion(a)lists.sourceforge.net >>>>>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>>>>> >>>>>> ------------------------------------------------------------------------------ >>>>>> All of the data generated in your IT infrastructure is seriously valuable. >>>>>> Why? It contains a definitive record of application performance, security >>>>>> threats, fraudulent activity, and more. Splunk takes this data and makes >>>>>> sense of it. IT sense. And common sense. >>>>>> http://p.sf.net/sfu/splunk-d2d-c2________________________________________... >>>>>> Cloudtm-discussion mailing list >>>>>> Cloudtm-discussion(a)lists.sourceforge.net >>>>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>>>> >>>>> >>>>> ------------------------------------------------------------------------------ >>>>> All of the data generated in your IT infrastructure is seriously valuable. >>>>> Why? It contains a definitive record of application performance, security >>>>> threats, fraudulent activity, and more. Splunk takes this data and makes >>>>> sense of it. IT sense. And common sense. >>>>> >>>>> http://p.sf.net/sfu/splunk-d2d-c2 >>>>> >>>>> _______________________________________________ >>>>> Cloudtm-discussion mailing list >>>>> >>>>> Cloudtm-discussion(a)lists.sourceforge.net >>>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>>> >>>> >>>> >>>> ------------------------------------------------------------------------------ >>>> All of the data generated in your IT infrastructure is seriously valuable. >>>> Why? It contains a definitive record of application performance, security >>>> threats, fraudulent activity, and more. Splunk takes this data and makes >>>> sense of it. IT sense. And common sense. >>>> >>>> http://p.sf.net/sfu/splunk-d2d-c2 >>>> >>>> >>>> _______________________________________________ >>>> Cloudtm-discussion mailing list >>>> >>>> Cloudtm-discussion(a)lists.sourceforge.net >>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>> >> > Begin forwarded message: > From: Sebastiano Peluso <peluso(a)gsd.inesc-id.pt&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 11 October 2011 09:49:11 GMT+01:00 > To: Mircea Markus <mircea.markus(a)jboss.com&gt; > Cc: Paolo Romano <romano(a)inesc-id.pt&gt; > > Hi Mircea, > > have you had the opportunity to take a look at the Radargun's tpcc branch in https://github.com/sebastianopeluso/radargun/tree/tpcc ? > > Can I issue anyway a pull request in Github? > > Thank you. > > Cheers > > Sebastiano > > Il 29/08/11 11:49, Sebastiano Peluso ha scritto: >> Hi Mircea, >> >> I've finalized the implementation of TPC-C code (New Order, Payment and Order Status transactions) within Radargun. >> You can find the result as Radargun's tpcc branch at this link: >> https://github.com/sebastianopeluso/radargun/tree/tpcc >> >> Can you take a look at the code? I wait for your feedback before issuing a pull request in Github. >> >> Thank you. >> >> Cheers, >> >> Sebastiano >> >> Il 12/07/11 11:20, Sebastiano Peluso ha scritto: >>> Hi Mircea, >>> >>> I've just forked Radargun's master version and I will start the integration (following your observations/comments in the last mail) in the next week . I hope to finalize the work by the end of July. >>> When I will have finished I will issue a pull request. >>> >>> Thank you for your helpfulness. >>> >>> Sebastiano >>> >>> >>> Il 11/07/11 13:28, Mircea Markus ha scritto: >>>> Thanks Sebastiano. Once you are finished, can you please issue a pull request for Radargun and we can take it from there? Please let me know if there'a anything I can help you with, e.g. github integration etc. >>>> >>>> On 8 Jul 2011, at 15:31, Sebastiano Peluso wrote: >>>> >>>>> Hi Mircea, >>>>> >>>>> thank you for your reply. My answers to your questions/comments are inline. >>>>> >>>>> Il 08/07/11 11:33, Mircea Markus ha scritto: >>>>>> Hi Sebastiano, >>>>>> >>>>>> Very interesting stuff! >>>>>> I think this is a nice approach of benchmarking transactions with a controlled degree contention. This is a feature currently missing in Radargun, but very important especially considering the stuff you guys are doing and the locking improvements we have scheduled for 5.1 release. >>>>>> Do you consider contributing this back to Radargun? >>>>> YES. >>>>>> >>>>>> Here are some observations I have after looking at the code: >>>>>> - The code should not be in the PutGetStressor, as this is used for benchmarking a web-session-like access pattern which we don't want to replace. It might make more sense to be placed in a stage of its own, e.g. org.radargun.stages.TpccBenchmarkStage >>>>> Ok. Your observation is correct. In this way the code is more modular and we can have both the web session and the tpcc benchmarks. >>>>>> - "isPrimary" argument of TPCCTerminal.choiceTransaction is true if the node is a jgroups coordinator. This has some limitations: >>>>>> - other data grid providers than Infinispan might not have an "coordinator" concept, and this might be difficult to infer >>>>>> - ATM only the coordinator does writes (on multiple threads). IMO this is too restricting as it doesn't induce deadlock situations that might appear when data is written by multiple nodes. Wouldn't it make more sense to spread the writes over all the nodes? In order to overcome the dependency on jgroup's coordinator, you can use instead the AbstractDistStage.slaveIndex. This is a Radargun-level view of the benchmark cluster, independent of the benchmarked product >>>>> This is partially correct. The "isPrimary" condition is evaluated only if the Primary-Backup replication mechanism is enabled because the code has been written in order to test this replication mechanism too. In this case only the primary node can perform write transactions. >>>>> But if 2-Phase-Commit is enabled (lines 72-83 in TPCCTerminal), transactions (read and write) are spread over all nodes. This is always the case, because we don't have Primary-Backup in official Infinispan releases. >>>>>> - PutGetStressor compares Strings based on the == operator, which is incorrect. e.g. e.getClass().getName() == "org.infinispan.CacheException" >>>>> Sorry, this is a mistake :-) . >>>>>> - As an design suggestion, I think it would make sense to have an TpccTransaction interface with an executeTransaction(CacheWrapper cacheWrapper) methods, with specialized implementations: NewOrderTransaction, PayementTransaction, OrderStatusTransaction. Basically to move all the code TPCCTerminal.executeTransaction into these three classes. Such an approach would reduce coupling between the transactions and most importantly would offer a simple way of adding new tpcc transactions to the system, by implementing the TpccTransaction interface. >>>>> I agree. I can create one class for each tpcc transaction and assign each "case" block (referring to the current implementation of the executeTransaction method) to the corresponding class. >>>>>> - class javadoc are the onese generater by IDEA >>>>>> - adding .idea directory in github might cause troubles for people using IDEA and forking your repo; I think the dir should be removed >>>>>> >>>>>> >>>>>> Another observation is about the way the project is managed in github: it is not a fork of Infinispan, but it was downloaded and committed as a totally different project. This means that all the nice review and merge feature that github and git offer cannot be used, but the merge needs to be made "by hand", comparing the source trees etc. I think it would make more sense to fork[1] from radargun's master[2] and apply all these nice feature on a fork. What do you think? >>>>> Ok. I'll remove .idea directory. >>>>> In addition, I'll insert tpcc implementation in a forked version of radargun's master in order to benefit from merge features provided by git/github. >>>>>> >>>>>> Cheers, >>>>>> Mircea >>>>> >>>>> Thank you again. >>>>> >>>>> Cheers, >>>>> >>>>> Sebastiano >>>>>> >>>>>> [1] http://help.github.com/fork-a-repo/ >>>>>> [2] https://github.com/radargun/radargun >>>>>> >>>>>> >>>>>> On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: >>>>>>> Hi all, >>>>>>> >>>>>>> I have added a new subsection >>>>>>> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >>>>>>> "Shared software resources" section on the Cloud-TM Wiki, in order to >>>>>>> share information about the RadargunTPCC benchmark. >>>>>>> >>>>>>> RadargunTPCC benchmark is an implementation of the Transaction Profiles >>>>>>> defined in the TPC Benchmark C specification [1] within Radargun >>>>>>> Benchamark [2], and it has been designed with the purpose of executing >>>>>>> the TPC-C benchmark against Infinispan. >>>>>>> >>>>>>> The source code is available at this link: >>>>>>> https://github.com/cloudtm/RadargunTPCC >>>>>>> >>>>>>> Regards, >>>>>>> Sebastiano Peluso >>>>>>> >>>>>>> >>>>>>> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >>>>>>> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >>>>>>> >>>>>>> ------------------------------------------------------------------------------ >>>>>>> Simplify data backup and recovery for your virtual environment with vRanger. >>>>>>> Installation's a snap, and flexible recovery options mean your data is safe, >>>>>>> secure and there when you need it. Data protection magic? >>>>>>> Nope - It's vRanger. Get your free trial download today. >>>>>>> http://p.sf.net/sfu/quest-sfdev2dev >>>>>>> _______________________________________________ >>>>>>> Cloudtm-discussion mailing list >>>>>>> Cloudtm-discussion(a)lists.sourceforge.net >>>>>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>>>>> >>>>>> >>>>>> ------------------------------------------------------------------------------ >>>>>> All of the data generated in your IT infrastructure is seriously valuable. >>>>>> Why? It contains a definitive record of application performance, security >>>>>> threats, fraudulent activity, and more. Splunk takes this data and makes >>>>>> sense of it. IT sense. And common sense. >>>>>> >>>>>> http://p.sf.net/sfu/splunk-d2d-c2 >>>>>> >>>>>> _______________________________________________ >>>>>> Cloudtm-discussion mailing list >>>>>> >>>>>> Cloudtm-discussion(a)lists.sourceforge.net >>>>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>>>> >>>>> ------------------------------------------------------------------------------ >>>>> All of the data generated in your IT infrastructure is seriously valuable. >>>>> Why? It contains a definitive record of application performance, security >>>>> threats, fraudulent activity, and more. Splunk takes this data and makes >>>>> sense of it. IT sense. And common sense. >>>>> http://p.sf.net/sfu/splunk-d2d-c2________________________________________... >>>>> Cloudtm-discussion mailing list >>>>> Cloudtm-discussion(a)lists.sourceforge.net >>>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>>> >>>> >>>> ------------------------------------------------------------------------------ >>>> All of the data generated in your IT infrastructure is seriously valuable. >>>> Why? It contains a definitive record of application performance, security >>>> threats, fraudulent activity, and more. Splunk takes this data and makes >>>> sense of it. IT sense. And common sense. >>>> >>>> http://p.sf.net/sfu/splunk-d2d-c2 >>>> >>>> _______________________________________________ >>>> Cloudtm-discussion mailing list >>>> >>>> Cloudtm-discussion(a)lists.sourceforge.net >>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>> >>> >>> >>> ------------------------------------------------------------------------------ >>> All of the data generated in your IT infrastructure is seriously valuable. >>> Why? It contains a definitive record of application performance, security >>> threats, fraudulent activity, and more. Splunk takes this data and makes >>> sense of it. IT sense. And common sense. >>> >>> http://p.sf.net/sfu/splunk-d2d-c2 >>> >>> >>> _______________________________________________ >>> Cloudtm-discussion mailing list >>> >>> Cloudtm-discussion(a)lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >> > Begin forwarded message: > From: Mircea Markus <mircea.markus(a)jboss.com&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 31 August 2011 17:36:59 GMT+01:00 > To: Sebastiano Peluso <peluso(a)gsd.inesc-id.pt&gt; > Cc: Paolo Romano <romano(a)inesc-id.pt&gt; > > Thank you Sebastiano, I'll give it a look this/next week. > > On 29 Aug 2011, at 11:49, Sebastiano Peluso wrote: > >> Hi Mircea, >> >> I've finalized the implementation of TPC-C code (New Order, Payment and Order Status transactions) within Radargun. >> You can find the result as Radargun's tpcc branch at this link: >> https://github.com/sebastianopeluso/radargun/tree/tpcc >> >> Can you take a look at the code? I wait for your feedback before issuing a pull request in Github. >> >> Thank you. >> >> Cheers, >> >> Sebastiano >> >> Il 12/07/11 11:20, Sebastiano Peluso ha scritto: >>> Hi Mircea, >>> >>> I've just forked Radargun's master version and I will start the integration (following your observations/comments in the last mail) in the next week . I hope to finalize the work by the end of July. >>> When I will have finished I will issue a pull request. >>> >>> Thank you for your helpfulness. >>> >>> Sebastiano >>> >>> >>> Il 11/07/11 13:28, Mircea Markus ha scritto: >>>> Thanks Sebastiano. Once you are finished, can you please issue a pull request for Radargun and we can take it from there? Please let me know if there'a anything I can help you with, e.g. github integration etc. >>>> >>>> On 8 Jul 2011, at 15:31, Sebastiano Peluso wrote: >>>> >>>>> Hi Mircea, >>>>> >>>>> thank you for your reply. My answers to your questions/comments are inline. >>>>> >>>>> Il 08/07/11 11:33, Mircea Markus ha scritto: >>>>>> Hi Sebastiano, >>>>>> >>>>>> Very interesting stuff! >>>>>> I think this is a nice approach of benchmarking transactions with a controlled degree contention. This is a feature currently missing in Radargun, but very important especially considering the stuff you guys are doing and the locking improvements we have scheduled for 5.1 release. >>>>>> Do you consider contributing this back to Radargun? >>>>> YES. >>>>>> >>>>>> Here are some observations I have after looking at the code: >>>>>> - The code should not be in the PutGetStressor, as this is used for benchmarking a web-session-like access pattern which we don't want to replace. It might make more sense to be placed in a stage of its own, e.g. org.radargun.stages.TpccBenchmarkStage >>>>> Ok. Your observation is correct. In this way the code is more modular and we can have both the web session and the tpcc benchmarks. >>>>>> - "isPrimary" argument of TPCCTerminal.choiceTransaction is true if the node is a jgroups coordinator. This has some limitations: >>>>>> - other data grid providers than Infinispan might not have an "coordinator" concept, and this might be difficult to infer >>>>>> - ATM only the coordinator does writes (on multiple threads). IMO this is too restricting as it doesn't induce deadlock situations that might appear when data is written by multiple nodes. Wouldn't it make more sense to spread the writes over all the nodes? In order to overcome the dependency on jgroup's coordinator, you can use instead the AbstractDistStage.slaveIndex. This is a Radargun-level view of the benchmark cluster, independent of the benchmarked product >>>>> This is partially correct. The "isPrimary" condition is evaluated only if the Primary-Backup replication mechanism is enabled because the code has been written in order to test this replication mechanism too. In this case only the primary node can perform write transactions. >>>>> But if 2-Phase-Commit is enabled (lines 72-83 in TPCCTerminal), transactions (read and write) are spread over all nodes. This is always the case, because we don't have Primary-Backup in official Infinispan releases. >>>>>> - PutGetStressor compares Strings based on the == operator, which is incorrect. e.g. e.getClass().getName() == "org.infinispan.CacheException" >>>>> Sorry, this is a mistake :-) . >>>>>> - As an design suggestion, I think it would make sense to have an TpccTransaction interface with an executeTransaction(CacheWrapper cacheWrapper) methods, with specialized implementations: NewOrderTransaction, PayementTransaction, OrderStatusTransaction. Basically to move all the code TPCCTerminal.executeTransaction into these three classes. Such an approach would reduce coupling between the transactions and most importantly would offer a simple way of adding new tpcc transactions to the system, by implementing the TpccTransaction interface. >>>>> I agree. I can create one class for each tpcc transaction and assign each "case" block (referring to the current implementation of the executeTransaction method) to the corresponding class. >>>>>> - class javadoc are the onese generater by IDEA >>>>>> - adding .idea directory in github might cause troubles for people using IDEA and forking your repo; I think the dir should be removed >>>>>> >>>>>> >>>>>> Another observation is about the way the project is managed in github: it is not a fork of Infinispan, but it was downloaded and committed as a totally different project. This means that all the nice review and merge feature that github and git offer cannot be used, but the merge needs to be made "by hand", comparing the source trees etc. I think it would make more sense to fork[1] from radargun's master[2] and apply all these nice feature on a fork. What do you think? >>>>> Ok. I'll remove .idea directory. >>>>> In addition, I'll insert tpcc implementation in a forked version of radargun's master in order to benefit from merge features provided by git/github. >>>>>> >>>>>> Cheers, >>>>>> Mircea >>>>> >>>>> Thank you again. >>>>> >>>>> Cheers, >>>>> >>>>> Sebastiano >>>>>> >>>>>> [1] http://help.github.com/fork-a-repo/ >>>>>> [2] https://github.com/radargun/radargun >>>>>> >>>>>> >>>>>> On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: >>>>>>> Hi all, >>>>>>> >>>>>>> I have added a new subsection >>>>>>> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >>>>>>> "Shared software resources" section on the Cloud-TM Wiki, in order to >>>>>>> share information about the RadargunTPCC benchmark. >>>>>>> >>>>>>> RadargunTPCC benchmark is an implementation of the Transaction Profiles >>>>>>> defined in the TPC Benchmark C specification [1] within Radargun >>>>>>> Benchamark [2], and it has been designed with the purpose of executing >>>>>>> the TPC-C benchmark against Infinispan. >>>>>>> >>>>>>> The source code is available at this link: >>>>>>> https://github.com/cloudtm/RadargunTPCC >>>>>>> >>>>>>> Regards, >>>>>>> Sebastiano Peluso >>>>>>> >>>>>>> >>>>>>> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >>>>>>> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >>>>>>> >>>>>>> ------------------------------------------------------------------------------ >>>>>>> Simplify data backup and recovery for your virtual environment with vRanger. >>>>>>> Installation's a snap, and flexible recovery options mean your data is safe, >>>>>>> secure and there when you need it. Data protection magic? >>>>>>> Nope - It's vRanger. Get your free trial download today. >>>>>>> http://p.sf.net/sfu/quest-sfdev2dev >>>>>>> _______________________________________________ >>>>>>> Cloudtm-discussion mailing list >>>>>>> Cloudtm-discussion(a)lists.sourceforge.net >>>>>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>>>>> >>>>>> >>>>>> ------------------------------------------------------------------------------ >>>>>> All of the data generated in your IT infrastructure is seriously valuable. >>>>>> Why? It contains a definitive record of application performance, security >>>>>> threats, fraudulent activity, and more. Splunk takes this data and makes >>>>>> sense of it. IT sense. And common sense. >>>>>> >>>>>> http://p.sf.net/sfu/splunk-d2d-c2 >>>>>> >>>>>> _______________________________________________ >>>>>> Cloudtm-discussion mailing list >>>>>> >>>>>> Cloudtm-discussion(a)lists.sourceforge.net >>>>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>>>> >>>>> ------------------------------------------------------------------------------ >>>>> All of the data generated in your IT infrastructure is seriously valuable. >>>>> Why? It contains a definitive record of application performance, security >>>>> threats, fraudulent activity, and more. Splunk takes this data and makes >>>>> sense of it. IT sense. And common sense. >>>>> http://p.sf.net/sfu/splunk-d2d-c2________________________________________... >>>>> Cloudtm-discussion mailing list >>>>> Cloudtm-discussion(a)lists.sourceforge.net >>>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>>> >>>> >>>> ------------------------------------------------------------------------------ >>>> All of the data generated in your IT infrastructure is seriously valuable. >>>> Why? It contains a definitive record of application performance, security >>>> threats, fraudulent activity, and more. Splunk takes this data and makes >>>> sense of it. IT sense. And common sense. >>>> >>>> http://p.sf.net/sfu/splunk-d2d-c2 >>>> >>>> _______________________________________________ >>>> Cloudtm-discussion mailing list >>>> >>>> Cloudtm-discussion(a)lists.sourceforge.net >>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>> >>> >>> >>> ------------------------------------------------------------------------------ >>> All of the data generated in your IT infrastructure is seriously valuable. >>> Why? It contains a definitive record of application performance, security >>> threats, fraudulent activity, and more. Splunk takes this data and makes >>> sense of it. IT sense. And common sense. >>> >>> http://p.sf.net/sfu/splunk-d2d-c2 >>> >>> >>> _______________________________________________ >>> Cloudtm-discussion mailing list >>> >>> Cloudtm-discussion(a)lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >> > Begin forwarded message: > From: Sebastiano Peluso <peluso(a)gsd.inesc-id.pt&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 29 August 2011 11:49:17 GMT+01:00 > To: Mircea Markus <mircea.markus(a)jboss.com&gt; > Cc: Paolo Romano <romano(a)inesc-id.pt&gt; > > Hi Mircea, > > I've finalized the implementation of TPC-C code (New Order, Payment and Order Status transactions) within Radargun. > You can find the result as Radargun's tpcc branch at this link: > https://github.com/sebastianopeluso/radargun/tree/tpcc > > Can you take a look at the code? I wait for your feedback before issuing a pull request in Github. > > Thank you. > > Cheers, > > Sebastiano > > Il 12/07/11 11:20, Sebastiano Peluso ha scritto: >> Hi Mircea, >> >> I've just forked Radargun's master version and I will start the integration (following your observations/comments in the last mail) in the next week . I hope to finalize the work by the end of July. >> When I will have finished I will issue a pull request. >> >> Thank you for your helpfulness. >> >> Sebastiano >> >> >> Il 11/07/11 13:28, Mircea Markus ha scritto: >>> Thanks Sebastiano. Once you are finished, can you please issue a pull request for Radargun and we can take it from there? Please let me know if there'a anything I can help you with, e.g. github integration etc. >>> >>> On 8 Jul 2011, at 15:31, Sebastiano Peluso wrote: >>> >>>> Hi Mircea, >>>> >>>> thank you for your reply. My answers to your questions/comments are inline. >>>> >>>> Il 08/07/11 11:33, Mircea Markus ha scritto: >>>>> Hi Sebastiano, >>>>> >>>>> Very interesting stuff! >>>>> I think this is a nice approach of benchmarking transactions with a controlled degree contention. This is a feature currently missing in Radargun, but very important especially considering the stuff you guys are doing and the locking improvements we have scheduled for 5.1 release. >>>>> Do you consider contributing this back to Radargun? >>>> YES. >>>>> >>>>> Here are some observations I have after looking at the code: >>>>> - The code should not be in the PutGetStressor, as this is used for benchmarking a web-session-like access pattern which we don't want to replace. It might make more sense to be placed in a stage of its own, e.g. org.radargun.stages.TpccBenchmarkStage >>>> Ok. Your observation is correct. In this way the code is more modular and we can have both the web session and the tpcc benchmarks. >>>>> - "isPrimary" argument of TPCCTerminal.choiceTransaction is true if the node is a jgroups coordinator. This has some limitations: >>>>> - other data grid providers than Infinispan might not have an "coordinator" concept, and this might be difficult to infer >>>>> - ATM only the coordinator does writes (on multiple threads). IMO this is too restricting as it doesn't induce deadlock situations that might appear when data is written by multiple nodes. Wouldn't it make more sense to spread the writes over all the nodes? In order to overcome the dependency on jgroup's coordinator, you can use instead the AbstractDistStage.slaveIndex. This is a Radargun-level view of the benchmark cluster, independent of the benchmarked product >>>> This is partially correct. The "isPrimary" condition is evaluated only if the Primary-Backup replication mechanism is enabled because the code has been written in order to test this replication mechanism too. In this case only the primary node can perform write transactions. >>>> But if 2-Phase-Commit is enabled (lines 72-83 in TPCCTerminal), transactions (read and write) are spread over all nodes. This is always the case, because we don't have Primary-Backup in official Infinispan releases. >>>>> - PutGetStressor compares Strings based on the == operator, which is incorrect. e.g. e.getClass().getName() == "org.infinispan.CacheException" >>>> Sorry, this is a mistake :-) . >>>>> - As an design suggestion, I think it would make sense to have an TpccTransaction interface with an executeTransaction(CacheWrapper cacheWrapper) methods, with specialized implementations: NewOrderTransaction, PayementTransaction, OrderStatusTransaction. Basically to move all the code TPCCTerminal.executeTransaction into these three classes. Such an approach would reduce coupling between the transactions and most importantly would offer a simple way of adding new tpcc transactions to the system, by implementing the TpccTransaction interface. >>>> I agree. I can create one class for each tpcc transaction and assign each "case" block (referring to the current implementation of the executeTransaction method) to the corresponding class. >>>>> - class javadoc are the onese generater by IDEA >>>>> - adding .idea directory in github might cause troubles for people using IDEA and forking your repo; I think the dir should be removed >>>>> >>>>> >>>>> Another observation is about the way the project is managed in github: it is not a fork of Infinispan, but it was downloaded and committed as a totally different project. This means that all the nice review and merge feature that github and git offer cannot be used, but the merge needs to be made "by hand", comparing the source trees etc. I think it would make more sense to fork[1] from radargun's master[2] and apply all these nice feature on a fork. What do you think? >>>> Ok. I'll remove .idea directory. >>>> In addition, I'll insert tpcc implementation in a forked version of radargun's master in order to benefit from merge features provided by git/github. >>>>> >>>>> Cheers, >>>>> Mircea >>>> >>>> Thank you again. >>>> >>>> Cheers, >>>> >>>> Sebastiano >>>>> >>>>> [1] http://help.github.com/fork-a-repo/ >>>>> [2] https://github.com/radargun/radargun >>>>> >>>>> >>>>> On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: >>>>>> Hi all, >>>>>> >>>>>> I have added a new subsection >>>>>> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >>>>>> "Shared software resources" section on the Cloud-TM Wiki, in order to >>>>>> share information about the RadargunTPCC benchmark. >>>>>> >>>>>> RadargunTPCC benchmark is an implementation of the Transaction Profiles >>>>>> defined in the TPC Benchmark C specification [1] within Radargun >>>>>> Benchamark [2], and it has been designed with the purpose of executing >>>>>> the TPC-C benchmark against Infinispan. >>>>>> >>>>>> The source code is available at this link: >>>>>> https://github.com/cloudtm/RadargunTPCC >>>>>> >>>>>> Regards, >>>>>> Sebastiano Peluso >>>>>> >>>>>> >>>>>> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >>>>>> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >>>>>> >>>>>> ------------------------------------------------------------------------------ >>>>>> Simplify data backup and recovery for your virtual environment with vRanger. >>>>>> Installation's a snap, and flexible recovery options mean your data is safe, >>>>>> secure and there when you need it. Data protection magic? >>>>>> Nope - It's vRanger. Get your free trial download today. >>>>>> http://p.sf.net/sfu/quest-sfdev2dev >>>>>> _______________________________________________ >>>>>> Cloudtm-discussion mailing list >>>>>> Cloudtm-discussion(a)lists.sourceforge.net >>>>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>>>> >>>>> >>>>> ------------------------------------------------------------------------------ >>>>> All of the data generated in your IT infrastructure is seriously valuable. >>>>> Why? It contains a definitive record of application performance, security >>>>> threats, fraudulent activity, and more. Splunk takes this data and makes >>>>> sense of it. IT sense. And common sense. >>>>> >>>>> http://p.sf.net/sfu/splunk-d2d-c2 >>>>> >>>>> _______________________________________________ >>>>> Cloudtm-discussion mailing list >>>>> >>>>> Cloudtm-discussion(a)lists.sourceforge.net >>>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>>> >>>> ------------------------------------------------------------------------------ >>>> All of the data generated in your IT infrastructure is seriously valuable. >>>> Why? It contains a definitive record of application performance, security >>>> threats, fraudulent activity, and more. Splunk takes this data and makes >>>> sense of it. IT sense. And common sense. >>>> http://p.sf.net/sfu/splunk-d2d-c2________________________________________... >>>> Cloudtm-discussion mailing list >>>> Cloudtm-discussion(a)lists.sourceforge.net >>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>> >>> >>> ------------------------------------------------------------------------------ >>> All of the data generated in your IT infrastructure is seriously valuable. >>> Why? It contains a definitive record of application performance, security >>> threats, fraudulent activity, and more. Splunk takes this data and makes >>> sense of it. IT sense. And common sense. >>> >>> http://p.sf.net/sfu/splunk-d2d-c2 >>> >>> _______________________________________________ >>> Cloudtm-discussion mailing list >>> >>> Cloudtm-discussion(a)lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >> >> >> >> ------------------------------------------------------------------------------ >> All of the data generated in your IT infrastructure is seriously valuable. >> Why? It contains a definitive record of application performance, security >> threats, fraudulent activity, and more. Splunk takes this data and makes >> sense of it. IT sense. And common sense. >> >> http://p.sf.net/sfu/splunk-d2d-c2 >> >> >> _______________________________________________ >> Cloudtm-discussion mailing list >> >> Cloudtm-discussion(a)lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion > Begin forwarded message: > From: Sebastiano Peluso <peluso(a)gsd.inesc-id.pt&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 12 July 2011 10:20:33 GMT+01:00 > To: cloudtm-discussion(a)lists.sourceforge.net > Reply-To: cloudtm-discussion(a)lists.sourceforge.net > > Hi Mircea, > > I've just forked Radargun's master version and I will start the integration (following your observations/comments in the last mail) in the next week . I hope to finalize the work by the end of July. > When I will have finished I will issue a pull request. > > Thank you for your helpfulness. > > Sebastiano > > > Il 11/07/11 13:28, Mircea Markus ha scritto: >> Thanks Sebastiano. Once you are finished, can you please issue a pull request for Radargun and we can take it from there? Please let me know if there'a anything I can help you with, e.g. github integration etc. >> >> On 8 Jul 2011, at 15:31, Sebastiano Peluso wrote: >> >>> Hi Mircea, >>> >>> thank you for your reply. My answers to your questions/comments are inline. >>> >>> Il 08/07/11 11:33, Mircea Markus ha scritto: >>>> Hi Sebastiano, >>>> >>>> Very interesting stuff! >>>> I think this is a nice approach of benchmarking transactions with a controlled degree contention. This is a feature currently missing in Radargun, but very important especially considering the stuff you guys are doing and the locking improvements we have scheduled for 5.1 release. >>>> Do you consider contributing this back to Radargun? >>> YES. >>>> >>>> Here are some observations I have after looking at the code: >>>> - The code should not be in the PutGetStressor, as this is used for benchmarking a web-session-like access pattern which we don't want to replace. It might make more sense to be placed in a stage of its own, e.g. org.radargun.stages.TpccBenchmarkStage >>> Ok. Your observation is correct. In this way the code is more modular and we can have both the web session and the tpcc benchmarks. >>>> - "isPrimary" argument of TPCCTerminal.choiceTransaction is true if the node is a jgroups coordinator. This has some limitations: >>>> - other data grid providers than Infinispan might not have an "coordinator" concept, and this might be difficult to infer >>>> - ATM only the coordinator does writes (on multiple threads). IMO this is too restricting as it doesn't induce deadlock situations that might appear when data is written by multiple nodes. Wouldn't it make more sense to spread the writes over all the nodes? In order to overcome the dependency on jgroup's coordinator, you can use instead the AbstractDistStage.slaveIndex. This is a Radargun-level view of the benchmark cluster, independent of the benchmarked product >>> This is partially correct. The "isPrimary" condition is evaluated only if the Primary-Backup replication mechanism is enabled because the code has been written in order to test this replication mechanism too. In this case only the primary node can perform write transactions. >>> But if 2-Phase-Commit is enabled (lines 72-83 in TPCCTerminal), transactions (read and write) are spread over all nodes. This is always the case, because we don't have Primary-Backup in official Infinispan releases. >>>> - PutGetStressor compares Strings based on the == operator, which is incorrect. e.g. e.getClass().getName() == "org.infinispan.CacheException" >>> Sorry, this is a mistake :-) . >>>> - As an design suggestion, I think it would make sense to have an TpccTransaction interface with an executeTransaction(CacheWrapper cacheWrapper) methods, with specialized implementations: NewOrderTransaction, PayementTransaction, OrderStatusTransaction. Basically to move all the code TPCCTerminal.executeTransaction into these three classes. Such an approach would reduce coupling between the transactions and most importantly would offer a simple way of adding new tpcc transactions to the system, by implementing the TpccTransaction interface. >>> I agree. I can create one class for each tpcc transaction and assign each "case" block (referring to the current implementation of the executeTransaction method) to the corresponding class. >>>> - class javadoc are the onese generater by IDEA >>>> - adding .idea directory in github might cause troubles for people using IDEA and forking your repo; I think the dir should be removed >>>> >>>> >>>> Another observation is about the way the project is managed in github: it is not a fork of Infinispan, but it was downloaded and committed as a totally different project. This means that all the nice review and merge feature that github and git offer cannot be used, but the merge needs to be made "by hand", comparing the source trees etc. I think it would make more sense to fork[1] from radargun's master[2] and apply all these nice feature on a fork. What do you think? >>> Ok. I'll remove .idea directory. >>> In addition, I'll insert tpcc implementation in a forked version of radargun's master in order to benefit from merge features provided by git/github. >>>> >>>> Cheers, >>>> Mircea >>> >>> Thank you again. >>> >>> Cheers, >>> >>> Sebastiano >>>> >>>> [1] http://help.github.com/fork-a-repo/ >>>> [2] https://github.com/radargun/radargun >>>> >>>> >>>> On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: >>>>> Hi all, >>>>> >>>>> I have added a new subsection >>>>> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >>>>> "Shared software resources" section on the Cloud-TM Wiki, in order to >>>>> share information about the RadargunTPCC benchmark. >>>>> >>>>> RadargunTPCC benchmark is an implementation of the Transaction Profiles >>>>> defined in the TPC Benchmark C specification [1] within Radargun >>>>> Benchamark [2], and it has been designed with the purpose of executing >>>>> the TPC-C benchmark against Infinispan. >>>>> >>>>> The source code is available at this link: >>>>> https://github.com/cloudtm/RadargunTPCC >>>>> >>>>> Regards, >>>>> Sebastiano Peluso >>>>> >>>>> >>>>> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >>>>> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >>>>> >>>>> ------------------------------------------------------------------------------ >>>>> Simplify data backup and recovery for your virtual environment with vRanger. >>>>> Installation's a snap, and flexible recovery options mean your data is safe, >>>>> secure and there when you need it. Data protection magic? >>>>> Nope - It's vRanger. Get your free trial download today. >>>>> http://p.sf.net/sfu/quest-sfdev2dev >>>>> _______________________________________________ >>>>> Cloudtm-discussion mailing list >>>>> Cloudtm-discussion(a)lists.sourceforge.net >>>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>>> >>>> >>>> ------------------------------------------------------------------------------ >>>> All of the data generated in your IT infrastructure is seriously valuable. >>>> Why? It contains a definitive record of application performance, security >>>> threats, fraudulent activity, and more. Splunk takes this data and makes >>>> sense of it. IT sense. And common sense. >>>> >>>> http://p.sf.net/sfu/splunk-d2d-c2 >>>> >>>> _______________________________________________ >>>> Cloudtm-discussion mailing list >>>> >>>> Cloudtm-discussion(a)lists.sourceforge.net >>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>> >>> ------------------------------------------------------------------------------ >>> All of the data generated in your IT infrastructure is seriously valuable. >>> Why? It contains a definitive record of application performance, security >>> threats, fraudulent activity, and more. Splunk takes this data and makes >>> sense of it. IT sense. And common sense. >>> http://p.sf.net/sfu/splunk-d2d-c2________________________________________... >>> Cloudtm-discussion mailing list >>> Cloudtm-discussion(a)lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >> >> >> ------------------------------------------------------------------------------ >> All of the data generated in your IT infrastructure is seriously valuable. >> Why? It contains a definitive record of application performance, security >> threats, fraudulent activity, and more. Splunk takes this data and makes >> sense of it. IT sense. And common sense. >> >> http://p.sf.net/sfu/splunk-d2d-c2 >> >> _______________________________________________ >> Cloudtm-discussion mailing list >> >> Cloudtm-discussion(a)lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion > > ------------------------------------------------------------------------------ > All of the data generated in your IT infrastructure is seriously valuable. > Why? It contains a definitive record of application performance, security > threats, fraudulent activity, and more. Splunk takes this data and makes > sense of it. IT sense. And common sense. > http://p.sf.net/sfu/splunk-d2d-c2________________________________________... > Cloudtm-discussion mailing list > Cloudtm-discussion(a)lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion Begin forwarded message: > From: Mircea Markus <mircea.markus(a)jboss.com&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 11 July 2011 13:28:15 GMT+01:00 > To: cloudtm-discussion(a)lists.sourceforge.net > > Thanks Sebastiano. Once you are finished, can you please issue a pull request for Radargun and we can take it from there? Please let me know if there'a anything I can help you with, e.g. github integration etc. > > On 8 Jul 2011, at 15:31, Sebastiano Peluso wrote: > >> Hi Mircea, >> >> thank you for your reply. My answers to your questions/comments are inline. >> >> Il 08/07/11 11:33, Mircea Markus ha scritto: >>> Hi Sebastiano, >>> >>> Very interesting stuff! >>> I think this is a nice approach of benchmarking transactions with a controlled degree contention. This is a feature currently missing in Radargun, but very important especially considering the stuff you guys are doing and the locking improvements we have scheduled for 5.1 release. >>> Do you consider contributing this back to Radargun? >> YES. >>> >>> Here are some observations I have after looking at the code: >>> - The code should not be in the PutGetStressor, as this is used for benchmarking a web-session-like access pattern which we don't want to replace. It might make more sense to be placed in a stage of its own, e.g. org.radargun.stages.TpccBenchmarkStage >> Ok. Your observation is correct. In this way the code is more modular and we can have both the web session and the tpcc benchmarks. >>> - "isPrimary" argument of TPCCTerminal.choiceTransaction is true if the node is a jgroups coordinator. This has some limitations: >>> - other data grid providers than Infinispan might not have an "coordinator" concept, and this might be difficult to infer >>> - ATM only the coordinator does writes (on multiple threads). IMO this is too restricting as it doesn't induce deadlock situations that might appear when data is written by multiple nodes. Wouldn't it make more sense to spread the writes over all the nodes? In order to overcome the dependency on jgroup's coordinator, you can use instead the AbstractDistStage.slaveIndex. This is a Radargun-level view of the benchmark cluster, independent of the benchmarked product >> This is partially correct. The "isPrimary" condition is evaluated only if the Primary-Backup replication mechanism is enabled because the code has been written in order to test this replication mechanism too. In this case only the primary node can perform write transactions. >> But if 2-Phase-Commit is enabled (lines 72-83 in TPCCTerminal), transactions (read and write) are spread over all nodes. This is always the case, because we don't have Primary-Backup in official Infinispan releases. >>> - PutGetStressor compares Strings based on the == operator, which is incorrect. e.g. e.getClass().getName() == "org.infinispan.CacheException" >> Sorry, this is a mistake :-) . >>> - As an design suggestion, I think it would make sense to have an TpccTransaction interface with an executeTransaction(CacheWrapper cacheWrapper) methods, with specialized implementations: NewOrderTransaction, PayementTransaction, OrderStatusTransaction. Basically to move all the code TPCCTerminal.executeTransaction into these three classes. Such an approach would reduce coupling between the transactions and most importantly would offer a simple way of adding new tpcc transactions to the system, by implementing the TpccTransaction interface. >> I agree. I can create one class for each tpcc transaction and assign each "case" block (referring to the current implementation of the executeTransaction method) to the corresponding class. >>> - class javadoc are the onese generater by IDEA >>> - adding .idea directory in github might cause troubles for people using IDEA and forking your repo; I think the dir should be removed >>> >>> >>> Another observation is about the way the project is managed in github: it is not a fork of Infinispan, but it was downloaded and committed as a totally different project. This means that all the nice review and merge feature that github and git offer cannot be used, but the merge needs to be made "by hand", comparing the source trees etc. I think it would make more sense to fork[1] from radargun's master[2] and apply all these nice feature on a fork. What do you think? >> Ok. I'll remove .idea directory. >> In addition, I'll insert tpcc implementation in a forked version of radargun's master in order to benefit from merge features provided by git/github. >>> >>> Cheers, >>> Mircea >> >> Thank you again. >> >> Cheers, >> >> Sebastiano >>> >>> [1] http://help.github.com/fork-a-repo/ >>> [2] https://github.com/radargun/radargun >>> >>> >>> On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: >>>> Hi all, >>>> >>>> I have added a new subsection >>>> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >>>> "Shared software resources" section on the Cloud-TM Wiki, in order to >>>> share information about the RadargunTPCC benchmark. >>>> >>>> RadargunTPCC benchmark is an implementation of the Transaction Profiles >>>> defined in the TPC Benchmark C specification [1] within Radargun >>>> Benchamark [2], and it has been designed with the purpose of executing >>>> the TPC-C benchmark against Infinispan. >>>> >>>> The source code is available at this link: >>>> https://github.com/cloudtm/RadargunTPCC >>>> >>>> Regards, >>>> Sebastiano Peluso >>>> >>>> >>>> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >>>> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >>>> >>>> ------------------------------------------------------------------------------ >>>> Simplify data backup and recovery for your virtual environment with vRanger. >>>> Installation's a snap, and flexible recovery options mean your data is safe, >>>> secure and there when you need it. Data protection magic? >>>> Nope - It's vRanger. Get your free trial download today. >>>> http://p.sf.net/sfu/quest-sfdev2dev >>>> _______________________________________________ >>>> Cloudtm-discussion mailing list >>>> Cloudtm-discussion(a)lists.sourceforge.net >>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>> >>> >>> ------------------------------------------------------------------------------ >>> All of the data generated in your IT infrastructure is seriously valuable. >>> Why? It contains a definitive record of application performance, security >>> threats, fraudulent activity, and more. Splunk takes this data and makes >>> sense of it. IT sense. And common sense. >>> >>> http://p.sf.net/sfu/splunk-d2d-c2 >>> >>> _______________________________________________ >>> Cloudtm-discussion mailing list >>> >>> Cloudtm-discussion(a)lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >> >> ------------------------------------------------------------------------------ >> All of the data generated in your IT infrastructure is seriously valuable. >> Why? It contains a definitive record of application performance, security >> threats, fraudulent activity, and more. Splunk takes this data and makes >> sense of it. IT sense. And common sense. >> http://p.sf.net/sfu/splunk-d2d-c2________________________________________... >> Cloudtm-discussion mailing list >> Cloudtm-discussion(a)lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion > Begin forwarded message: > From: Mircea Markus <mircea.markus(a)jboss.com&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 11 July 2011 13:28:15 GMT+01:00 > To: cloudtm-discussion(a)lists.sourceforge.net > Reply-To: cloudtm-discussion(a)lists.sourceforge.net > > Thanks Sebastiano. Once you are finished, can you please issue a pull request for Radargun and we can take it from there? Please let me know if there'a anything I can help you with, e.g. github integration etc. > > On 8 Jul 2011, at 15:31, Sebastiano Peluso wrote: > >> Hi Mircea, >> >> thank you for your reply. My answers to your questions/comments are inline. >> >> Il 08/07/11 11:33, Mircea Markus ha scritto: >>> Hi Sebastiano, >>> >>> Very interesting stuff! >>> I think this is a nice approach of benchmarking transactions with a controlled degree contention. This is a feature currently missing in Radargun, but very important especially considering the stuff you guys are doing and the locking improvements we have scheduled for 5.1 release. >>> Do you consider contributing this back to Radargun? >> YES. >>> >>> Here are some observations I have after looking at the code: >>> - The code should not be in the PutGetStressor, as this is used for benchmarking a web-session-like access pattern which we don't want to replace. It might make more sense to be placed in a stage of its own, e.g. org.radargun.stages.TpccBenchmarkStage >> Ok. Your observation is correct. In this way the code is more modular and we can have both the web session and the tpcc benchmarks. >>> - "isPrimary" argument of TPCCTerminal.choiceTransaction is true if the node is a jgroups coordinator. This has some limitations: >>> - other data grid providers than Infinispan might not have an "coordinator" concept, and this might be difficult to infer >>> - ATM only the coordinator does writes (on multiple threads). IMO this is too restricting as it doesn't induce deadlock situations that might appear when data is written by multiple nodes. Wouldn't it make more sense to spread the writes over all the nodes? In order to overcome the dependency on jgroup's coordinator, you can use instead the AbstractDistStage.slaveIndex. This is a Radargun-level view of the benchmark cluster, independent of the benchmarked product >> This is partially correct. The "isPrimary" condition is evaluated only if the Primary-Backup replication mechanism is enabled because the code has been written in order to test this replication mechanism too. In this case only the primary node can perform write transactions. >> But if 2-Phase-Commit is enabled (lines 72-83 in TPCCTerminal), transactions (read and write) are spread over all nodes. This is always the case, because we don't have Primary-Backup in official Infinispan releases. >>> - PutGetStressor compares Strings based on the == operator, which is incorrect. e.g. e.getClass().getName() == "org.infinispan.CacheException" >> Sorry, this is a mistake :-) . >>> - As an design suggestion, I think it would make sense to have an TpccTransaction interface with an executeTransaction(CacheWrapper cacheWrapper) methods, with specialized implementations: NewOrderTransaction, PayementTransaction, OrderStatusTransaction. Basically to move all the code TPCCTerminal.executeTransaction into these three classes. Such an approach would reduce coupling between the transactions and most importantly would offer a simple way of adding new tpcc transactions to the system, by implementing the TpccTransaction interface. >> I agree. I can create one class for each tpcc transaction and assign each "case" block (referring to the current implementation of the executeTransaction method) to the corresponding class. >>> - class javadoc are the onese generater by IDEA >>> - adding .idea directory in github might cause troubles for people using IDEA and forking your repo; I think the dir should be removed >>> >>> >>> Another observation is about the way the project is managed in github: it is not a fork of Infinispan, but it was downloaded and committed as a totally different project. This means that all the nice review and merge feature that github and git offer cannot be used, but the merge needs to be made "by hand", comparing the source trees etc. I think it would make more sense to fork[1] from radargun's master[2] and apply all these nice feature on a fork. What do you think? >> Ok. I'll remove .idea directory. >> In addition, I'll insert tpcc implementation in a forked version of radargun's master in order to benefit from merge features provided by git/github. >>> >>> Cheers, >>> Mircea >> >> Thank you again. >> >> Cheers, >> >> Sebastiano >>> >>> [1] http://help.github.com/fork-a-repo/ >>> [2] https://github.com/radargun/radargun >>> >>> >>> On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: >>>> Hi all, >>>> >>>> I have added a new subsection >>>> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >>>> "Shared software resources" section on the Cloud-TM Wiki, in order to >>>> share information about the RadargunTPCC benchmark. >>>> >>>> RadargunTPCC benchmark is an implementation of the Transaction Profiles >>>> defined in the TPC Benchmark C specification [1] within Radargun >>>> Benchamark [2], and it has been designed with the purpose of executing >>>> the TPC-C benchmark against Infinispan. >>>> >>>> The source code is available at this link: >>>> https://github.com/cloudtm/RadargunTPCC >>>> >>>> Regards, >>>> Sebastiano Peluso >>>> >>>> >>>> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >>>> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >>>> >>>> ------------------------------------------------------------------------------ >>>> Simplify data backup and recovery for your virtual environment with vRanger. >>>> Installation's a snap, and flexible recovery options mean your data is safe, >>>> secure and there when you need it. Data protection magic? >>>> Nope - It's vRanger. Get your free trial download today. >>>> http://p.sf.net/sfu/quest-sfdev2dev >>>> _______________________________________________ >>>> Cloudtm-discussion mailing list >>>> Cloudtm-discussion(a)lists.sourceforge.net >>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>> >>> >>> ------------------------------------------------------------------------------ >>> All of the data generated in your IT infrastructure is seriously valuable. >>> Why? It contains a definitive record of application performance, security >>> threats, fraudulent activity, and more. Splunk takes this data and makes >>> sense of it. IT sense. And common sense. >>> >>> http://p.sf.net/sfu/splunk-d2d-c2 >>> >>> _______________________________________________ >>> Cloudtm-discussion mailing list >>> >>> Cloudtm-discussion(a)lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >> >> ------------------------------------------------------------------------------ >> All of the data generated in your IT infrastructure is seriously valuable. >> Why? It contains a definitive record of application performance, security >> threats, fraudulent activity, and more. Splunk takes this data and makes >> sense of it. IT sense. And common sense. >> http://p.sf.net/sfu/splunk-d2d-c2________________________________________... >> Cloudtm-discussion mailing list >> Cloudtm-discussion(a)lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion > > ------------------------------------------------------------------------------ > All of the data generated in your IT infrastructure is seriously valuable. > Why? It contains a definitive record of application performance, security > threats, fraudulent activity, and more. Splunk takes this data and makes > sense of it. IT sense. And common sense. > http://p.sf.net/sfu/splunk-d2d-c2________________________________________... > Cloudtm-discussion mailing list > Cloudtm-discussion(a)lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion Begin forwarded message: > From: Sebastiano Peluso <peluso(a)gsd.inesc-id.pt&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 8 July 2011 15:31:03 GMT+01:00 > To: cloudtm-discussion(a)lists.sourceforge.net > Reply-To: cloudtm-discussion(a)lists.sourceforge.net > > Hi Mircea, > > thank you for your reply. My answers to your questions/comments are inline. > > Il 08/07/11 11:33, Mircea Markus ha scritto: >> Hi Sebastiano, >> >> Very interesting stuff! >> I think this is a nice approach of benchmarking transactions with a controlled degree contention. This is a feature currently missing in Radargun, but very important especially considering the stuff you guys are doing and the locking improvements we have scheduled for 5.1 release. >> Do you consider contributing this back to Radargun? > YES. >> >> Here are some observations I have after looking at the code: >> - The code should not be in the PutGetStressor, as this is used for benchmarking a web-session-like access pattern which we don't want to replace. It might make more sense to be placed in a stage of its own, e.g. org.radargun.stages.TpccBenchmarkStage > Ok. Your observation is correct. In this way the code is more modular and we can have both the web session and the tpcc benchmarks. >> - "isPrimary" argument of TPCCTerminal.choiceTransaction is true if the node is a jgroups coordinator. This has some limitations: >> - other data grid providers than Infinispan might not have an "coordinator" concept, and this might be difficult to infer >> - ATM only the coordinator does writes (on multiple threads). IMO this is too restricting as it doesn't induce deadlock situations that might appear when data is written by multiple nodes. Wouldn't it make more sense to spread the writes over all the nodes? In order to overcome the dependency on jgroup's coordinator, you can use instead the AbstractDistStage.slaveIndex. This is a Radargun-level view of the benchmark cluster, independent of the benchmarked product > This is partially correct. The "isPrimary" condition is evaluated only if the Primary-Backup replication mechanism is enabled because the code has been written in order to test this replication mechanism too. In this case only the primary node can perform write transactions. > But if 2-Phase-Commit is enabled (lines 72-83 in TPCCTerminal), transactions (read and write) are spread over all nodes. This is always the case, because we don't have Primary-Backup in official Infinispan releases. >> - PutGetStressor compares Strings based on the == operator, which is incorrect. e.g. e.getClass().getName() == "org.infinispan.CacheException" > Sorry, this is a mistake :-) . >> - As an design suggestion, I think it would make sense to have an TpccTransaction interface with an executeTransaction(CacheWrapper cacheWrapper) methods, with specialized implementations: NewOrderTransaction, PayementTransaction, OrderStatusTransaction. Basically to move all the code TPCCTerminal.executeTransaction into these three classes. Such an approach would reduce coupling between the transactions and most importantly would offer a simple way of adding new tpcc transactions to the system, by implementing the TpccTransaction interface. > I agree. I can create one class for each tpcc transaction and assign each "case" block (referring to the current implementation of the executeTransaction method) to the corresponding class. >> - class javadoc are the onese generater by IDEA >> - adding .idea directory in github might cause troubles for people using IDEA and forking your repo; I think the dir should be removed >> >> >> Another observation is about the way the project is managed in github: it is not a fork of Infinispan, but it was downloaded and committed as a totally different project. This means that all the nice review and merge feature that github and git offer cannot be used, but the merge needs to be made "by hand", comparing the source trees etc. I think it would make more sense to fork[1] from radargun's master[2] and apply all these nice feature on a fork. What do you think? > Ok. I'll remove .idea directory. > In addition, I'll insert tpcc implementation in a forked version of radargun's master in order to benefit from merge features provided by git/github. >> >> Cheers, >> Mircea > > Thank you again. > > Cheers, > > Sebastiano >> >> [1] http://help.github.com/fork-a-repo/ >> [2] https://github.com/radargun/radargun >> >> >> On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: >>> Hi all, >>> >>> I have added a new subsection >>> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >>> "Shared software resources" section on the Cloud-TM Wiki, in order to >>> share information about the RadargunTPCC benchmark. >>> >>> RadargunTPCC benchmark is an implementation of the Transaction Profiles >>> defined in the TPC Benchmark C specification [1] within Radargun >>> Benchamark [2], and it has been designed with the purpose of executing >>> the TPC-C benchmark against Infinispan. >>> >>> The source code is available at this link: >>> https://github.com/cloudtm/RadargunTPCC >>> >>> Regards, >>> Sebastiano Peluso >>> >>> >>> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >>> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >>> >>> ------------------------------------------------------------------------------ >>> Simplify data backup and recovery for your virtual environment with vRanger. >>> Installation's a snap, and flexible recovery options mean your data is safe, >>> secure and there when you need it. Data protection magic? >>> Nope - It's vRanger. Get your free trial download today. >>> http://p.sf.net/sfu/quest-sfdev2dev >>> _______________________________________________ >>> Cloudtm-discussion mailing list >>> Cloudtm-discussion(a)lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >> >> >> ------------------------------------------------------------------------------ >> All of the data generated in your IT infrastructure is seriously valuable. >> Why? It contains a definitive record of application performance, security >> threats, fraudulent activity, and more. Splunk takes this data and makes >> sense of it. IT sense. And common sense. >> >> http://p.sf.net/sfu/splunk-d2d-c2 >> >> _______________________________________________ >> Cloudtm-discussion mailing list >> >> Cloudtm-discussion(a)lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion > > ------------------------------------------------------------------------------ > All of the data generated in your IT infrastructure is seriously valuable. > Why? It contains a definitive record of application performance, security > threats, fraudulent activity, and more. Splunk takes this data and makes > sense of it. IT sense. And common sense. > http://p.sf.net/sfu/splunk-d2d-c2________________________________________... > Cloudtm-discussion mailing list > Cloudtm-discussion(a)lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion Begin forwarded message: > From: Manik Surtani <msurtani(a)redhat.com&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 8 July 2011 12:04:56 GMT+01:00 > To: Mircea Markus <mircea.markus(a)jboss.com&gt; > Cc: Dan Berindei <dberinde(a)redhat.com&gt; > > > On 8 Jul 2011, at 11:41, Mircea Markus wrote: > >> I did, search this email for "Here are some observations I have after looking". > > Ok, didn't read the forwarded email. :) > > -- > Manik Surtani > manik(a)jboss.org > twitter.com/maniksurtani > > Lead, Infinispan > http://www.infinispan.org > > > Begin forwarded message: > From: Mircea Markus <mircea.markus(a)jboss.com&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 8 July 2011 11:41:42 GMT+01:00 > To: Manik Surtani <msurtani(a)redhat.com&gt; > Cc: Dan Berindei <dberinde(a)redhat.com&gt; > > I did, search this email for "Here are some observations I have after looking". > > On 8 Jul 2011, at 11:39, Manik Surtani wrote: > >> Could you give them some pointers on how to make this easier to integrate? >> >> Sent from my mobile phone >> >> On 8 Jul 2011, at 11:36, Mircea Markus <mircea.markus(a)jboss.com&gt; wrote: >> >>> FYI, I reviewed the code and is far from being integrable in its current shape. The idea is really nice though, hope they'll update and we can at least take some of it. >>> >>> Begin forwarded message: >>> >>>> From: Mircea Markus <mircea.markus(a)jboss.com&gt; >>>> Date: 8 July 2011 11:33:41 GMT+01:00 >>>> To: cloudtm-discussion(a)lists.sourceforge.net >>>> Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark >>>> Reply-To: cloudtm-discussion(a)lists.sourceforge.net >>>> >>>> Hi Sebastiano, >>>> >>>> Very interesting stuff! >>>> I think this is a nice approach of benchmarking transactions with a controlled degree contention. This is a feature currently missing in Radargun, but very important especially considering the stuff you guys are doing and the locking improvements we have scheduled for 5.1 release. >>>> Do you consider contributing this back to Radargun? >>>> >>>> Here are some observations I have after looking at the code: >>>> - The code should not be in the PutGetStressor, as this is used for benchmarking a web-session-like access pattern which we don't want to replace. It might make more sense to be placed in a stage of its own, e.g. org.radargun.stages.TpccBenchmarkStage >>>> - "isPrimary" argument of TPCCTerminal.choiceTransaction is true if the node is a jgroups coordinator. This has some limitations: >>>> - other data grid providers than Infinispan might not have an "coordinator" concept, and this might be difficult to infer >>>> - ATM only the coordinator does writes (on multiple threads). IMO this is too restricting as it doesn't induce deadlock situations that might appear when data is written by multiple nodes. Wouldn't it make more sense to spread the writes over all the nodes? >>>> In order to overcome the dependency on jgroup's coordinator, you can use instead the AbstractDistStage.slaveIndex. This is a Radargun-level view of the benchmark cluster, independent of the benchmarked product >>>> - PutGetStressor compares Strings based on the == operator, which is incorrect. e.g. e.getClass().getName() == "org.infinispan.CacheException" >>>> - As an design suggestion, I think it would make sense to have an TpccTransaction interface with an executeTransaction(CacheWrapper cacheWrapper) methods, with specialized implementations: NewOrderTransaction, PayementTransaction, OrderStatusTransaction. Basically to move all the code TPCCTerminal.executeTransaction into these three classes. Such an approach would reduce coupling between the transactions and most importantly would offer a simple way of adding new tpcc transactions to the system, by implementing the TpccTransaction interface. >>>> - class javadoc are the onese generater by IDEA >>>> - adding .idea directory in github might cause troubles for people using IDEA and forking your repo; I think the dir should be removed >>>> >>>> >>>> Another observation is about the way the project is managed in github: it is not a fork of Infinispan, but it was downloaded and committed as a totally different project. This means that all the nice review and merge feature that github and git offer cannot be used, but the merge needs to be made "by hand", comparing the source trees etc. I think it would make more sense to fork[1] from radargun's master[2] and apply all these nice feature on a fork. What do you think? >>>> >>>> Cheers, >>>> Mircea >>>> >>>> [1] http://help.github.com/fork-a-repo/ >>>> [2] https://github.com/radargun/radargun >>>> >>>> >>>> On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: >>>>> Hi all, >>>>> >>>>> I have added a new subsection >>>>> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >>>>> "Shared software resources" section on the Cloud-TM Wiki, in order to >>>>> share information about the RadargunTPCC benchmark. >>>>> >>>>> RadargunTPCC benchmark is an implementation of the Transaction Profiles >>>>> defined in the TPC Benchmark C specification [1] within Radargun >>>>> Benchamark [2], and it has been designed with the purpose of executing >>>>> the TPC-C benchmark against Infinispan. >>>>> >>>>> The source code is available at this link: >>>>> https://github.com/cloudtm/RadargunTPCC >>>>> >>>>> Regards, >>>>> Sebastiano Peluso >>>>> >>>>> >>>>> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >>>>> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >>>>> >>>>> ------------------------------------------------------------------------------ >>>>> Simplify data backup and recovery for your virtual environment with vRanger. >>>>> Installation's a snap, and flexible recovery options mean your data is safe, >>>>> secure and there when you need it. Data protection magic? >>>>> Nope - It's vRanger. Get your free trial download today. >>>>> http://p.sf.net/sfu/quest-sfdev2dev >>>>> _______________________________________________ >>>>> Cloudtm-discussion mailing list >>>>> Cloudtm-discussion(a)lists.sourceforge.net >>>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>>> >>>> ------------------------------------------------------------------------------ >>>> All of the data generated in your IT infrastructure is seriously valuable. >>>> Why? It contains a definitive record of application performance, security >>>> threats, fraudulent activity, and more. Splunk takes this data and makes >>>> sense of it. IT sense. And common sense. >>>> http://p.sf.net/sfu/splunk-d2d-c2________________________________________... >>>> Cloudtm-discussion mailing list >>>> Cloudtm-discussion(a)lists.sourceforge.net >>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>> > Begin forwarded message: > From: Manik Surtani <msurtani(a)redhat.com&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 8 July 2011 11:39:37 GMT+01:00 > To: Mircea Markus <mircea.markus(a)jboss.com&gt; > Cc: Dan Berindei <dberinde(a)redhat.com&gt;, Manik Surtani <manik(a)jboss.org&gt; > > Could you give them some pointers on how to make this easier to integrate? > > Sent from my mobile phone > > On 8 Jul 2011, at 11:36, Mircea Markus <mircea.markus(a)jboss.com&gt; wrote: > >> FYI, I reviewed the code and is far from being integrable in its current shape. The idea is really nice though, hope they'll update and we can at least take some of it. >> >> Begin forwarded message: >> >>> From: Mircea Markus <mircea.markus(a)jboss.com&gt; >>> Date: 8 July 2011 11:33:41 GMT+01:00 >>> To: cloudtm-discussion(a)lists.sourceforge.net >>> Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark >>> Reply-To: cloudtm-discussion(a)lists.sourceforge.net >>> >>> Hi Sebastiano, >>> >>> Very interesting stuff! >>> I think this is a nice approach of benchmarking transactions with a controlled degree contention. This is a feature currently missing in Radargun, but very important especially considering the stuff you guys are doing and the locking improvements we have scheduled for 5.1 release. >>> Do you consider contributing this back to Radargun? >>> >>> Here are some observations I have after looking at the code: >>> - The code should not be in the PutGetStressor, as this is used for benchmarking a web-session-like access pattern which we don't want to replace. It might make more sense to be placed in a stage of its own, e.g. org.radargun.stages.TpccBenchmarkStage >>> - "isPrimary" argument of TPCCTerminal.choiceTransaction is true if the node is a jgroups coordinator. This has some limitations: >>> - other data grid providers than Infinispan might not have an "coordinator" concept, and this might be difficult to infer >>> - ATM only the coordinator does writes (on multiple threads). IMO this is too restricting as it doesn't induce deadlock situations that might appear when data is written by multiple nodes. Wouldn't it make more sense to spread the writes over all the nodes? >>> In order to overcome the dependency on jgroup's coordinator, you can use instead the AbstractDistStage.slaveIndex. This is a Radargun-level view of the benchmark cluster, independent of the benchmarked product >>> - PutGetStressor compares Strings based on the == operator, which is incorrect. e.g. e.getClass().getName() == "org.infinispan.CacheException" >>> - As an design suggestion, I think it would make sense to have an TpccTransaction interface with an executeTransaction(CacheWrapper cacheWrapper) methods, with specialized implementations: NewOrderTransaction, PayementTransaction, OrderStatusTransaction. Basically to move all the code TPCCTerminal.executeTransaction into these three classes. Such an approach would reduce coupling between the transactions and most importantly would offer a simple way of adding new tpcc transactions to the system, by implementing the TpccTransaction interface. >>> - class javadoc are the onese generater by IDEA >>> - adding .idea directory in github might cause troubles for people using IDEA and forking your repo; I think the dir should be removed >>> >>> >>> Another observation is about the way the project is managed in github: it is not a fork of Infinispan, but it was downloaded and committed as a totally different project. This means that all the nice review and merge feature that github and git offer cannot be used, but the merge needs to be made "by hand", comparing the source trees etc. I think it would make more sense to fork[1] from radargun's master[2] and apply all these nice feature on a fork. What do you think? >>> >>> Cheers, >>> Mircea >>> >>> [1] http://help.github.com/fork-a-repo/ >>> [2] https://github.com/radargun/radargun >>> >>> >>> On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: >>>> Hi all, >>>> >>>> I have added a new subsection >>>> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >>>> "Shared software resources" section on the Cloud-TM Wiki, in order to >>>> share information about the RadargunTPCC benchmark. >>>> >>>> RadargunTPCC benchmark is an implementation of the Transaction Profiles >>>> defined in the TPC Benchmark C specification [1] within Radargun >>>> Benchamark [2], and it has been designed with the purpose of executing >>>> the TPC-C benchmark against Infinispan. >>>> >>>> The source code is available at this link: >>>> https://github.com/cloudtm/RadargunTPCC >>>> >>>> Regards, >>>> Sebastiano Peluso >>>> >>>> >>>> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >>>> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >>>> >>>> ------------------------------------------------------------------------------ >>>> Simplify data backup and recovery for your virtual environment with vRanger. >>>> Installation's a snap, and flexible recovery options mean your data is safe, >>>> secure and there when you need it. Data protection magic? >>>> Nope - It's vRanger. Get your free trial download today. >>>> http://p.sf.net/sfu/quest-sfdev2dev >>>> _______________________________________________ >>>> Cloudtm-discussion mailing list >>>> Cloudtm-discussion(a)lists.sourceforge.net >>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>> >>> ------------------------------------------------------------------------------ >>> All of the data generated in your IT infrastructure is seriously valuable. >>> Why? It contains a definitive record of application performance, security >>> threats, fraudulent activity, and more. Splunk takes this data and makes >>> sense of it. IT sense. And common sense. >>> http://p.sf.net/sfu/splunk-d2d-c2________________________________________... >>> Cloudtm-discussion mailing list >>> Cloudtm-discussion(a)lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >> Begin forwarded message: > From: Mircea Markus <mircea.markus(a)jboss.com&gt; > Subject: Fwd: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 8 July 2011 11:36:29 GMT+01:00 > To: Dan Berindei <dberinde(a)redhat.com&gt;, Manik Surtani <manik(a)jboss.org&gt; > > FYI, I reviewed the code and is far from being integrable in its current shape. The idea is really nice though, hope they'll update and we can at least take some of it. > > Begin forwarded message: > >> From: Mircea Markus <mircea.markus(a)jboss.com&gt; >> Date: 8 July 2011 11:33:41 GMT+01:00 >> To: cloudtm-discussion(a)lists.sourceforge.net >> Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark >> Reply-To: cloudtm-discussion(a)lists.sourceforge.net >> >> Hi Sebastiano, >> >> Very interesting stuff! >> I think this is a nice approach of benchmarking transactions with a controlled degree contention. This is a feature currently missing in Radargun, but very important especially considering the stuff you guys are doing and the locking improvements we have scheduled for 5.1 release. >> Do you consider contributing this back to Radargun? >> >> Here are some observations I have after looking at the code: >> - The code should not be in the PutGetStressor, as this is used for benchmarking a web-session-like access pattern which we don't want to replace. It might make more sense to be placed in a stage of its own, e.g. org.radargun.stages.TpccBenchmarkStage >> - "isPrimary" argument of TPCCTerminal.choiceTransaction is true if the node is a jgroups coordinator. This has some limitations: >> - other data grid providers than Infinispan might not have an "coordinator" concept, and this might be difficult to infer >> - ATM only the coordinator does writes (on multiple threads). IMO this is too restricting as it doesn't induce deadlock situations that might appear when data is written by multiple nodes. Wouldn't it make more sense to spread the writes over all the nodes? >> In order to overcome the dependency on jgroup's coordinator, you can use instead the AbstractDistStage.slaveIndex. This is a Radargun-level view of the benchmark cluster, independent of the benchmarked product >> - PutGetStressor compares Strings based on the == operator, which is incorrect. e.g. e.getClass().getName() == "org.infinispan.CacheException" >> - As an design suggestion, I think it would make sense to have an TpccTransaction interface with an executeTransaction(CacheWrapper cacheWrapper) methods, with specialized implementations: NewOrderTransaction, PayementTransaction, OrderStatusTransaction. Basically to move all the code TPCCTerminal.executeTransaction into these three classes. Such an approach would reduce coupling between the transactions and most importantly would offer a simple way of adding new tpcc transactions to the system, by implementing the TpccTransaction interface. >> - class javadoc are the onese generater by IDEA >> - adding .idea directory in github might cause troubles for people using IDEA and forking your repo; I think the dir should be removed >> >> >> Another observation is about the way the project is managed in github: it is not a fork of Infinispan, but it was downloaded and committed as a totally different project. This means that all the nice review and merge feature that github and git offer cannot be used, but the merge needs to be made "by hand", comparing the source trees etc. I think it would make more sense to fork[1] from radargun's master[2] and apply all these nice feature on a fork. What do you think? >> >> Cheers, >> Mircea >> >> [1] http://help.github.com/fork-a-repo/ >> [2] https://github.com/radargun/radargun >> >> >> On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: >>> Hi all, >>> >>> I have added a new subsection >>> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >>> "Shared software resources" section on the Cloud-TM Wiki, in order to >>> share information about the RadargunTPCC benchmark. >>> >>> RadargunTPCC benchmark is an implementation of the Transaction Profiles >>> defined in the TPC Benchmark C specification [1] within Radargun >>> Benchamark [2], and it has been designed with the purpose of executing >>> the TPC-C benchmark against Infinispan. >>> >>> The source code is available at this link: >>> https://github.com/cloudtm/RadargunTPCC >>> >>> Regards, >>> Sebastiano Peluso >>> >>> >>> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >>> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >>> >>> ------------------------------------------------------------------------------ >>> Simplify data backup and recovery for your virtual environment with vRanger. >>> Installation's a snap, and flexible recovery options mean your data is safe, >>> secure and there when you need it. Data protection magic? >>> Nope - It's vRanger. Get your free trial download today. >>> http://p.sf.net/sfu/quest-sfdev2dev >>> _______________________________________________ >>> Cloudtm-discussion mailing list >>> Cloudtm-discussion(a)lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >> >> ------------------------------------------------------------------------------ >> All of the data generated in your IT infrastructure is seriously valuable. >> Why? It contains a definitive record of application performance, security >> threats, fraudulent activity, and more. Splunk takes this data and makes >> sense of it. IT sense. And common sense. >> http://p.sf.net/sfu/splunk-d2d-c2________________________________________... >> Cloudtm-discussion mailing list >> Cloudtm-discussion(a)lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion > Begin forwarded message: > From: Mircea Markus <mircea.markus(a)jboss.com&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 8 July 2011 11:33:41 GMT+01:00 > To: cloudtm-discussion(a)lists.sourceforge.net > > Hi Sebastiano, > > Very interesting stuff! > I think this is a nice approach of benchmarking transactions with a controlled degree contention. This is a feature currently missing in Radargun, but very important especially considering the stuff you guys are doing and the locking improvements we have scheduled for 5.1 release. > Do you consider contributing this back to Radargun? > > Here are some observations I have after looking at the code: > - The code should not be in the PutGetStressor, as this is used for benchmarking a web-session-like access pattern which we don't want to replace. It might make more sense to be placed in a stage of its own, e.g. org.radargun.stages.TpccBenchmarkStage > - "isPrimary" argument of TPCCTerminal.choiceTransaction is true if the node is a jgroups coordinator. This has some limitations: > - other data grid providers than Infinispan might not have an "coordinator" concept, and this might be difficult to infer > - ATM only the coordinator does writes (on multiple threads). IMO this is too restricting as it doesn't induce deadlock situations that might appear when data is written by multiple nodes. Wouldn't it make more sense to spread the writes over all the nodes? > In order to overcome the dependency on jgroup's coordinator, you can use instead the AbstractDistStage.slaveIndex. This is a Radargun-level view of the benchmark cluster, independent of the benchmarked product > - PutGetStressor compares Strings based on the == operator, which is incorrect. e.g. e.getClass().getName() == "org.infinispan.CacheException" > - As an design suggestion, I think it would make sense to have an TpccTransaction interface with an executeTransaction(CacheWrapper cacheWrapper) methods, with specialized implementations: NewOrderTransaction, PayementTransaction, OrderStatusTransaction. Basically to move all the code TPCCTerminal.executeTransaction into these three classes. Such an approach would reduce coupling between the transactions and most importantly would offer a simple way of adding new tpcc transactions to the system, by implementing the TpccTransaction interface. > - class javadoc are the onese generater by IDEA > - adding .idea directory in github might cause troubles for people using IDEA and forking your repo; I think the dir should be removed > > > Another observation is about the way the project is managed in github: it is not a fork of Infinispan, but it was downloaded and committed as a totally different project. This means that all the nice review and merge feature that github and git offer cannot be used, but the merge needs to be made "by hand", comparing the source trees etc. I think it would make more sense to fork[1] from radargun's master[2] and apply all these nice feature on a fork. What do you think? > > Cheers, > Mircea > > [1] http://help.github.com/fork-a-repo/ > [2] https://github.com/radargun/radargun > > > On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: >> Hi all, >> >> I have added a new subsection >> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >> "Shared software resources" section on the Cloud-TM Wiki, in order to >> share information about the RadargunTPCC benchmark. >> >> RadargunTPCC benchmark is an implementation of the Transaction Profiles >> defined in the TPC Benchmark C specification [1] within Radargun >> Benchamark [2], and it has been designed with the purpose of executing >> the TPC-C benchmark against Infinispan. >> >> The source code is available at this link: >> https://github.com/cloudtm/RadargunTPCC >> >> Regards, >> Sebastiano Peluso >> >> >> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >> >> ------------------------------------------------------------------------------ >> Simplify data backup and recovery for your virtual environment with vRanger. >> Installation's a snap, and flexible recovery options mean your data is safe, >> secure and there when you need it. Data protection magic? >> Nope - It's vRanger. Get your free trial download today. >> http://p.sf.net/sfu/quest-sfdev2dev >> _______________________________________________ >> Cloudtm-discussion mailing list >> Cloudtm-discussion(a)lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion > Begin forwarded message: > From: Mircea Markus <mircea.markus(a)jboss.com&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 8 July 2011 11:33:41 GMT+01:00 > To: cloudtm-discussion(a)lists.sourceforge.net > Reply-To: cloudtm-discussion(a)lists.sourceforge.net > > Hi Sebastiano, > > Very interesting stuff! > I think this is a nice approach of benchmarking transactions with a controlled degree contention. This is a feature currently missing in Radargun, but very important especially considering the stuff you guys are doing and the locking improvements we have scheduled for 5.1 release. > Do you consider contributing this back to Radargun? > > Here are some observations I have after looking at the code: > - The code should not be in the PutGetStressor, as this is used for benchmarking a web-session-like access pattern which we don't want to replace. It might make more sense to be placed in a stage of its own, e.g. org.radargun.stages.TpccBenchmarkStage > - "isPrimary" argument of TPCCTerminal.choiceTransaction is true if the node is a jgroups coordinator. This has some limitations: > - other data grid providers than Infinispan might not have an "coordinator" concept, and this might be difficult to infer > - ATM only the coordinator does writes (on multiple threads). IMO this is too restricting as it doesn't induce deadlock situations that might appear when data is written by multiple nodes. Wouldn't it make more sense to spread the writes over all the nodes? > In order to overcome the dependency on jgroup's coordinator, you can use instead the AbstractDistStage.slaveIndex. This is a Radargun-level view of the benchmark cluster, independent of the benchmarked product > - PutGetStressor compares Strings based on the == operator, which is incorrect. e.g. e.getClass().getName() == "org.infinispan.CacheException" > - As an design suggestion, I think it would make sense to have an TpccTransaction interface with an executeTransaction(CacheWrapper cacheWrapper) methods, with specialized implementations: NewOrderTransaction, PayementTransaction, OrderStatusTransaction. Basically to move all the code TPCCTerminal.executeTransaction into these three classes. Such an approach would reduce coupling between the transactions and most importantly would offer a simple way of adding new tpcc transactions to the system, by implementing the TpccTransaction interface. > - class javadoc are the onese generater by IDEA > - adding .idea directory in github might cause troubles for people using IDEA and forking your repo; I think the dir should be removed > > > Another observation is about the way the project is managed in github: it is not a fork of Infinispan, but it was downloaded and committed as a totally different project. This means that all the nice review and merge feature that github and git offer cannot be used, but the merge needs to be made "by hand", comparing the source trees etc. I think it would make more sense to fork[1] from radargun's master[2] and apply all these nice feature on a fork. What do you think? > > Cheers, > Mircea > > [1] http://help.github.com/fork-a-repo/ > [2] https://github.com/radargun/radargun > > > On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: >> Hi all, >> >> I have added a new subsection >> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >> "Shared software resources" section on the Cloud-TM Wiki, in order to >> share information about the RadargunTPCC benchmark. >> >> RadargunTPCC benchmark is an implementation of the Transaction Profiles >> defined in the TPC Benchmark C specification [1] within Radargun >> Benchamark [2], and it has been designed with the purpose of executing >> the TPC-C benchmark against Infinispan. >> >> The source code is available at this link: >> https://github.com/cloudtm/RadargunTPCC >> >> Regards, >> Sebastiano Peluso >> >> >> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >> >> ------------------------------------------------------------------------------ >> Simplify data backup and recovery for your virtual environment with vRanger. >> Installation's a snap, and flexible recovery options mean your data is safe, >> secure and there when you need it. Data protection magic? >> Nope - It's vRanger. Get your free trial download today. >> http://p.sf.net/sfu/quest-sfdev2dev >> _______________________________________________ >> Cloudtm-discussion mailing list >> Cloudtm-discussion(a)lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion > > ------------------------------------------------------------------------------ > All of the data generated in your IT infrastructure is seriously valuable. > Why? It contains a definitive record of application performance, security > threats, fraudulent activity, and more. Splunk takes this data and makes > sense of it. IT sense. And common sense. > http://p.sf.net/sfu/splunk-d2d-c2________________________________________... > Cloudtm-discussion mailing list > Cloudtm-discussion(a)lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion Begin forwarded message: > From: Mircea Markus <mircea.markus(a)jboss.com&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 28 June 2011 13:36:40 GMT+01:00 > To: Manik Surtani <manik(a)jboss.org&gt; > Cc: cloudtm-discussion(a)lists.sourceforge.net > > Didn't look at it in details but looks promising. I'll take a look later on this week. > > On 22 Jun 2011, at 18:44, Manik Surtani wrote: >> Cool. I'm adding Mircea in cc, who maintains the upstream Radargun. Mircea, does it make sense to have this in Radargun upstream? >> >> On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: >> >>> Hi all, >>> >>> I have added a new subsection >>> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >>> "Shared software resources" section on the Cloud-TM Wiki, in order to >>> share information about the RadargunTPCC benchmark. >>> >>> RadargunTPCC benchmark is an implementation of the Transaction Profiles >>> defined in the TPC Benchmark C specification [1] within Radargun >>> Benchamark [2], and it has been designed with the purpose of executing >>> the TPC-C benchmark against Infinispan. >>> >>> The source code is available at this link: >>> https://github.com/cloudtm/RadargunTPCC >>> >>> Regards, >>> Sebastiano Peluso >>> >>> >>> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >>> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >>> >>> ------------------------------------------------------------------------------ >>> Simplify data backup and recovery for your virtual environment with vRanger. >>> Installation's a snap, and flexible recovery options mean your data is safe, >>> secure and there when you need it. Data protection magic? >>> Nope - It's vRanger. Get your free trial download today. >>> http://p.sf.net/sfu/quest-sfdev2dev >>> _______________________________________________ >>> Cloudtm-discussion mailing list >>> Cloudtm-discussion(a)lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >> >> -- >> Manik Surtani >> manik(a)jboss.org >> twitter.com/maniksurtani >> >> Lead, Infinispan >> http://www.infinispan.org >> >> >> > Begin forwarded message: > From: Mircea Markus <mircea.markus(a)jboss.com&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 28 June 2011 13:36:40 GMT+01:00 > To: Manik Surtani <manik(a)jboss.org&gt; > Cc: cloudtm-discussion(a)lists.sourceforge.net > Reply-To: cloudtm-discussion(a)lists.sourceforge.net > > Didn't look at it in details but looks promising. I'll take a look later on this week. > > On 22 Jun 2011, at 18:44, Manik Surtani wrote: >> Cool. I'm adding Mircea in cc, who maintains the upstream Radargun. Mircea, does it make sense to have this in Radargun upstream? >> >> On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: >> >>> Hi all, >>> >>> I have added a new subsection >>> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >>> "Shared software resources" section on the Cloud-TM Wiki, in order to >>> share information about the RadargunTPCC benchmark. >>> >>> RadargunTPCC benchmark is an implementation of the Transaction Profiles >>> defined in the TPC Benchmark C specification [1] within Radargun >>> Benchamark [2], and it has been designed with the purpose of executing >>> the TPC-C benchmark against Infinispan. >>> >>> The source code is available at this link: >>> https://github.com/cloudtm/RadargunTPCC >>> >>> Regards, >>> Sebastiano Peluso >>> >>> >>> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >>> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >>> >>> ------------------------------------------------------------------------------ >>> Simplify data backup and recovery for your virtual environment with vRanger. >>> Installation's a snap, and flexible recovery options mean your data is safe, >>> secure and there when you need it. Data protection magic? >>> Nope - It's vRanger. Get your free trial download today. >>> http://p.sf.net/sfu/quest-sfdev2dev >>> _______________________________________________ >>> Cloudtm-discussion mailing list >>> Cloudtm-discussion(a)lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >> >> -- >> Manik Surtani >> manik(a)jboss.org >> twitter.com/maniksurtani >> >> Lead, Infinispan >> http://www.infinispan.org >> >> >> > > > ------------------------------------------------------------------------------ > All of the data generated in your IT infrastructure is seriously valuable. > Why? It contains a definitive record of application performance, security > threats, fraudulent activity, and more. Splunk takes this data and makes > sense of it. IT sense. And common sense. > http://p.sf.net/sfu/splunk-d2d-c2 > _______________________________________________ > Cloudtm-discussion mailing list > Cloudtm-discussion(a)lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion Begin forwarded message: > From: Mircea Markus <mircea.markus(a)jboss.com&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 27 June 2011 17:01:01 GMT+01:00 > To: Paolo Romano <romano(a)inesc-id.pt&gt; > Cc: Sebastiano Peluso <peluso(a)gsd.inesc-id.pt&gt;, mmarkus(a)redhat.com, Manik Surtani <manik(a)jboss.org&gt; > > Thanks wiling to contribute this! > I've received my account details and I'll take a look this/next week at it. > > Cheers, > Mircea > > On 22 Jun 2011, at 19:41, Paolo Romano wrote: > >> I am afraid that Mircea will not be able to log-in to the wiki. >> >> Sebastiano, please, can you create an account for him? >> >> Cheers, >> >> Paolo >> >> >> On 6/22/11 6:44 PM, Manik Surtani wrote: >>> Cool. I'm adding Mircea in cc, who maintains the upstream Radargun. Mircea, does it make sense to have this in Radargun upstream? >>> >>> On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: >>> >>>> Hi all, >>>> >>>> I have added a new subsection >>>> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >>>> "Shared software resources" section on the Cloud-TM Wiki, in order to >>>> share information about the RadargunTPCC benchmark. >>>> >>>> RadargunTPCC benchmark is an implementation of the Transaction Profiles >>>> defined in the TPC Benchmark C specification [1] within Radargun >>>> Benchamark [2], and it has been designed with the purpose of executing >>>> the TPC-C benchmark against Infinispan. >>>> >>>> The source code is available at this link: >>>> https://github.com/cloudtm/RadargunTPCC >>>> >>>> Regards, >>>> Sebastiano Peluso >>>> >>>> >>>> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >>>> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >>>> >>>> ------------------------------------------------------------------------------ >>>> Simplify data backup and recovery for your virtual environment with vRanger. >>>> Installation's a snap, and flexible recovery options mean your data is safe, >>>> secure and there when you need it. Data protection magic? >>>> Nope - It's vRanger. Get your free trial download today. >>>> http://p.sf.net/sfu/quest-sfdev2dev >>>> _______________________________________________ >>>> Cloudtm-discussion mailing list >>>> Cloudtm-discussion(a)lists.sourceforge.net >>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>> -- >>> Manik Surtani >>> manik(a)jboss.org >>> twitter.com/maniksurtani >>> >>> Lead, Infinispan >>> http://www.infinispan.org >>> >>> >>> >>> >>> ------------------------------------------------------------------------------ >>> Simplify data backup and recovery for your virtual environment with vRanger. >>> Installation's a snap, and flexible recovery options mean your data is safe, >>> secure and there when you need it. Data protection magic? >>> Nope - It's vRanger. Get your free trial download today. >>> http://p.sf.net/sfu/quest-sfdev2dev >>> _______________________________________________ >>> Cloudtm-discussion mailing list >>> Cloudtm-discussion(a)lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >> >> > Begin forwarded message: > From: Mircea Markus <mircea.markus(a)jboss.com&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 27 June 2011 17:01:01 GMT+01:00 > To: Paolo Romano <romano(a)inesc-id.pt&gt; > Cc: Sebastiano Peluso <peluso(a)gsd.inesc-id.pt&gt;, mmarkus(a)redhat.com, Manik Surtani <manik(a)jboss.org&gt; > > Thanks wiling to contribute this! > I've received my account details and I'll take a look this/next week at it. > > Cheers, > Mircea > > On 22 Jun 2011, at 19:41, Paolo Romano wrote: > >> I am afraid that Mircea will not be able to log-in to the wiki. >> >> Sebastiano, please, can you create an account for him? >> >> Cheers, >> >> Paolo >> >> >> On 6/22/11 6:44 PM, Manik Surtani wrote: >>> Cool. I'm adding Mircea in cc, who maintains the upstream Radargun. Mircea, does it make sense to have this in Radargun upstream? >>> >>> On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: >>> >>>> Hi all, >>>> >>>> I have added a new subsection >>>> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >>>> "Shared software resources" section on the Cloud-TM Wiki, in order to >>>> share information about the RadargunTPCC benchmark. >>>> >>>> RadargunTPCC benchmark is an implementation of the Transaction Profiles >>>> defined in the TPC Benchmark C specification [1] within Radargun >>>> Benchamark [2], and it has been designed with the purpose of executing >>>> the TPC-C benchmark against Infinispan. >>>> >>>> The source code is available at this link: >>>> https://github.com/cloudtm/RadargunTPCC >>>> >>>> Regards, >>>> Sebastiano Peluso >>>> >>>> >>>> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >>>> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >>>> >>>> ------------------------------------------------------------------------------ >>>> Simplify data backup and recovery for your virtual environment with vRanger. >>>> Installation's a snap, and flexible recovery options mean your data is safe, >>>> secure and there when you need it. Data protection magic? >>>> Nope - It's vRanger. Get your free trial download today. >>>> http://p.sf.net/sfu/quest-sfdev2dev >>>> _______________________________________________ >>>> Cloudtm-discussion mailing list >>>> Cloudtm-discussion(a)lists.sourceforge.net >>>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >>> -- >>> Manik Surtani >>> manik(a)jboss.org >>> twitter.com/maniksurtani >>> >>> Lead, Infinispan >>> http://www.infinispan.org >>> >>> >>> >>> >>> ------------------------------------------------------------------------------ >>> Simplify data backup and recovery for your virtual environment with vRanger. >>> Installation's a snap, and flexible recovery options mean your data is safe, >>> secure and there when you need it. Data protection magic? >>> Nope - It's vRanger. Get your free trial download today. >>> http://p.sf.net/sfu/quest-sfdev2dev >>> _______________________________________________ >>> Cloudtm-discussion mailing list >>> Cloudtm-discussion(a)lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >> >> > Begin forwarded message: > From: Paolo Romano <romano(a)inesc-id.pt&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 22 June 2011 19:41:00 GMT+01:00 > To: Sebastiano Peluso <peluso(a)gsd.inesc-id.pt&gt;, mmarkus(a)redhat.com > Cc: Manik Surtani <manik(a)jboss.org&gt; > > I am afraid that Mircea will not be able to log-in to the wiki. > > Sebastiano, please, can you create an account for him? > > Cheers, > > Paolo > > > On 6/22/11 6:44 PM, Manik Surtani wrote: >> Cool. I'm adding Mircea in cc, who maintains the upstream Radargun. Mircea, does it make sense to have this in Radargun upstream? >> >> On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: >> >>> Hi all, >>> >>> I have added a new subsection >>> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >>> "Shared software resources" section on the Cloud-TM Wiki, in order to >>> share information about the RadargunTPCC benchmark. >>> >>> RadargunTPCC benchmark is an implementation of the Transaction Profiles >>> defined in the TPC Benchmark C specification [1] within Radargun >>> Benchamark [2], and it has been designed with the purpose of executing >>> the TPC-C benchmark against Infinispan. >>> >>> The source code is available at this link: >>> https://github.com/cloudtm/RadargunTPCC >>> >>> Regards, >>> Sebastiano Peluso >>> >>> >>> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >>> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >>> >>> ------------------------------------------------------------------------------ >>> Simplify data backup and recovery for your virtual environment with vRanger. >>> Installation's a snap, and flexible recovery options mean your data is safe, >>> secure and there when you need it. Data protection magic? >>> Nope - It's vRanger. Get your free trial download today. >>> http://p.sf.net/sfu/quest-sfdev2dev >>> _______________________________________________ >>> Cloudtm-discussion mailing list >>> Cloudtm-discussion(a)lists.sourceforge.net >>> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion >> -- >> Manik Surtani >> manik(a)jboss.org >> twitter.com/maniksurtani >> >> Lead, Infinispan >> http://www.infinispan.org >> >> >> >> >> ------------------------------------------------------------------------------ >> Simplify data backup and recovery for your virtual environment with vRanger. >> Installation's a snap, and flexible recovery options mean your data is safe, >> secure and there when you need it. Data protection magic? >> Nope - It's vRanger. Get your free trial download today. >> http://p.sf.net/sfu/quest-sfdev2dev >> _______________________________________________ >> Cloudtm-discussion mailing list >> Cloudtm-discussion(a)lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion > > Begin forwarded message: > From: Manik Surtani <manik(a)jboss.org&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 22 June 2011 18:44:50 GMT+01:00 > To: cloudtm-discussion(a)lists.sourceforge.net > Cc: Mircea Markus <mmarkus(a)redhat.com&gt; > > Cool. I'm adding Mircea in cc, who maintains the upstream Radargun. Mircea, does it make sense to have this in Radargun upstream? > > On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: > >> Hi all, >> >> I have added a new subsection >> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >> "Shared software resources" section on the Cloud-TM Wiki, in order to >> share information about the RadargunTPCC benchmark. >> >> RadargunTPCC benchmark is an implementation of the Transaction Profiles >> defined in the TPC Benchmark C specification [1] within Radargun >> Benchamark [2], and it has been designed with the purpose of executing >> the TPC-C benchmark against Infinispan. >> >> The source code is available at this link: >> https://github.com/cloudtm/RadargunTPCC >> >> Regards, >> Sebastiano Peluso >> >> >> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >> >> ------------------------------------------------------------------------------ >> Simplify data backup and recovery for your virtual environment with vRanger. >> Installation's a snap, and flexible recovery options mean your data is safe, >> secure and there when you need it. Data protection magic? >> Nope - It's vRanger. Get your free trial download today. >> http://p.sf.net/sfu/quest-sfdev2dev >> _______________________________________________ >> Cloudtm-discussion mailing list >> Cloudtm-discussion(a)lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion > > -- > Manik Surtani > manik(a)jboss.org > twitter.com/maniksurtani > > Lead, Infinispan > http://www.infinispan.org > > > Begin forwarded message: > From: Manik Surtani <manik(a)jboss.org&gt; > Subject: Re: [Cloudtm-discussion] RadargunTPCC benchmark > Date: 22 June 2011 18:44:50 GMT+01:00 > To: cloudtm-discussion(a)lists.sourceforge.net > Cc: Mircea Markus <mmarkus(a)redhat.com&gt; > Reply-To: cloudtm-discussion(a)lists.sourceforge.net > > Cool. I'm adding Mircea in cc, who maintains the upstream Radargun. Mircea, does it make sense to have this in Radargun upstream? > > On 22 Jun 2011, at 18:06, Sebastiano Peluso wrote: > >> Hi all, >> >> I have added a new subsection >> (https://cloudtm.ist.utl.pt/wiki/index.php/RadargunTPCC), within the >> "Shared software resources" section on the Cloud-TM Wiki, in order to >> share information about the RadargunTPCC benchmark. >> >> RadargunTPCC benchmark is an implementation of the Transaction Profiles >> defined in the TPC Benchmark C specification [1] within Radargun >> Benchamark [2], and it has been designed with the purpose of executing >> the TPC-C benchmark against Infinispan. >> >> The source code is available at this link: >> https://github.com/cloudtm/RadargunTPCC >> >> Regards, >> Sebastiano Peluso >> >> >> [1] http://www.tpc.org/tpcc/spec/tpcc_current.pdf >> [2] http://sourceforge.net/apps/trac/radargun/wiki/WikiStart >> >> ------------------------------------------------------------------------------ >> Simplify data backup and recovery for your virtual environment with vRanger. >> Installation's a snap, and flexible recovery options mean your data is safe, >> secure and there when you need it. Data protection magic? >> Nope - It's vRanger. Get your free trial download today. >> http://p.sf.net/sfu/quest-sfdev2dev >> _______________________________________________ >> Cloudtm-discussion mailing list >> Cloudtm-discussion(a)lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion > > -- > Manik Surtani > manik(a)jboss.org > twitter.com/maniksurtani > > Lead, Infinispan > http://www.infinispan.org > > > > > ------------------------------------------------------------------------------ > Simplify data backup and recovery for your virtual environment with vRanger. > Installation's a snap, and flexible recovery options mean your data is safe, > secure and there when you need it. Data protection magic? > Nope - It's vRanger. Get your free trial download today. > http://p.sf.net/sfu/quest-sfdev2dev > _______________________________________________ > Cloudtm-discussion mailing list > Cloudtm-discussion(a)lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/cloudtm-discussion