[ISSUE-669]: Update libthrift to 0.13.0 (#672)

Resolves #669

* Update libthrift to 0.13.0
* Fix code accordingly

Issue: #669
Signed-off-by: Emmanuel Hugonnet <[email protected]>

Author: ehsavoie

build.gradle

@@ -19,7 +19,7 @@ plugins {
 
 ext.opentracingVersion = getProperty('opentracingVersion','0.33.0')
 ext.guavaVersion = getProperty('guavaVersion','18.0')
-ext.apacheThriftVersion = getProperty('apacheThriftVersion','0.12.0')
+ext.apacheThriftVersion = getProperty('apacheThriftVersion','0.13.0')
 ext.jerseyVersion = getProperty('jerseyVersion','2.22.2')
 ext.slf4jVersion = getProperty('slf4jVersion','1.7.28')
 ext.gsonVersion = getProperty('gsonVersion','2.8.6')

jaeger-thrift/src/main/java/io/jaegertracing/thrift/internal/senders/ThriftSenderBase.java

@@ -79,7 +79,7 @@ protected byte[] serialize(TBase<?,?> thriftBase) throws Exception {
   public int getSize(TBase<?,?> thriftBase) throws Exception {
     memoryTransport.reset();
     thriftBase.write(protocolFactory.getProtocol(memoryTransport));
-    return memoryTransport.getPos();
+    return memoryTransport.getLength();
   }
 
 }

jaeger-thrift/src/test/java/io/jaegertracing/thrift/internal/reporters/protocols/ThriftSenderBaseTest.java

@@ -55,13 +55,13 @@ private int calculateBatchOverheadDifference(int numberOfSpans) throws Exception
     }
 
     memoryClient.emitBatch(new Batch(new io.jaegertracing.thriftjava.Process(SERVICE_NAME), spans));
-    int emitBatchOverheadMultipleSpans = memoryTransport.getPos();
+    int emitBatchOverheadMultipleSpans = memoryTransport.getLength();
 
     memoryTransport.reset();
     for (int j = 0; j < numberOfSpans; j++) {
       span.write(new TCompactProtocol(memoryTransport));
     }
-    int writeBatchOverheadMultipleSpans = memoryTransport.getPos();
+    int writeBatchOverheadMultipleSpans = memoryTransport.getLength();
 
     return emitBatchOverheadMultipleSpans - writeBatchOverheadMultipleSpans;
   }