[task] optimize okhttp3 connection factory

[Aias00]

What's changed?

optimize okhttp3 connection factory, link #2528

Checklist

• I have read the Contributing Guide
• I have written the necessary doc or comment.
• I have added the necessary unit tests and all cases have passed.

Add or update API

• I have added the necessary e2e tests and all cases have passed.

[tomsun28]

I found this depend the httpclient5, there also have httpclient 4.5.14, okhttp in the system, suggest we unify them and use only one

[Aias00]

I found this depend the httpclient5, there also have httpclient 4.5.14, okhttp in the system, suggest we unify them and use only one

okay，i will fix them later

[Aias00]

I found this depend the httpclient5, there also have httpclient 4.5.14, okhttp in the system, suggest we unify them and use only one

hi, how about i submit another pr, and unify the httpcliens version in that pr; cause this modification will effect many codes

[Aias00]

I found this depend the httpclient5, there also have httpclient 4.5.14, okhttp in the system, suggest we unify them and use only one

hi. i found many depnedencies import the httpclient 4 version , maybe i should change the httpclient5 to httpclient4

[tomsun28]

okay👍👍, this seem effect many components.

[yuluo-yx]

btw, I found only

hertzbeat/warehouse/src/main/java/org/apache/hertzbeat/warehouse/store/history/influxdb/InfluxdbDataStorage.java

Line 89 in 361e573

var client = new OkHttpClient.Builder()

okhttp is used here Library.

maybe we can use httpclient instead in our project. spring has given up on okhttp, spring-projects/spring-framework#30919 @Aias00 cc @tomsun28

[Aias00]

@tomsun28 hi, i tried to degrade the httpclient 5 to httpclient 4. but the truth is that spring-web:6.1.4 would not support httpclient 4 anymore. so i think we can do that changed later, or i can see whether we can upgrade all httpclient4 to httpclient5

[Aias00]

btw, I found only

hertzbeat/warehouse/src/main/java/org/apache/hertzbeat/warehouse/store/history/influxdb/InfluxdbDataStorage.java

Line 89 in 361e573

var client = new OkHttpClient.Builder()

okhttp is used here Library.
maybe we can use httpclient instead in our project. spring has given up on okhttp, spring-projects/spring-framework#30919 @Aias00 cc @tomsun28

hi, the influxdb-java library has binding okhttp , we can not change that now

[yuluo-yx]

LGTM.

[tomsun28]

@tomsun28 hi, i tried to degrade the httpclient 5 to httpclient 4. but the truth is that spring-web:6.1.4 would not support httpclient 4 anymore. so i think we can do that changed later, or i can see whether we can upgrade all httpclient4 to httpclient5

Hi, yes, maybe we can upgrade httpclient4 to httpclient5 later

it seem need to unify the httpclient version