[GitHub] activemq-artemis issue #1395: ARTEMIS-1269 New actor framework

2017-07-11 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1395 I don't want to make a framework. This is just to diminish GC on a few executors. Instead of creating a new runnable everybtime we pass in the data. --- If

[GitHub] activemq-artemis issue #1395: ARTEMIS-1269 New actor framework

2017-07-11 Thread mtaylor
Github user mtaylor commented on the issue: https://github.com/apache/activemq-artemis/pull/1395 @clebertsuconic I have two comments on this. Firstly, what is the goal of this patch. Is the aim to reduce the GC overhead by reusing Runnables? If so surely this could be done

[GitHub] activemq-artemis issue #1395: ARTEMIS-1269 New actor framework

2017-07-10 Thread michaelandrepearce
Github user michaelandrepearce commented on the issue: https://github.com/apache/activemq-artemis/pull/1395 don't mean to be a pain, but why don't we use an existing framework, akka/vert.x? instead of making our own framework we will have to maintain? --- If your project is set up

[GitHub] activemq-artemis issue #1395: ARTEMIS-1269 New actor framework

2017-07-10 Thread clebertsuconic
Github user clebertsuconic commented on the issue: https://github.com/apache/activemq-artemis/pull/1395 There are a lot of possibilities for this... Example: https://github.com/clebertsuconic/activemq-artemis/commit/116fb580a97514fac2a78598cc970d78717bb23d