[GitHub] [kafka] dajac commented on pull request #9945: KAFKA-12212; Bump Metadata API version to remove `ClusterAuthorizedOperations` fields

2021-01-22 Thread GitBox


dajac commented on pull request #9945:
URL: https://github.com/apache/kafka/pull/9945#issuecomment-765218202


   Failed tests are not related:
   * Build / JDK 8 / 
org.apache.kafka.clients.consumer.KafkaConsumerTest.testCloseWithTimeUnit()
   * Build / JDK 8 / 
org.apache.kafka.clients.consumer.internals.FetcherTest.testEarlierOffsetResetArrivesLate()
   
   Merging to trunk.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [kafka] dajac commented on pull request #9945: KAFKA-12212; Bump Metadata API version to remove `ClusterAuthorizedOperations` fields

2021-01-21 Thread GitBox


dajac commented on pull request #9945:
URL: https://github.com/apache/kafka/pull/9945#issuecomment-764726538


   @chia7712 Thanks for your comments. I have updated the PR to address them.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




[GitHub] [kafka] dajac commented on pull request #9945: KAFKA-12212; Bump Metadata API version to remove `ClusterAuthorizedOperations` fields

2021-01-21 Thread GitBox


dajac commented on pull request #9945:
URL: https://github.com/apache/kafka/pull/9945#issuecomment-764726538


   @chia7712 Thanks for your comments. I have updated the PR to address them.



This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org