[jira] [Updated] (KAFKA-16027) Refactor MetadataTest#testUpdatePartitionLeadership

2023-12-18 Thread Philip Nee (Jira)


 [ 
https://issues.apache.org/jira/browse/KAFKA-16027?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Philip Nee updated KAFKA-16027:
---
Description: 
MetadataTest#testUpdatePartitionLeadership is extremely long.  I think it is 
pretty close to the 160 line method limit - I tried to modfity it but it would 
hit the limit when i tried to break things into separated lines.

The test also contains two tests, so it is best to split it into two separate 
tests.

  was:MetadataTest#testUpdatePartitionLeadership is extremely long.  I think it 
is pretty close to the 160 line method limit.  The test also contains two 
tests, so it is best to split it into two separate tests.


> Refactor MetadataTest#testUpdatePartitionLeadership
> ---
>
> Key: KAFKA-16027
> URL: https://issues.apache.org/jira/browse/KAFKA-16027
> Project: Kafka
>  Issue Type: Improvement
>Reporter: Philip Nee
>Priority: Minor
>  Labels: newbie
>
> MetadataTest#testUpdatePartitionLeadership is extremely long.  I think it is 
> pretty close to the 160 line method limit - I tried to modfity it but it 
> would hit the limit when i tried to break things into separated lines.
> The test also contains two tests, so it is best to split it into two separate 
> tests.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (KAFKA-16027) Refactor MetadataTest#testUpdatePartitionLeadership

2023-12-18 Thread Philip Nee (Jira)


 [ 
https://issues.apache.org/jira/browse/KAFKA-16027?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Philip Nee updated KAFKA-16027:
---
Description: 
MetadataTest#testUpdatePartitionLeadership is extremely long.  I think it is 
pretty close to the 160 line method limit - I tried to modfity it but it would 
hit the limit when i tried to break things into separated lines.

The test also contains two tests, so it is best to split it into two separate 
tests.

We should also move this to ConsumerMetadata.java

  was:
MetadataTest#testUpdatePartitionLeadership is extremely long.  I think it is 
pretty close to the 160 line method limit - I tried to modfity it but it would 
hit the limit when i tried to break things into separated lines.

The test also contains two tests, so it is best to split it into two separate 
tests.


> Refactor MetadataTest#testUpdatePartitionLeadership
> ---
>
> Key: KAFKA-16027
> URL: https://issues.apache.org/jira/browse/KAFKA-16027
> Project: Kafka
>  Issue Type: Improvement
>Reporter: Philip Nee
>Priority: Minor
>  Labels: newbie
>
> MetadataTest#testUpdatePartitionLeadership is extremely long.  I think it is 
> pretty close to the 160 line method limit - I tried to modfity it but it 
> would hit the limit when i tried to break things into separated lines.
> The test also contains two tests, so it is best to split it into two separate 
> tests.
> We should also move this to ConsumerMetadata.java



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (KAFKA-16027) Refactor MetadataTest#testUpdatePartitionLeadership

2023-12-21 Thread Alexander Aghili (Jira)


 [ 
https://issues.apache.org/jira/browse/KAFKA-16027?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Alexander Aghili updated KAFKA-16027:
-
External issue URL:   (was: https://github.com/apache/kafka/pull/15055)

> Refactor MetadataTest#testUpdatePartitionLeadership
> ---
>
> Key: KAFKA-16027
> URL: https://issues.apache.org/jira/browse/KAFKA-16027
> Project: Kafka
>  Issue Type: Improvement
>Reporter: Philip Nee
>Assignee: Alexander Aghili
>Priority: Minor
>  Labels: newbie
>
> MetadataTest#testUpdatePartitionLeadership is extremely long.  I think it is 
> pretty close to the 160 line method limit - I tried to modfity it but it 
> would hit the limit when i tried to break things into separated lines.
> The test also contains two tests, so it is best to split it into two separate 
> tests.
> We should also move this to ConsumerMetadata.java



--
This message was sent by Atlassian Jira
(v8.20.10#820010)


[jira] [Updated] (KAFKA-16027) Refactor MetadataTest#testUpdatePartitionLeadership

2023-12-21 Thread Alexander Aghili (Jira)


 [ 
https://issues.apache.org/jira/browse/KAFKA-16027?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Alexander Aghili updated KAFKA-16027:
-
External issue URL: https://github.com/apache/kafka/pull/15055

> Refactor MetadataTest#testUpdatePartitionLeadership
> ---
>
> Key: KAFKA-16027
> URL: https://issues.apache.org/jira/browse/KAFKA-16027
> Project: Kafka
>  Issue Type: Improvement
>Reporter: Philip Nee
>Assignee: Alexander Aghili
>Priority: Minor
>  Labels: newbie
>
> MetadataTest#testUpdatePartitionLeadership is extremely long.  I think it is 
> pretty close to the 160 line method limit - I tried to modfity it but it 
> would hit the limit when i tried to break things into separated lines.
> The test also contains two tests, so it is best to split it into two separate 
> tests.
> We should also move this to ConsumerMetadata.java



--
This message was sent by Atlassian Jira
(v8.20.10#820010)