OkHttpClient连接池

背景

最近项目中接入了一个三方的预览云服务,开发、测试环境用起来很流畅,没有什么问题。但是上线之后,发现文件在上传到云服务这个过程中:间歇性出现超时(频率挺高),几分钟那种,这谁能忍啊?

问题定位

首先怀疑是文件太大

经过排查发现,超时的文件只有200+KB,甚至有些文件只有40+KB,这就很离谱

云服务提供商接口问题

这么小的文件都能超时,肯定是服务商接口问题超时。基于这种思想,直接就开始联系对方的技术支持。经过对方排查,没有发现任何问题。
在等待对方排查的过程中,我们对上传接口做了重试。结果并没有什么用!让运维检查了我们的服务器网络状态、带宽、并发,结果都很正常!

服务端请求路由策略问题

经过新增重试逻辑,发现超时文件重试的时候还是很容易超时,并且即使上传成功,都需要2、3分钟的时间。还是没有得到改善之后。我们怀疑是对方服务的请求路由问题。因为慢的文件,始终都慢!,于是联系对方技术排查求证,但是对方回复并没有根据请求信息做路由控制。

代码问题

经过我们双方排查,都没有发现问题所在,这是我们回到我们的代码中。**如果客观环境都没有问题,那么问题可能还是在我们自身。上传代码如下:

public ApiResponse<FileUploadResponse> upload(File file) {
    MultipartBody.Builder builder = new MultipartBody.Builder().setType(MultipartBody.FORM);
    builder.addFormDataPart("file", file.getName(), RequestBody.create(file, MediaType.parse("multipart/form-data")));
    builder.addFormDataPart("appId", properties.getAppId());
    Map<String, String> params = Maps.newHashMap();
    params.put("appId", properties.getAppId());
    builder.addFormDataPart("sign", createSign(params));
    Request request = new Request.Builder()
        .post(builder.build())
        .url(UPLOAD_API)
        .build();

    final Stopwatch stopwatch = Stopwatch.createStarted();
    return RetryTemplateUtils.getCommonRetryTemplate()
        .execute(context -> {
          try (Response response = uploadHttpClient.newCall(request).execute()) {
            String resp = Objects.requireNonNull(response.body()).string();
            log.info("[上传文件] 响应 : {}, 总耗时 : {} ms", resp, stopwatch.elapsed(TimeUnit.MILLISECONDS));
            ApiResponse<FileUploadResponse> result = JSON.parseObject(resp, new TypeReference<ApiResponse<FileUploadResponse>>() {});
            return result.getData();
          } catch (Exception e) {
            log.info("[上传文件] 失败, 重试次数 : {}, 耗时 : {} ms", context.getRetryCount(), stopwatch.elapsed(TimeUnit.MILLISECONDS), e);
            throw new ConversionException(e);
          }
        });
}

uploadHttpClient 配置:

uploadHttpClient = new OkHttpClient.Builder()
        .connectTimeout(200, TimeUnit.SECONDS)
        .readTimeout(10, TimeUnit.SECONDS)
        .writeTimeout(180, TimeUnit.SECONDS)
        .build();

很中规中矩的一个利用OkHttpClient上传文件的代码,没有什么花里胡哨的操作。似乎问题陷入了僵局,但是线上的失败请求一直在报警。

HTTP连接池

到目前,都没有发现问题所在,只能再找到运维,排查一下网关、路由。最终都没发现有什么问题,也没有发现网络波动,一切都很正常。没有办法了,只能再度回到我们的代码中。这时运维提出,不要用连接池,每次都重新建立连接,试试。(PS:OkHttp还是常见的一个框架,连接重用应该是处理的好的吧,但是实在没办法了,先试试吧!)

OkHttpClient连接池配置

这也是一个曲折的过程

  • 设置连接池为NULL,很遗憾,源码是这样的:
public Builder connectionPool(ConnectionPool connectionPool) {
      if (connectionPool == null) throw new NullPointerException("connectionPool == null");
      this.connectionPool = connectionPool;
      return this;
    }
  • 扩展ConnectionPool类,重写获取连接的方法,也很遗憾ConnectionPool是final的

  • 没有办法了,只能研究一下ConnectionPool的构造参数了,与缓存连接的方法了,最终在方法connectionBecameIdle(RealConnection connection)中发现:

boolean connectionBecameIdle(RealConnection connection) {
    assert (Thread.holdsLock(this));
    if (connection.noNewStreams || maxIdleConnections == 0) {
      connections.remove(connection);
      return true;
    } else {
      notifyAll(); // Awake the cleanup thread: we may have exceeded the idle connection limit.
      return false;
    }
  }

maxIdleConnections设置为0就可以实现在连接变成空闲的时候,直接删除连接,不缓存。
所以最终的uploadHttpClient配置为:

uploadHttpClient = new OkHttpClient.Builder()
        .connectTimeout(200, TimeUnit.SECONDS)
        .readTimeout(10, TimeUnit.SECONDS)
        .writeTimeout(180, TimeUnit.SECONDS)
        .connectionPool(new ConnectionPool(0, 1, TimeUnit.NANOSECONDS)) // 不缓存连接,其实后面两个参数就无所谓了
        .build();

改完,发版,惊喜的发现问题解决了!就这样解决了?

反思&总结

虽然问题这样解决了,但是我一开始是不信的。但是回过头来想想,如果这还是不能解决,我肯定是要打人了。
总的来说,过程比较曲折,耗时比较久,但是这种问题确实是不太好排查。尤其是权限受限的情况下,需要这别人一起处理,其中沟通、同步成本还是不容忽视。

最终问题(猜测):

文件上传成功之后,云服务端会主动关闭TCP连接。
OkHttpClient连接池,可能没有处理好服务端主动关闭TCP连接的这种情况。

待后续验证

okhttp3版本: 4.2.1

    <dependency>
      <groupId>com.squareup.okhttp3</groupId>
      <artifactId>okhttp</artifactId>
      <version>4.2.1</version>
    </dependency>

JVM: java8

©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 215,294评论 6 497
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 91,780评论 3 391
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 161,001评论 0 351
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 57,593评论 1 289
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 66,687评论 6 388
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 50,679评论 1 294
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 39,667评论 3 415
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 38,426评论 0 270
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 44,872评论 1 307
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 37,180评论 2 331
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 39,346评论 1 345
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 35,019评论 5 340
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 40,658评论 3 323
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 31,268评论 0 21
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 32,495评论 1 268
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 47,275评论 2 368
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 44,207评论 2 352

推荐阅读更多精彩内容