fix java8 lambda

[wuwen5]

在java8中，如果使用含有lambda语法会有异常：

七月 12, 2016 9:48:01 下午 com.alibaba.ttl.threadpool.agent.TtlTransformer transform
严重: Fail to transform class null, cause: null
七月 12, 2016 9:48:01 下午 com.alibaba.ttl.threadpool.agent.TtlTransformer transform
严重: Fail to transform class null, cause: null
七月 12, 2016 9:48:01 下午 com.alibaba.ttl.threadpool.agent.TtlTransformer transform
严重: Fail to transform class null, cause: null
七月 12, 2016 9:48:01 下午 com.alibaba.ttl.threadpool.agent.TtlTransformer transform
严重: Fail to transform class null, cause: null
七月 12, 2016 9:48:01 下午 com.alibaba.ttl.threadpool.agent.TtlTransformer transform
严重: Fail to transform class null, cause: null
七月 12, 2016 9:48:01 下午 com.alibaba.ttl.threadpool.agent.TtlTransformer transform
严重: Fail to transform class null, cause: null
七月 12, 2016 9:48:01 下午 com.alibaba.ttl.threadpool.agent.TtlTransformer transform
严重: Fail to transform class null, cause: null
七月 12, 2016 9:48:01 下午 com.alibaba.ttl.threadpool.agent.TtlTransformer transform
严重: Fail to transform class null, cause: null
七月 12, 2016 9:48:01 下午 com.alibaba.ttl.threadpool.agent.TtlTransformer transform
严重: Fail to transform class null, cause: null
七月 12, 2016 9:48:01 下午 com.alibaba.ttl.threadpool.agent.TtlTransformer transform
严重: Fail to transform class null, cause: null

分析发现仅在环境中使用了lambda的时候, classFile是空的，导致空指针异常。

public byte[] transform(ClassLoader loader, String classFile, Class<?> classBeingRedefined,
                            ProtectionDomain protectionDomain, byte[] classFileBuffer) 

我把classFileBuffer 写到文件然后查看发现是这样的：

javap 96fcce0c-0b37-4228-8da6-9bf45b59f94f
警告: 二进制文件96fcce0c-0b37-4228-8da6-9bf45b59f94f包含java.lang.invoke.LambdaForm$MH
Compiled from "LambdaForm$MH"
final class java.lang.invoke.LambdaForm$MH {
  static java.lang.Object linkToTargetMethod(java.lang.Object);
}

javap 3e8da79c-ece4-4699-bc16-25427040b04f
警告: 二进制文件3e8da79c-ece4-4699-bc16-25427040b04f包含com.ofpay.ofdc.Demo$$Lambda$1
final class com.ofpay.ofdc.Demo$$Lambda$1 implements java.lang.Runnable {
  public void run();
}

[CLAassistant]

All committers have signed the CLA.

[codecov-io]

Current coverage is 84.39%

Merging #68 into master will not change coverage

@@             master        #68   diff @@
==========================================
  Files             7          7          
  Lines           173        173          
  Methods           0          0          
  Messages          0          0          
  Branches         27         27          
==========================================
  Hits            146        146          
  Misses           17         17          
  Partials         10         10          

Powered by Codecov. Last updated by 6bb5991...88d4786

[oldratlee]

@wuwen5 非常感谢 问题汇报和解决。

我看一下 ，如果OK，我就发一个新版本出来 😄

[oldratlee]

@wuwen5

在java8中，如果使用含有lambda语法会有异常：

能给一下 『使用含有lambda语法』的代码， 方便复现问题后确认方案是有效解决的。

[wuwen5]

这是本地验证的时候的测试代码.

public class Demo {
    public static void main(String[] args) throws InterruptedException {
        ExecutorService executorService = Executors.newFixedThreadPool(1);
        executorService.execute(() -> System.out.println("++++"));
    }
}

同样是jdk 8
如果这么写也不会出现classFile空值的情况

executorService.execute(new Runnable() {
    @Override
    public void run() {
        System.out.println("---run test----")
    }
});

[oldratlee]

@wuwen5 已Release新版本 2.0.2
https://github.com/alibaba/transmittable-thread-local/releases/tag/v2.0.2

[wuwen5]

@oldratlee 有个疑问请教下，我看代码transform的返回值改了，将不需要转换的class改成返回空数组了。byte[] EMPTY_BYTE_ARRAY = {};

但是我很好奇ClassFileTransformer API的注释写的 不转换是返回 null，jvm这块处理空数组和null会是一样么？

@return  a well-formed class file buffer (the result of the transform),
                or <code>null</code> if no transform is performed.

[oldratlee]

用 empty array 来自代码质量改进的一个pull request #56
你可以从commit log( 4a5b292 )里查到。

我看持续集成多jdk版本都是通过的，就接纳了这个pr。

[haogrgr]

Lambda是运行时生成的字节码, new Runnable是编译期的.

[oldratlee]

@haogrgr 因为运行时生成字节码所以没class file。Got!