Fixes

Author: martin-traverse

adapter/avro/src/main/java/org/apache/arrow/adapter/avro/ArrowToAvroUtils.java

@@ -180,7 +180,7 @@ private static BaseAvroProducer<?> createProducer(FieldVector vector, boolean nu
         Producer<?>[] unionChildProducers = new Producer<?>[unionChildVectors.size()];
         for (int i = 0; i < unionChildVectors.size(); i++) {
           FieldVector unionChildVector = unionChildVectors.get(i);
-          unionChildProducers[i] = createProducer(unionChildVector, /* nullable = */ false);  // Do not nest union types
+          unionChildProducers[i] = createProducer(unionChildVector, /* nullable = */ false); // Do not nest union types
         }
         return new AvroUnionsProducer(unionVector, unionChildProducers);
 

adapter/avro/src/main/java/org/apache/arrow/adapter/avro/producers/AvroStringProducer.java

@@ -37,11 +37,15 @@ public AvroStringProducer(VarCharVector vector) {
 
   @Override
   public void produce(Encoder encoder) throws IOException {
+
+    int start = vector.getStartOffset(currentIndex);
+    int end = vector.getEndOffset(currentIndex);
+    int length = end - start;
+
     // The nio ByteBuffer is created once per call, but underlying data is not copied
-    long offset = vector.getStartOffset(currentIndex);
-    int length = vector.getEndOffset(currentIndex);
-    ByteBuffer nioBuffer = vector.getDataBuffer().nioBuffer(offset, length);
+    ByteBuffer nioBuffer = vector.getDataBuffer().nioBuffer(start, length);
     encoder.writeBytes(nioBuffer);
+
     currentIndex++;
   }
 }