[GitHub] [kafka] dima5rr opened a new pull request #8704: KAFKA-10030 allow fetching a key from a single partition

[GitBox <gi...@apache.org>]

dima5rr opened a new pull request #8704:
URL: https://github.com/apache/kafka/pull/8704


   StreamThreadStateStoreProvider#stores throws exception whenever taskId is not found, which is not correct behaviour in multi-threaded env where state store partitions are distributed among several StreamTasks. 
   
   final Task task = tasks.get(keyTaskId);
   if (task == null) {
    throw new InvalidStateStoreException(
    String.format("The specified partition %d for store %s does not exist.",
    storeQueryParams.partition(),
    storeName));
   }
   Reproducible with KStream number of threads more then 1 
   
   StoreQueryIntegrationTest#streamsConfiguration
   
   config.put(StreamsConfig.NUM_STREAM_THREADS_CONFIG, 2);
   
    
   
   Suggested solution is to not throw exception if at least one state store is found, which is always true when using StoreQueryParameters.withPartition


----------------------------------------------------------------
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:
users@infra.apache.org

[GitHub] [kafka] dima5rr closed pull request #8704: KAFKA-10030 allow fetching a key from a single partition

[GitBox <gi...@apache.org>]

dima5rr closed pull request #8704:
URL: https://github.com/apache/kafka/pull/8704


   


----------------------------------------------------------------
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:
users@infra.apache.org