Re: /var/tmp in FailureDetector

2010-10-21 Thread Aaron Morton
To quick for me :) Aaron On 21 Oct 2010, at 17:52, Jonathan Ellis jbel...@gmail.com wrote: Done in r1025822 On Wed, Oct 20, 2010 at 12:54 PM, Gary Dusbabek gdusba...@gmail.com wrote: You're right! It looks like dead code that should be removed. Gary. On Wed, Oct 20, 2010 at 12:50,

/var/tmp in FailureDetector

2010-10-20 Thread aaron morton
I was reading through some code and noticed the following in FailureDetector.dumpInterArrivealTimes() FileOutputStream fos = new FileOutputStream(/var/tmp/output- + System.currentTimeMillis() + .dat, true); If this is meant to be cross platform I'm happy to create a bug and change

Re: /var/tmp in FailureDetector

2010-10-20 Thread aaron morton
I should have mentioned the FailureDetectorMBean only has the parameterless dumpInterArrivalTimes(). The overload that takes InetAddress is not available through JMX. A On 21 Oct 2010, at 01:55, Gary Dusbabek wrote: Yes, we should generate it in the right temp directory. That method is an

Re: /var/tmp in FailureDetector

2010-10-20 Thread Jonathan Ellis
Done in r1025822 On Wed, Oct 20, 2010 at 12:54 PM, Gary Dusbabek gdusba...@gmail.com wrote: You're right!  It looks like dead code that should be removed. Gary. On Wed, Oct 20, 2010 at 12:50, aaron morton aa...@thelastpickle.com wrote: I should have mentioned the FailureDetectorMBean only