[GitHub] [kafka] junrao commented on pull request #13990: KAFKA-14937: Refactoring for client code to reduce boilerplate

2023-08-14 Thread via GitHub


junrao commented on PR #13990:
URL: https://github.com/apache/kafka/pull/13990#issuecomment-1677476713

   @kirktrue : JDK 17 build was aborted. Did that pass locally?


-- 
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.

To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org

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



[GitHub] [kafka] junrao commented on pull request #13990: KAFKA-14937: Refactoring for client code to reduce boilerplate

2023-08-11 Thread via GitHub


junrao commented on PR #13990:
URL: https://github.com/apache/kafka/pull/13990#issuecomment-1675203219

   @kirktrue : Thanks for the update. If there is clear evidence that test 
failures are unrelated, it's reasonable to merge the PR. The latest build has 3 
red failures due to a connect checkstyle issue tough.


-- 
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.

To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org

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



[GitHub] [kafka] junrao commented on pull request #13990: KAFKA-14937: Refactoring for client code to reduce boilerplate

2023-08-01 Thread via GitHub


junrao commented on PR #13990:
URL: https://github.com/apache/kafka/pull/13990#issuecomment-1661118945

   @kirktrue : For consistent infra issues, we could file an infra jira for the 
ASF infra team to help. For transient test failures, if we could identify when 
the test failure first occurred and correlate it with the PR that might have 
introduced it, we could file a jira and ping the PR author to take a look.


-- 
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.

To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org

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



[GitHub] [kafka] junrao commented on pull request #13990: KAFKA-14937: Refactoring for client code to reduce boilerplate

2023-07-31 Thread via GitHub


junrao commented on PR #13990:
URL: https://github.com/apache/kafka/pull/13990#issuecomment-1659202647

   Thanks @kirktrue. @satishd : Are you aware of any transient failures in 
:storage:unitTest? Thanks.


-- 
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.

To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org

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



[GitHub] [kafka] junrao commented on pull request #13990: KAFKA-14937: Refactoring for client code to reduce boilerplate

2023-07-25 Thread via GitHub


junrao commented on PR #13990:
URL: https://github.com/apache/kafka/pull/13990#issuecomment-1650204432

   @kirktrue : For JDK 11 tests, it seems that there were 2 checkstyle failures 
`:clients:checkstyleTest`and `:server-common:checkstyleTest`.
   
   


-- 
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.

To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org

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



[GitHub] [kafka] junrao commented on pull request #13990: KAFKA-14937: Refactoring for client code to reduce boilerplate

2023-07-21 Thread via GitHub


junrao commented on PR #13990:
URL: https://github.com/apache/kafka/pull/13990#issuecomment-1646244882

   @kirktrue : In the latest run, it seems that most test failures are from 
java 11. I'd suggest looking at the jenkins's log to see  the first test that 
failed. Sometimes, a failed test could lead to other test failures, if not 
designed well.


-- 
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.

To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org

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



[GitHub] [kafka] junrao commented on pull request #13990: KAFKA-14937: Refactoring for client code to reduce boilerplate

2023-07-20 Thread via GitHub


junrao commented on PR #13990:
URL: https://github.com/apache/kafka/pull/13990#issuecomment-1644221461

   @kirktrue : It seems there were 4 test failures for jdk 11. But the tests 
for jdk 17 and 20 were aborted. Do you know why?


-- 
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.

To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org

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



[GitHub] [kafka] junrao commented on pull request #13990: KAFKA-14937: Refactoring for client code to reduce boilerplate

2023-07-18 Thread via GitHub


junrao commented on PR #13990:
URL: https://github.com/apache/kafka/pull/13990#issuecomment-1640481893

   @kirktrue : Thanks for the updated PR. It LGTM. However, it seems there are 
built failures?


-- 
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.

To unsubscribe, e-mail: jira-unsubscr...@kafka.apache.org

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