chukwa-user mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Ariel Rabkin <asrab...@gmail.com>
Subject Re: alerts with chukwa
Date Mon, 26 Oct 2009 23:09:53 GMT
Hey hey.  Patches are great.  And you're absolutely right; that's a bug.

The right thing to do is to open a JIRA -- go to
https://issues.apache.org/jira/browse/CHUKWA and "create new issue".
Attach the patch to that.

--Ari

On Mon, Oct 26, 2009 at 4:05 PM, Thushara Wijeratna <thushw@gmail.com> wrote:
> thanks Ariel.
>
> SocketTeeWriter seems very useful. In testing this, I may have
> found/fixed a minor bug as well.
> inside SockteTeeWriter.Tee.setup(), the output steam should be created
> early on in order for parser exceptions to be properly sent to the
> client.
> the patch is to simply move the creation of the output stream up like this:
>
>        in = new BufferedReader(new InputStreamReader(sock.getInputStream()));
>        out = new DataOutputStream(sock.getOutputStream());
>
> i can submit a patch if you let me know the process.
>
> thanks,
> thushara
>


-- 
Ari Rabkin asrabkin@gmail.com
UC Berkeley Computer Science Department

Mime
View raw message