[ 
https://issues.apache.org/jira/browse/KAFKA-5237?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16009868#comment-16009868
 ] 

ASF GitHub Bot commented on KAFKA-5237:
---------------------------------------

GitHub user deinspanjer opened a pull request:

    https://github.com/apache/kafka/pull/3052

    KAFKA-5237 - change logging of termination message to use stderr instead of 
stdout

    

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/deinspanjer/kafka 
improvement/simple_consumer_shell_logging

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/kafka/pull/3052.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #3052
    
----
commit 01a8fef379bd83e38e4938b34a1272d3347bb075
Author: Daniel Einspanjer <deinspan...@gmail.com>
Date:   2017-05-14T22:09:34Z

    KAFKA-5237 - change logging of termination message to use stderr instead of 
stdout

----


> SimpleConsumerShell logs terminating message to stdout instead of stderr
> ------------------------------------------------------------------------
>
>                 Key: KAFKA-5237
>                 URL: https://issues.apache.org/jira/browse/KAFKA-5237
>             Project: Kafka
>          Issue Type: Improvement
>          Components: clients
>    Affects Versions: 0.10.2.1
>            Reporter: Daniel Einspanjer
>            Priority: Trivial
>
> The SimpleConsumerShell has one big advantage over the standard 
> kafka-console-consumer client, it supports the --no-wait-at-logend parameter 
> which lets you script its use without having to rely on a timeout or dealing 
> with the exception and stacktrace thrown by said timeout.
> Unfortunately, when you use this option, it will write a termination message 
> to stdout when it is finished.  This means if you are using it to dump the 
> contents of a topic to a file, you get an extra line.
> This pull request just changes that one line to call System.err.println 
> instead of println.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to