Rename ByteSequencer to ByteSequenceProvider

Author: karllessard

tensorflow-core/tensorflow-core-api/src/main/java/org/tensorflow/internal/buffer/ByteSequencer.java renamed to tensorflow-core/tensorflow-core-api/src/main/java/org/tensorflow/internal/buffer/ByteSequenceProvider.java

@@ -10,15 +10,15 @@
  *
  * @param <T> source of bytes (byte arrays or strings)
  */
-public class ByteSequencer<T> implements Iterable<byte[]> {
+public class ByteSequenceProvider<T> implements Iterable<byte[]> {
 
   /**
    * Constructor
    *
    * @param source source of data
    * @param byteExtractor method that converts one value of the source into a sequence of bytes
    */
-  public ByteSequencer(NdArray<T> source, Function<T, byte[]> byteExtractor) {
+  public ByteSequenceProvider(NdArray<T> source, Function<T, byte[]> byteExtractor) {
     this.source = source;
     this.byteExtractor = byteExtractor;
   }

tensorflow-core/tensorflow-core-api/src/main/java/org/tensorflow/internal/buffer/ByteSequenceTensorBuffer.java

@@ -55,15 +55,15 @@ public class ByteSequenceTensorBuffer extends AbstractDataBuffer<byte[]> {
   /**
    * Computes how many bytes are required to store the given data in a string buffer.
    *
-   * @param byteSequencer produces sequences of bytes
+   * @param byteSequenceProvider produces sequences of bytes
    * @return number of bytes required to store the data.
    */
-  public static <T> long computeSize(ByteSequencer<?> byteSequencer) {
+  public static <T> long computeSize(ByteSequenceProvider<?> byteSequenceProvider) {
     // reserve space to store 64-bit offsets
-    long size = byteSequencer.numSequences() * Long.BYTES;
+    long size = byteSequenceProvider.numSequences() * Long.BYTES;
 
     // reserve space to store length and data of each values
-    for (byte[] elementBytes : byteSequencer) {
+    for (byte[] elementBytes : byteSequenceProvider) {
       size += elementBytes.length + ByteSequenceTensorBuffer.varintLength(elementBytes.length);
     }
     return size;
@@ -77,11 +77,11 @@ public static <T> long computeSize(ByteSequencer<?> byteSequencer) {
    * same set of data, calling {@link #computeSize(NdArray, Function)} priory to make sure there is
    * enough space to store it.
    *
-   * @param byteSequencer produces sequences of bytes to use as the tensor data
+   * @param byteSequenceProvider produces sequences of bytes to use as the tensor data
    */
-  public <T> void init(ByteSequencer<T> byteSequencer) {
+  public <T> void init(ByteSequenceProvider<T> byteSequenceProvider) {
     InitDataWriter writer = new InitDataWriter();
-    byteSequencer.forEach(writer::writeNext);
+    byteSequenceProvider.forEach(writer::writeNext);
   }
 
   @Override

tensorflow-core/tensorflow-core-api/src/main/java/org/tensorflow/internal/types/TStringInitializer.java

@@ -3,7 +3,7 @@
 import java.util.function.Consumer;
 import java.util.function.Function;
 import org.tensorflow.internal.buffer.ByteSequenceTensorBuffer;
-import org.tensorflow.internal.buffer.ByteSequencer;
+import org.tensorflow.internal.buffer.ByteSequenceProvider;
 import org.tensorflow.internal.types.TStringMapper.TStringInternal;
 import org.tensorflow.ndarray.NdArray;
 import org.tensorflow.types.TString;
@@ -16,23 +16,23 @@
 public final class TStringInitializer<T> implements Consumer<TString> {
 
   public TStringInitializer(NdArray<T> source, Function<T, byte[]> byteExtractor) {
-    this.byteSequencer = new ByteSequencer<>(source, byteExtractor);
+    this.byteSequenceProvider = new ByteSequenceProvider<>(source, byteExtractor);
   }
 
   /**
    * Compute the minimum size for a tensor to hold all the data provided by the source.
    *
    * @return minimum tensor size, in bytes
-   * @see ByteSequenceTensorBuffer#computeSize(ByteSequencer)
+   * @see ByteSequenceTensorBuffer#computeSize(ByteSequenceProvider)
    */
   public long computeRequiredSize() {
-    return ByteSequenceTensorBuffer.computeSize(byteSequencer);
+    return ByteSequenceTensorBuffer.computeSize(byteSequenceProvider);
   }
 
   @Override
   public void accept(TString tensor) {
-    ((TStringInternal)tensor).init(byteSequencer);
+    ((TStringInternal)tensor).init(byteSequenceProvider);
   }
 
-  private final ByteSequencer<T> byteSequencer;
+  private final ByteSequenceProvider<T> byteSequenceProvider;
 }

tensorflow-core/tensorflow-core-api/src/main/java/org/tensorflow/internal/types/TStringMapper.java

@@ -2,12 +2,11 @@
 
 import java.nio.charset.Charset;
 import java.nio.charset.StandardCharsets;
-import java.util.function.Function;
 import org.tensorflow.DataType;
 import org.tensorflow.RawTensor;
 import org.tensorflow.TensorMapper;
 import org.tensorflow.internal.buffer.ByteSequenceTensorBuffer;
-import org.tensorflow.internal.buffer.ByteSequencer;
+import org.tensorflow.internal.buffer.ByteSequenceProvider;
 import org.tensorflow.internal.buffer.TensorBuffers;
 import org.tensorflow.ndarray.NdArray;
 import org.tensorflow.ndarray.NdArrays;
@@ -40,17 +39,17 @@ interface TStringInternal extends TString {
     /**
      * Initialize the buffer of this string tensor using the provided byte sequencer.
      *
-     * @param byteSequencer produces sequences of bytes to use as the tensor data
+     * @param byteSequenceProvider produces sequences of bytes to use as the tensor data
      * @param <T> source of bytes ({@code byte[]} or {@code String})
      */
-    <T> void init(ByteSequencer<T> byteSequencer);
+    <T> void init(ByteSequenceProvider<T> byteSequenceProvider);
   }
 
   private static final class DenseTString extends DenseNdArray<String> implements TStringInternal {
 
     @Override
-    public <T> void init(ByteSequencer<T> byteSequencer) {
-      buffer.init(byteSequencer);
+    public <T> void init(ByteSequenceProvider<T> byteSequenceProvider) {
+      buffer.init(byteSequenceProvider);
     }
 
     @Override