[GitHub] kafka pull request #3938: HOTFIX: ConsumerGroupCommand - Offset and partitio...

2017-09-22 Thread asfgit
Github user asfgit closed the pull request at:

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


---


[GitHub] kafka pull request #3938: HOTFIX: ConsumerGroupCommand - Offset and partitio...

2017-09-21 Thread eu657
GitHub user eu657 opened a pull request:

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

HOTFIX: ConsumerGroupCommand - Offset and partition numbers are not 
converted to long and int correctly

Running the command line with --from-file option causes the following 
exception:

java.lang.ClassCastException: java.lang.String cannot be cast to 
java.lang.Integer

Reason: asInstanceOf used for the conversion.

Also, unit test is using --to-earliest and --from-file together when 
executing the test. This is executing --to-earliest option only and ignoring 
--from-file option. Since the preparation part is also using --to-earliest to 
create the file, this unit test passes. Fixed the unit test too.



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

$ git pull https://github.com/eu657/kafka eu657-patch-1

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

https://github.com/apache/kafka/pull/3938.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 #3938






---