[jira] [Commented] (KAFKA-10030) Throw exception while fetching a key from a single partition

2020-05-29 Thread John Roesler (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-10030?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17119968#comment-17119968
 ] 

John Roesler commented on KAFKA-10030:
--

I'm getting ready for the 2.5.1 release at the moment. It looks like this PR is 
close, so I'll leave it targeted for 2.5.1 for now.

> Throw exception while fetching a key from a single partition
> 
>
> Key: KAFKA-10030
> URL: https://issues.apache.org/jira/browse/KAFKA-10030
> Project: Kafka
>  Issue Type: Bug
>  Components: streams
>Affects Versions: 2.5.0
> Environment: StreamsConfig.NUM_STREAM_THREADS_CONFIG=2
>Reporter: Dima R
>Assignee: Dima R
>Priority: Major
>  Labels: KAFKA-9445, KIP-562
> Fix For: 2.6.0, 2.5.1
>
>
> 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. 
> {code:java}
> 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));
> }{code}
> 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



--
This message was sent by Atlassian Jira
(v8.3.4#803005)


[jira] [Commented] (KAFKA-10030) Throw exception while fetching a key from a single partition

2020-05-21 Thread Matthias J. Sax (Jira)


[ 
https://issues.apache.org/jira/browse/KAFKA-10030?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=17113449#comment-17113449
 ] 

Matthias J. Sax commented on KAFKA-10030:
-

Thanks for the bug report and PR! – I added you to the list of contributors and 
assigned the ticket to you. You can now also self-assign tickets.

> Throw exception while fetching a key from a single partition
> 
>
> Key: KAFKA-10030
> URL: https://issues.apache.org/jira/browse/KAFKA-10030
> Project: Kafka
>  Issue Type: Bug
>  Components: streams
>Affects Versions: 2.5.0
> Environment: StreamsConfig.NUM_STREAM_THREADS_CONFIG=2
>Reporter: Dima R
>Assignee: Dima R
>Priority: Major
>  Labels: KAFKA-9445, KIP-562
> Fix For: 2.6.0, 2.5.1
>
>
> 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. 
> {code:java}
> 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));
> }{code}
> 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



--
This message was sent by Atlassian Jira
(v8.3.4#803005)