[GitHub] kafka pull request #1883: Use pre-calculated value in trace message

2016-09-19 Thread lukezaparaniuk
GitHub user lukezaparaniuk opened a pull request: https://github.com/apache/kafka/pull/1883 Use pre-calculated value in trace message Replaced the math.min(size, sizeInBytes) call with the count variable that has already been calculated within the writeTo method. Replaced string

[GitHub] kafka pull request #1877: Use pre-calculated value in trace message

2016-09-19 Thread lukezaparaniuk
Github user lukezaparaniuk closed the pull request at: https://github.com/apache/kafka/pull/1877 --- 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

[GitHub] kafka pull request #1877: Use pre-calculated value in trace message

2016-09-18 Thread lukezaparaniuk
GitHub user lukezaparaniuk opened a pull request: https://github.com/apache/kafka/pull/1877 Use pre-calculated value in trace message Replaced the math.min(size, sizeInBytes) call with the count variable that has already been calculated within the writeTo method You can merge this

[GitHub] kafka pull request #1876: Matched method comment argument name to actual arg...

2016-09-18 Thread lukezaparaniuk
GitHub user lukezaparaniuk opened a pull request: https://github.com/apache/kafka/pull/1876 Matched method comment argument name to actual argument name Changed the lowerBound argument reference in the summary comment of the translateOffset method to match the actual argument name