You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@geode.apache.org by "upthewaterspout (GitHub)" <gi...@apache.org> on 2018/12/11 00:33:11 UTC

[GitHub] [geode-benchmarks] upthewaterspout opened pull request #21: GEODE-6172: Configure geode properties for benchmarks

Setting some geode properties on all benchmarks to improve the performance and enable additional statistics.

Refactoring the location of JVMParameters and the util package.

[ Full content available at: https://github.com/apache/geode-benchmarks/pull/21 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode-benchmarks] upthewaterspout closed pull request #21: GEODE-6172: Configure geode properties for benchmarks

Posted by "upthewaterspout (GitHub)" <gi...@apache.org>.
[ pull request closed by upthewaterspout ]

[ Full content available at: https://github.com/apache/geode-benchmarks/pull/21 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode-benchmarks] nabarunnag commented on pull request #21: GEODE-6172: Configure geode properties for benchmarks

Posted by "nabarunnag (GitHub)" <gi...@apache.org>.
Do we configure clients too ? Is that within the scope of the ticket

[ Full content available at: https://github.com/apache/geode-benchmarks/pull/21 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode-benchmarks] upthewaterspout commented on pull request #21: GEODE-6172: Configure geode properties for benchmarks

Posted by "upthewaterspout (GitHub)" <gi...@apache.org>.
We definitely don't configure the same properties on the client - but I guess we probably want some of these - like enabling time stats. I'll look into creating separate serverProperties(), clientProperties(), and locatorProperties() methods.

[ Full content available at: https://github.com/apache/geode-benchmarks/pull/21 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org

[GitHub] [geode-benchmarks] nabarunnag commented on pull request #21: GEODE-6172: Configure geode properties for benchmarks

Posted by "nabarunnag (GitHub)" <gi...@apache.org>.
Locator taking serverProperties seems a bit out of place. Possible refactor?

[ Full content available at: https://github.com/apache/geode-benchmarks/pull/21 ]
This message was relayed via gitbox.apache.org for notifications@geode.apache.org