> On Nov. 22, 2014, 2:24 a.m., Joel Koshy wrote: > > Looks good, although the patch does not apply - can you rebase; check that > > it builds properly and then we can close this out.
This is a little wired.. both you and Gwen said it does not apply but when I apply the patch (patch -p1 --dry-run < patch.file) it works fine. Will verify that one last time and commit. - Guozhang ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/28268/#review62721 ----------------------------------------------------------- On Nov. 20, 2014, 12:23 a.m., Guozhang Wang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/28268/ > ----------------------------------------------------------- > > (Updated Nov. 20, 2014, 12:23 a.m.) > > > Review request for kafka. > > > Bugs: KAFKA-1624 > https://issues.apache.org/jira/browse/KAFKA-1624 > > > Repository: kafka > > > Description > ------- > > Compiles with Java 1.6/7/8 > > > Diffs > ----- > > bin/kafka-run-class.sh 36c742b67a7259fa35c3ed862f7ccc4673b69d9f > bin/windows/kafka-run-class.bat 8e9780e2eb74a35a72678715cccc5c09b151d0ba > build.gradle 11eb11355efddacf62d61690ad13b9c82a200230 > gradle.properties 5d3155fd4461438d8b2ec4faa9534cc2383d4951 > scala.gradle 6adf9af7dbbe71e68a07b387c3854d8c9ad339e0 > > Diff: https://reviews.apache.org/r/28268/diff/ > > > Testing > ------- > > > Thanks, > > Guozhang Wang > >
