[GitHub] [kafka] chia7712 commented on pull request #8657: KAFKA-8334 Make sure the thread which tries to complete delayed reque…

[GitBox]

chia7712 commented on pull request #8657:
URL: https://github.com/apache/kafka/pull/8657#issuecomment-647233460


   > This is a second run of the system tests. 
http://testing.confluent.io/confluent-kafka-branch-builder-system-test-results/?prefix=2020-06-19--001.1592614513--chia7712--fix_8334_avoid_deadlock--142a6c4c0/
   
   ```ConsumeBenchTest``` pass on second run.
   
   
   ```StreamsOptimizedTest``` still fail and the error message is 
   ```
   Exception in thread 
"StreamsOptimizedTest-239c8c2e-5338-4a4c-a009-729ca21e1673-StreamThread-1" 
java.lang.IllegalStateException: Tried to lookup lag for unknown task 2_0
        at 
org.apache.kafka.streams.processor.internals.assignment.ClientState.lagFor(ClientState.java:306)
        at 
java.util.Comparator.lambda$comparingLong$6043328a$1(Comparator.java:511)
        at 
java.util.Comparator.lambda$thenComparing$36697e65$1(Comparator.java:216)
        at java.util.TreeMap.put(TreeMap.java:552)
        at java.util.TreeSet.add(TreeSet.java:255)
        at java.util.AbstractCollection.addAll(AbstractCollection.java:344)
        at java.util.TreeSet.addAll(TreeSet.java:312)
        at 
org.apache.kafka.streams.processor.internals.StreamsPartitionAssignor.getPreviousTasksByLag(StreamsPartitionAssignor.java:1250)
        at 
org.apache.kafka.streams.processor.internals.StreamsPartitionAssignor.assignTasksToThreads(StreamsPartitionAssignor.java:1164)
        at 
org.apache.kafka.streams.processor.internals.StreamsPartitionAssignor.computeNewAssignment(StreamsPartitionAssignor.java:920)
        at 
org.apache.kafka.streams.processor.internals.StreamsPartitionAssignor.assign(StreamsPartitionAssignor.java:391)
        at 
org.apache.kafka.clients.consumer.internals.ConsumerCoordinator.performAssignment(ConsumerCoordinator.java:583)
   ```
   
   need to dig in it :(
   
   @junrao Could you trigger a system test for master branch ?


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