kafka-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ij...@apache.org
Subject [kafka] branch trunk updated: MINOR: Remove unused commitSync in ConsoleConsumer (#5845)
Date Fri, 26 Oct 2018 19:18:59 GMT
This is an automated email from the ASF dual-hosted git repository.

ijuma pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/kafka.git


The following commit(s) were added to refs/heads/trunk by this push:
     new 5106179  MINOR: Remove unused commitSync in ConsoleConsumer (#5845)
5106179 is described below

commit 51061792ca6120eeffe9845826962b49dc7939a1
Author: Ismael Juma <ismael@juma.me.uk>
AuthorDate: Fri Oct 26 12:18:50 2018 -0700

    MINOR: Remove unused commitSync in ConsoleConsumer (#5845)
    
    Dead code is confusing.
    
    Reviewers: Colin Patrick McCabe <colin@cmccabe.xyz>
---
 core/src/main/scala/kafka/tools/ConsoleConsumer.scala | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/core/src/main/scala/kafka/tools/ConsoleConsumer.scala b/core/src/main/scala/kafka/tools/ConsoleConsumer.scala
index 06705d5..be50be1 100755
--- a/core/src/main/scala/kafka/tools/ConsoleConsumer.scala
+++ b/core/src/main/scala/kafka/tools/ConsoleConsumer.scala
@@ -454,9 +454,6 @@ object ConsoleConsumer extends Logging {
       this.consumer.close()
     }
 
-    def commitSync() {
-      this.consumer.commitSync()
-    }
   }
 }
 


Mime
View raw message