apache / apache/servicecomb-java-chassis

[BUG] - Java-Chassis3.2.1之后版本servicecomb.rest.client下的connection.maxPoolSize、http2.maxPoolSize、maxWaitQueueSize等配置项失效,只能使用固定默认值

Abierto
#4,854 1 comentario 0 reacciones 0 asignados Ver en GitHub
bug
Lenguaje dominante
Java
Estrellas
1.9k
Forks
814
Merge medio
8 d 23 h
PR fusionados (30 d)
1

Descripción

### Steps to Reproduce

Java-Chassis3.2.1 版本引入修改,创建HttpClient时原本使用 `xxx.createHttpClient(httpClientOptions)`, 修改为使用 `xxx.with(httpClientOptions)`

![Image](https://github.com/user-attachments/assets/b7137f86-25bd-49d2-bd8c-d1747ed3b165)

而`xxx.with(httpClientOptions)`只设置了`HttpClientOptions`

![Image](https://github.com/user-attachments/assets/926b91a9-4c03-4376-af89-6e4a95539b9c)

原本`xxx.createHttpClient(httpClientOptions)` 预期为同时修改`HttpClientOptions` 与 `PoolOptions`

![Image](https://github.com/user-attachments/assets/b1cd8d9f-6a4d-472c-9a48-063b4e6ce6d9)

导致在后续buildHttpClient时,`this.poolOptions`一直都为null,所以都是使用的默认`PoolOptions`

![Image](https://github.com/user-attachments/assets/a1a8f956-9af7-470f-ae43-2e8a1d7f1ed3)

![Image](https://github.com/user-attachments/assets/2b3f7a03-f3d7-4dc5-93ac-6ead1b70684c)

### Expected Behavior

预期只需要修改把`poolOptions`设置进去即可

![Image](https://github.com/user-attachments/assets/82c6dffd-e2f1-4976-8dcd-0a8163406db8)

### Servicecomb Version

_No response_

### Additional Context

_No response_

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Rastrea el cambio de Java-Chassis3.2.1 de createHttpClient(httpClientOptions) a with(httpClientOptions), y luego sigue buildHttpClient y el manejo de HttpClientOptions y PoolOptions. Verifica que connection.maxPoolSize, http2.maxPoolSize y maxWaitQueueSize lleguen a PoolOptions en lugar de usar valores predeterminados, y añade o ejecuta una prueba de regresión para esos ajustes.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
java
Área
backend
Tipo de issue
Error
Dificultad
2/5
Tiempo estimado
Medio día
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.