[GitHub] kafka pull request #4042: [KAFKA-5835][CLIENTS] CommitFailedException except...

2017-10-08 Thread rekhajoshm
GitHub user rekhajoshm opened a pull request: https://github.com/apache/kafka/pull/4042 [KAFKA-5835][CLIENTS] CommitFailedException exception versus KafkaConsumer flow The invalid offset/out of range offset is captured in IllegalArgumentException/RunTimeException. The

[GitHub] kafka pull request #2273: KAFKA-4555: Using Hamcrest for expressive intent i...

2017-02-06 Thread rekhajoshm
Github user rekhajoshm closed the pull request at: https://github.com/apache/kafka/pull/2273 --- 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 is

[GitHub] kafka pull request #45: KAFKA-1972: JMXTool multiple attributes

2016-12-20 Thread rekhajoshm
Github user rekhajoshm closed the pull request at: https://github.com/apache/kafka/pull/45 --- 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 is

[GitHub] kafka pull request #49: KAFKA-269: run-test.sh async test

2016-12-20 Thread rekhajoshm
Github user rekhajoshm closed the pull request at: https://github.com/apache/kafka/pull/49 --- 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 is

[GitHub] kafka pull request #2273: KAFKA-4555: Using Hamcrest for expressive intent i...

2016-12-18 Thread rekhajoshm
GitHub user rekhajoshm opened a pull request: https://github.com/apache/kafka/pull/2273 KAFKA-4555: Using Hamcrest for expressive intent in tests - Adding hamcrest in gradle files - Using hamcrest in couple of tests - SourceTaskOffsetCommitterTest, MetadataTest You can merge

[GitHub] kafka pull request #2222: KAFKA-4500: code corrections on client module

2016-12-06 Thread rekhajoshm
GitHub user rekhajoshm opened a pull request: https://github.com/apache/kafka/pull/ KAFKA-4500: code corrections on client module - Removed redundant modifiers, not needed String.format() - Removed unnecessary semicolon, additional assignment, inlined return - Added few

[GitHub] kafka pull request #1697: KAFKA-4014; Using Collections.singletonList instea...

2016-08-06 Thread rekhajoshm
Github user rekhajoshm closed the pull request at: https://github.com/apache/kafka/pull/1697 --- 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 is

[GitHub] kafka pull request #1696: KAFKA-4010; ConfigDef.toRst() to have grouped sect...

2016-08-02 Thread rekhajoshm
GitHub user rekhajoshm reopened a pull request: https://github.com/apache/kafka/pull/1696 KAFKA-4010; ConfigDef.toRst() to have grouped sections with dependents info - Added sort method with group order - Added dependents info You can merge this pull request into a Git

[GitHub] kafka pull request #1696: KAFKA-4010; ConfigDef.toRst() to have grouped sect...

2016-08-02 Thread rekhajoshm
Github user rekhajoshm closed the pull request at: https://github.com/apache/kafka/pull/1696 --- 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 is

[GitHub] kafka pull request #1697: KAFKA-4014; Using Collections.singletonList instea...

2016-08-02 Thread rekhajoshm
GitHub user rekhajoshm opened a pull request: https://github.com/apache/kafka/pull/1697 KAFKA-4014; Using Collections.singletonList instead of Arrays.asList for single arguments Using Collections.singletonList instead of Arrays.asList for single arguments You can merge this pull

[GitHub] kafka pull request #1696: KAFKA-4010; ConfigDef.toRst() to have grouped sect...

2016-08-02 Thread rekhajoshm
GitHub user rekhajoshm opened a pull request: https://github.com/apache/kafka/pull/1696 KAFKA-4010; ConfigDef.toRst() to have grouped sections with dependents info - Added sort method with group order - Added dependents info You can merge this pull request into a Git

[GitHub] kafka pull request #1601: KAFKA-3905; Handle invalid collection of topics, p...

2016-07-09 Thread rekhajoshm
GitHub user rekhajoshm opened a pull request: https://github.com/apache/kafka/pull/1601 KAFKA-3905; Handle invalid collection of topics, patterns on subscription for list of topics, with patterns, and with assignments KAFKA-3905: Handling null/empty topics and collections, patterns

[GitHub] kafka pull request #1561: KAFKA-3905; Handle invalid collection of topics, p...

2016-07-09 Thread rekhajoshm
Github user rekhajoshm closed the pull request at: https://github.com/apache/kafka/pull/1561 --- 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 is

[GitHub] kafka pull request #1561: KAFKA-3905:remove null from subscribed topics in K...

2016-06-27 Thread rekhajoshm
GitHub user rekhajoshm opened a pull request: https://github.com/apache/kafka/pull/1561 KAFKA-3905:remove null from subscribed topics in KafkaConsumer#subscribe KAFKA-3905:remove null from subscribed topics in KafkaConsumer#subscribe You can merge this pull request into a Git

[GitHub] kafka pull request #1559: KAFKA-3844; Sort configuration items in log

2016-06-27 Thread rekhajoshm
GitHub user rekhajoshm opened a pull request: https://github.com/apache/kafka/pull/1559 KAFKA-3844; Sort configuration items in log KAFKA-3844; Sort configuration items in log You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] kafka pull request #1469: KAFKA-724; Send, receive buffer size set if not -1

2016-06-04 Thread rekhajoshm
GitHub user rekhajoshm opened a pull request: https://github.com/apache/kafka/pull/1469 KAFKA-724; Send, receive buffer size set if not -1 PR #50 closed. Fixed, after rebase on trunk. You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] kafka pull request #50: KAFKA-724: auto socket buffer set

