[jira] [Commented] (KAFKA-13103) Should group admin handlers consider REBALANCE_IN_PROGRESS and GROUP_AUTHORIZATION_FAILED as retryable errors?

2021-07-20 Thread Luke Chen (Jira)


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

Luke Chen commented on KAFKA-13103:
---

In the PR, I put the `REBALANCE_IN_PROGRESS` into retriable error. But keep 
`GROUP_AUTHORIZATION_FAILED` as failed since I also think it doesn't make sense 
to put `GROUP_AUTHORIZATION_FAILED` as retriable error. Thanks.

> Should group admin handlers consider REBALANCE_IN_PROGRESS and 
> GROUP_AUTHORIZATION_FAILED as retryable errors?
> --
>
> Key: KAFKA-13103
> URL: https://issues.apache.org/jira/browse/KAFKA-13103
> Project: Kafka
>  Issue Type: Improvement
>Reporter: David Jacot
>Assignee: Luke Chen
>Priority: Major
>
> [~rajinisiva...@gmail.com] and I were discussing if we should consider 
> REBALANCE_IN_PROGRESS and GROUP_AUTHORIZATION_FAILED as retryable errors in 
> the group handlers. I think that this could make sense, especially for 
> `REBALANCE_IN_PROGRESS`. `GROUP_AUTHORIZATION_FAILED` is more debatable as it 
> means that the handler would retry until the request timeout is reached. It 
> might be armful if the authorisation is really denied.



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


[jira] [Commented] (KAFKA-13103) Should group admin handlers consider REBALANCE_IN_PROGRESS and GROUP_AUTHORIZATION_FAILED as retryable errors?

2021-07-19 Thread Luke Chen (Jira)


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

Luke Chen commented on KAFKA-13103:
---

I'll get the PR ready today (my time) in case this improvement wants to get 
into v3.0. Thanks.

> Should group admin handlers consider REBALANCE_IN_PROGRESS and 
> GROUP_AUTHORIZATION_FAILED as retryable errors?
> --
>
> Key: KAFKA-13103
> URL: https://issues.apache.org/jira/browse/KAFKA-13103
> Project: Kafka
>  Issue Type: Improvement
>Reporter: David Jacot
>Assignee: Luke Chen
>Priority: Major
>
> [~rajinisiva...@gmail.com] and I were discussing if we should consider 
> REBALANCE_IN_PROGRESS and GROUP_AUTHORIZATION_FAILED as retryable errors in 
> the group handlers. I think that this could make sense, especially for 
> `REBALANCE_IN_PROGRESS`. `GROUP_AUTHORIZATION_FAILED` is more debatable as it 
> means that the handler would retry until the request timeout is reached. It 
> might be armful if the authorisation is really denied.



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


[jira] [Commented] (KAFKA-13103) Should group admin handlers consider REBALANCE_IN_PROGRESS and GROUP_AUTHORIZATION_FAILED as retryable errors?

2021-07-19 Thread Ismael Juma (Jira)


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

Ismael Juma commented on KAFKA-13103:
-

What would be the justification for `GROUP_AUTHORIZATION_FAILED`? 
`REBALANCE_IN_PROGRESS` seems really important though, would this be something 
we want to do for 3.0? Or is there a reason why it's ok for now?

> Should group admin handlers consider REBALANCE_IN_PROGRESS and 
> GROUP_AUTHORIZATION_FAILED as retryable errors?
> --
>
> Key: KAFKA-13103
> URL: https://issues.apache.org/jira/browse/KAFKA-13103
> Project: Kafka
>  Issue Type: Improvement
>Reporter: David Jacot
>Assignee: Luke Chen
>Priority: Major
>
> [~rajinisiva...@gmail.com] and I were discussing if we should consider 
> REBALANCE_IN_PROGRESS and GROUP_AUTHORIZATION_FAILED as retryable errors in 
> the group handlers. I think that this could make sense, especially for 
> `REBALANCE_IN_PROGRESS`. `GROUP_AUTHORIZATION_FAILED` is more debatable as it 
> means that the handler would retry until the request timeout is reached. It 
> might be armful if the authorisation is really denied.



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