[GitHub] kafka pull request #4229: MINOR: Code cleanup

2017-11-17 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/4229 MINOR: Code cleanup - Used arrays directly instead of constructing List while getting storeNames - Used proper Integer identifier in the Log. *More detailed description of your change

[GitHub] kafka pull request #4010: KAFKA-5909; Removed the source jars from classpath

2017-10-03 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/4010 KAFKA-5909; Removed the source jars from classpath Executing CLI tools don't require source jars so removed it from the classpath. You can merge this pull request into a Git repository by running

[GitHub] kafka pull request #3871: KAFKA-5909; Removed the source jars from classpath...

2017-10-03 Thread Kamal15
Github user Kamal15 closed the pull request at: https://github.com/apache/kafka/pull/3871 ---

[GitHub] kafka pull request #3871: KAFKA-5909; Removed the source jars from classpath...

2017-09-15 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/3871 KAFKA-5909; Removed the source jars from classpath while executing CL… …I tools. - redundant `for` loops removed. - I did this change with assumption that there are no priorities

[GitHub] kafka pull request #3815: KAFKA-5846; Used Singleton NoOpConsumerRebalanceLi...

2017-09-08 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/3815 KAFKA-5846; Used Singleton NoOpConsumerRebalanceListener in the Kafka… …Consumer#subscribe call. - I haven't removed the internal class 'NoOpConsumerRebalanceListener', as it might

[GitHub] kafka-site pull request #73: Add coding guidelines for Streams API

2017-09-04 Thread Kamal15
Github user Kamal15 commented on a diff in the pull request: https://github.com/apache/kafka-site/pull/73#discussion_r136894052 --- Diff: coding-guide.html --- @@ -102,6 +102,47 @@ We should attempt to maintain API compatibility when possible, though at this point

[GitHub] kafka pull request #3741: KAFKA-5762; LogContext used to capture the clientI...

2017-08-25 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/3741 KAFKA-5762; LogContext used to capture the clientId implicitly in the… … logs. You can merge this pull request into a Git repository by running: $ git pull https://github.com/Kamal15/kafka

[GitHub] kafka pull request #3465: MINOR: Fixed the invalid link to Log Compaction

2017-08-25 Thread Kamal15
Github user Kamal15 closed the pull request at: https://github.com/apache/kafka/pull/3465 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] kafka pull request #3695: MINOR: Typographical error corrected in the Stream...

2017-08-19 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/3695 MINOR: Typographical error corrected in the StreamsBuilder Javadoc. You can merge this pull request into a Git repository by running: $ git pull https://github.com/Kamal15/kafka typo_error

[GitHub] kafka pull request #3481: MINOR: Configuration name corrected in the upgrade...

2017-07-22 Thread Kamal15
Github user Kamal15 closed the pull request at: https://github.com/apache/kafka/pull/3481 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] kafka pull request #3520: MINOR: Typo error corrected in the KStream Javadoc...

2017-07-11 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/3520 MINOR: Typo error corrected in the KStream Javadoc. You can merge this pull request into a Git repository by running: $ git pull https://github.com/Kamal15/kafka stream_doc Alternatively you

[GitHub] kafka pull request #3481: MINOR: Configuration name corrected in the upgrade...

2017-07-03 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/3481 MINOR: Configuration name corrected in the upgrade docs. You can merge this pull request into a Git repository by running: $ git pull https://github.com/Kamal15/kafka upgrade_doc

[GitHub] kafka pull request #3475: MINOR: Unit test added for InFlightRequests class.

2017-07-01 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/3475 MINOR: Unit test added for InFlightRequests class. You can merge this pull request into a Git repository by running: $ git pull https://github.com/Kamal15/kafka unittest Alternatively you

[GitHub] kafka pull request #3467: MINOR: InFlightRequests#isEmpty(node) method corre...

2017-06-30 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/3467 MINOR: InFlightRequests#isEmpty(node) method corrected. - In clearAll method, get operation is removed. - variable name `requestTimeout` changed to `requestTimeoutMs` for clarity You can merge

[GitHub] kafka pull request #3465: MINOR: Fixed the invalid link to Log Compaction

2017-06-30 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/3465 MINOR: Fixed the invalid link to Log Compaction - variable name updated in the doc example. You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] kafka pull request #3365: MINOR: Unused logger removed.

2017-06-16 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/3365 MINOR: Unused logger removed. You can merge this pull request into a Git repository by running: $ git pull https://github.com/Kamal15/kafka logger Alternatively you can review and apply

[GitHub] kafka pull request #3350: MINOR: Redirected the creation of new Thread to Ka...

2017-06-15 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/3350 MINOR: Redirected the creation of new Thread to KafkaThread class. - removed `newThread` & `daemonThread` methods from the Utils. You can merge this pull request into a Git repository by run

[GitHub] kafka pull request #3349: MINOR: Used Kafka native constants in KafkaLog4jAp...

