incubator-cassandra-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Gary Dusbabek <gdusba...@gmail.com>
Subject Re: /var/tmp in FailureDetector
Date Wed, 20 Oct 2010 12:55:53 GMT
Yes, we should generate it in the right temp directory.  That method
is an implementation of an interface method (FailureDetectorMBean),
meant to be invoked by JMX, which is why no other code calls it.

Gary.

On Wed, Oct 20, 2010 at 03:48, aaron morton <aaron@thelastpickle.com> wrote:
> 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 it to use
File.createTempFile() .
>
> Also I could not find any use of the  dumpInterArrivalTimes(InetAddress ep) overload.
Anyone know if it should be kept?
>
> thanks
> Aaron
>
>

Mime
View raw message