No.  That needs to be fixed.  I'll do that right now.

On Mon, Jun 15, 2009 at 11:04 AM, Cheng Zhang<[email protected]> wrote:
> In Chukwa-5 patch, the file offset is also included in md5 calculation. It 
> means that the same file could be added many times as long as the offsets are 
> different. Is it expected behavior?
>
> Thanks,
> Cheng
>
>> -----Original Message-----
>> From: Ariel Rabkin [mailto:[email protected]]
>> Sent: Monday, June 15, 2009 10:45 AM
>> To: [email protected]
>> Subject: Re: what does Adaptor.getStreamName() do?
>>
>> As of CHUKWA-5, this is no longer true.  The agent keeps a unique name
>> for each adaptor. (By default, the MD5 of its arguments).  So I think
>> getStreamName() can go away.
>>
>> On Mon, Jun 15, 2009 at 9:51 AM, Cheng Zhang<[email protected]> wrote:
>> > Currently ChukwaAgent uses this method to avoid tailing the same file
>> > more than once.
>> >
>> > Thanks,
>> > Cheng
>> >
>> >> -----Original Message-----
>> >> From: Ariel Rabkin [mailto:[email protected]]
>> >> Sent: Sunday, June 14, 2009 9:35 PM
>> >> To: [email protected]
>> >> Subject: what does Adaptor.getStreamName() do?
>> >>
>> >> We have this method adaptor.getStreamName().  But hardly anybody calls
>> >> it. As near as I can tell, it's only used for debugging.
>> >>
>> >> I would like to rename it to something like debuggingName(), or else
>> >> remove it entirely.
>> >> Also, AbstractAdaptor.toString() should invoke this method.
>> >>
>> >> --Ari
>> >>
>> >> --
>> >> Ari Rabkin [email protected]
>> >> UC Berkeley Computer Science Department
>> >
>>
>>
>>
>> --
>> Ari Rabkin [email protected]
>> UC Berkeley Computer Science Department
>



-- 
Ari Rabkin [email protected]
UC Berkeley Computer Science Department

Reply via email to