2017-06-15 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/3349 MINOR: Used Kafka native constants in KafkaLog4jAppender You can merge this pull request into a Git repository by running: $ git pull https://github.com/Kamal15/kafka log4j2 Alternatively

[GitHub] kafka pull request #3337: MINOR: Delegated the creation of new Thread to Kaf...

2017-06-15 Thread Kamal15
Github user Kamal15 closed the pull request at: https://github.com/apache/kafka/pull/3337 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] kafka pull request #3337: MINOR: Delegated the creation of new Thread to Kaf...

2017-06-14 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/3337 MINOR: Delegated the creation of new Thread to KafkaThread class. You can merge this pull request into a Git repository by running: $ git pull https://github.com/Kamal15/kafka cleanup

[GitHub] kafka pull request #3079: MINOR: Removed the redundant code snippets.

2017-05-30 Thread Kamal15
Github user Kamal15 closed the pull request at: https://github.com/apache/kafka/pull/3079 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] kafka pull request #3079: MINOR: Removed the redundant code snippets.

2017-05-17 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/3079 MINOR: Removed the redundant code snippets. - Removed the unwanted null check in the hashCode and equals method. You can merge this pull request into a Git repository by running: $ git pull

[GitHub] kafka pull request #2885: MINOR: Logger parameter corrected.

2017-04-21 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/2885 MINOR: Logger parameter corrected. - typo error corrected (spelling) You can merge this pull request into a Git repository by running: $ git pull https://github.com/Kamal15/kafka log

[GitHub] kafka pull request #2709: MINOR: Map `mkString` format updated to default ja...

2017-03-20 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/2709 MINOR: Map `mkString` format updated to default java format This is a minor change but it helps to improve the log readability. You can merge this pull request into a Git repository by running

[GitHub] kafka pull request #2700: MINOR: Precondition check corrected

2017-03-17 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/2700 MINOR: Precondition check corrected You can merge this pull request into a Git repository by running: $ git pull https://github.com/Kamal15/kafka precondition Alternatively you can review

[GitHub] kafka pull request #2667: MINOR: Used in-built TimeUnit conversion function

2017-03-10 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/2667 MINOR: Used in-built TimeUnit conversion function You can merge this pull request into a Git repository by running: $ git pull https://github.com/Kamal15/kafka timeunit Alternatively you can

[GitHub] kafka pull request #2666: MINOR: Used in-built TimeUnit conversion function

2017-03-10 Thread Kamal15
Github user Kamal15 closed the pull request at: https://github.com/apache/kafka/pull/2666 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] kafka pull request #2666: MINOR: Used in-built TimeUnit conversion function

2017-03-10 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/2666 MINOR: Used in-built TimeUnit conversion function You can merge this pull request into a Git repository by running: $ git pull https://github.com/Kamal15/kafka trunk Alternatively you can

[GitHub] kafka pull request #2578: MINOR: Variable name corrected

2017-02-21 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/2578 MINOR: Variable name corrected - Java doc updated - Used final modifier where-ever it's applicable - Unnecessary type-cast removed - cleanup You can merge this pull request into a Git

[GitHub] kafka pull request #2345: MINOR: ConfigDef `parseType` exception message upd...

2017-01-11 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/2345 MINOR: ConfigDef `parseType` exception message updated. - When Kafka configurations are provided programmatically, for the below configuration incorrect error message gets printed

[GitHub] kafka pull request #2340: MINOR: ConfigDef `parseType` exception message upd...

2017-01-11 Thread Kamal15
Github user Kamal15 closed the pull request at: https://github.com/apache/kafka/pull/2340 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] kafka pull request #2340: MINOR: ConfigDef `parseType` exception message upd...

2017-01-10 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/2340 MINOR: ConfigDef `parseType` exception message updated. - When Kafka configurations are provided programmatically, for the below configuration incorrect error message gets printed

[GitHub] kafka pull request #2306: MINOR: PartitionInfo toString corrected

2017-01-03 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/2306 MINOR: PartitionInfo toString corrected - Removed the extra ',' character while printing the replicas / in-sync replicas identifier You can merge this pull request into a Git repository by running

[GitHub] kafka pull request #1438: MINOR: Kafka And ZooKeeper Stop Scripts doesn't re...

2016-09-14 Thread Kamal15
Github user Kamal15 closed the pull request at: https://github.com/apache/kafka/pull/1438 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] kafka pull request: MINOR: Kafka And ZooKeeper Stop Scripts doesn'...

2016-05-27 Thread Kamal15
GitHub user Kamal15 opened a pull request: https://github.com/apache/kafka/pull/1438 MINOR: Kafka And ZooKeeper Stop Scripts doesn't return PID In Centos 6.5, 'ps -ax | grep java' command doesn't returns the full process name. So, the command doesn't returns the PID even Kafka