CompletableFuture中未修改值的supplyAsync+thenCompose是否冗余？

在并发编程课程学习时，我遇到一段测试代码：

@Test
void promiseTestCompose2() throws Exception {
  CompletableFuture<Integer> future1 = CompletableFuture
    .supplyAsync(this::slowInit)
    .thenApply(this::slowIncrement);

  CompletableFuture<Integer> thenCompose = future1
    .thenCompose(value -> CompletableFuture.supplyAsync(() -> value))
    .thenApply(this::slowIncrement);

  int result = thenCompose.get();
  assertEquals(result, 3);
}

这段代码里的.thenCompose(value -> CompletableFuture.supplyAsync(() -> value))没有修改传入值，移除后直接用future1.thenApply(this::slowIncrement)测试依然通过。想问这段thenCompose代码是不是真的冗余？能不能在不修改传入值的场景下直接移除它？

这段thenCompose代码确实是冗余的，完全可以安全移除，原因如下：

1. 功能完全等价：thenCompose的核心作用是将前一个Future的结果传入返回新Future的函数，实现链式调用的扁平化。但这里的lambda只是把传入值原封不动包装成新的supplyAsync Future，没有任何实际逻辑处理。移除后直接调用thenApply(this::slowIncrement)，最终效果完全一致——都是对future1的结果执行slowIncrement操作。
2. 避免不必要的性能损耗：额外的thenCompose会多创建一个无意义的CompletableFuture实例，而且supplyAsync默认会用ForkJoinPool.commonPool()的线程执行，哪怕只是返回原值，也会带来不必要的线程调度开销。
3. 提升代码可读性：移除冗余代码后，逻辑更直接清晰，避免了无意义的链式调用嵌套。

如果这段代码是教学场景下用来演示thenCompose的语法结构，那它有示例价值，但从实际业务开发和性能优化的角度看，完全可以移除，不会影响测试结果或业务逻辑。

内容的提问来源于stack exchange，提问作者user25343943