2016-06-04 Thread rekhajoshm
Github user rekhajoshm closed the pull request at: https://github.com/apache/kafka/pull/50 --- 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 is

[GitHub] kafka pull request: KAFKA-3771; Improving Kafka core code

2016-05-30 Thread rekhajoshm
GitHub user rekhajoshm opened a pull request: https://github.com/apache/kafka/pull/1451 KAFKA-3771; Improving Kafka core code - Used flatMap instead of map and flatten - Use isEmpty, NonEmpty, isDefined as appropriate - Used head, keys and keySet where appropriate - Used

[GitHub] kafka pull request: [KAFKA-3756] [Javadoc] Improving Javadoc

2016-05-29 Thread rekhajoshm
Github user rekhajoshm closed the pull request at: https://github.com/apache/kafka/pull/1432 --- 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 is

[GitHub] kafka pull request: [KAFKA-3765] Minor Code style corrections

2016-05-27 Thread rekhajoshm
GitHub user rekhajoshm opened a pull request: https://github.com/apache/kafka/pull/1442 [KAFKA-3765] Minor Code style corrections [KAFKA-3765] Minor Code style corrections: Removed explicit returns, not needed parentheses, corrected variables, removed unused imports You can

[GitHub] kafka pull request: [KAFKA-3756] [Javadoc] Improving Javadoc

2016-05-25 Thread rekhajoshm
GitHub user rekhajoshm opened a pull request: https://github.com/apache/kafka/pull/1432 [KAFKA-3756] [Javadoc] Improving Javadoc [KAFKA-3756] [Javadoc] Improving Javadoc You can merge this pull request into a Git repository by running: $ git pull https://github.com/rekhajoshm

[GitHub] kafka pull request: KAFKA-1621 : Standardize --messages option

2015-07-20 Thread rekhajoshm
Github user rekhajoshm closed the pull request at: https://github.com/apache/kafka/pull/58 --- 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 is

[GitHub] kafka pull request: KAFKA-1621 : Standardize --messages option

2015-04-27 Thread rekhajoshm
GitHub user rekhajoshm opened a pull request: https://github.com/apache/kafka/pull/58 KAFKA-1621 : Standardize --messages option As per review comments from @nehanarkhede Thanks. You can merge this pull request into a Git repository by running: $ git pull https://github.com

[GitHub] kafka pull request: KAFKA-2098: gradle files

2015-04-06 Thread rekhajoshm
GitHub user rekhajoshm opened a pull request: https://github.com/apache/kafka/pull/54 KAFKA-2098: gradle files gradle files, tiny footprint.lets have it in.thanks You can merge this pull request into a Git repository by running: $ git pull https://github.com/rekhajoshm/kafka

[GitHub] kafka pull request: KAFKA-724: auto socket buffer set

2015-02-22 Thread rekhajoshm
GitHub user rekhajoshm opened a pull request: https://github.com/apache/kafka/pull/50 KAFKA-724: auto socket buffer set KAFKA-724: Allow automatic socket.send.buffer from operating system If socket.receive.buffer.bytes/socket.send.buffer.bytes set to non-zero/-1, the OS

[GitHub] kafka pull request: KAFKA-269: run-test.sh async test

2015-02-22 Thread rekhajoshm
GitHub user rekhajoshm opened a pull request: https://github.com/apache/kafka/pull/49 KAFKA-269: run-test.sh async test KAFKA-269: run-test.sh async test You can merge this pull request into a Git repository by running: $ git pull https://github.com/rekhajoshm/kafka KAFKA-269

[GitHub] kafka pull request: KAFKA-1545: KafkaHealthcheck.register failure

2015-02-22 Thread rekhajoshm
GitHub user rekhajoshm opened a pull request: https://github.com/apache/kafka/pull/48 KAFKA-1545: KafkaHealthcheck.register failure KAFKA-1545: java.net.InetAddress.getLocalHost in KafkaHealthcheck.register may fail on some irregular hostnames You can merge this pull request into

[GitHub] kafka pull request: KAFKA-1545: KafkaHealthcheck.register failure

2015-02-22 Thread rekhajoshm
Github user rekhajoshm closed the pull request at: https://github.com/apache/kafka/pull/47 --- 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 is

[GitHub] kafka pull request: KAFKA-1545: KafkaHealthcheck.register failure

2015-02-22 Thread rekhajoshm
GitHub user rekhajoshm opened a pull request: https://github.com/apache/kafka/pull/47 KAFKA-1545: KafkaHealthcheck.register failure KAFKA-1545: java.net.InetAddress.getLocalHost in KafkaHealthcheck.register may fail on some irregular hostnames You can merge this pull request into

[GitHub] kafka pull request: KAFKA-1621 : Standardize --messages option

2015-02-22 Thread rekhajoshm
GitHub user rekhajoshm opened a pull request: https://github.com/apache/kafka/pull/46 KAFKA-1621 : Standardize --messages option KAFKA-1621: Standardize --messages option in perf scripts You can merge this pull request into a Git repository by running: $ git pull https

[GitHub] kafka pull request: KAFKA-1972: JMXTool multiple attributes

2015-02-22 Thread rekhajoshm
GitHub user rekhajoshm opened a pull request: https://github.com/apache/kafka/pull/45 KAFKA-1972: JMXTool multiple attributes KAFKA-1972: JMX Tool output for CSV format does not handle attributes with comma in their value You can merge this pull request into a Git repository by