logging-flume icon indicating copy to clipboard operation
logging-flume copied to clipboard

FLUME-3258: Add ScribeCLIClient for scribe source

Open JohnZZGithub opened this issue 6 years ago • 5 comments

JohnZZGithub avatar Jul 14 '18 05:07 JohnZZGithub

Can one of the admins verify this patch?

asfgit avatar Aug 17 '18 13:08 asfgit

It's kind of hard to write unittest for core logic as it uses thrift. I tested it manually, it worked well with both sync/async client. Please review it when you get a chance, thanks.

JohnZZGithub avatar Dec 14 '18 07:12 JohnZZGithub

@turcsanyip Any chance to review it? It shouldn't impact any existing code. Thanks

JohnZZGithub avatar Jun 20 '19 20:06 JohnZZGithub

@mpercy Could you please help with the review? Thanks

JohnZZGithub avatar May 01 '20 20:05 JohnZZGithub

Please see the attachment for UTs result

FLUME-3258-UTs.txt

JohnZZGithub avatar Aug 06 '20 02:08 JohnZZGithub