More mod replacement

Author: JoshRosen

sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/UnsafeRowConverter.scala

@@ -87,7 +87,7 @@ private class StringUnsafeColumnWriter private() extends UnsafeColumnWriter[UTF8
       numBytes
     )
     row.setLong(columnNumber, appendCursor)
-    8 + ((numBytes / 8) + (if (numBytes % 8 == 0) 0 else 1)) * 8
+    8 + ByteArrayMethods.roundNumberOfBytesToNearestWord(numBytes)
   }
 }
 private object StringUnsafeColumnWriter extends StringUnsafeColumnWriter

unsafe/src/test/java/org/apache/spark/unsafe/string/TestUTF8String.java

@@ -17,20 +17,24 @@
 
 package org.apache.spark.unsafe.string;
 
+import java.io.UnsupportedEncodingException;
+import java.lang.String;
+
 import org.junit.Assert;
 import org.junit.Test;
 
 import org.apache.spark.unsafe.memory.MemoryLocation;
 import org.apache.spark.unsafe.memory.MemoryBlock;
-import java.lang.String;
+import org.apache.spark.unsafe.array.ByteArrayMethods;
 
 public class TestUTF8String {
 
   @Test
-  public void toStringTest() {
+  public void toStringTest() throws UnsupportedEncodingException {
     final String javaStr = "Hello, World!";
     final byte[] javaStrBytes = javaStr.getBytes();
-    final int paddedSizeInWords = javaStrBytes.length / 8 + (javaStrBytes.length % 8 == 0 ? 0 : 1);
+    final int paddedSizeInWords =
+      ByteArrayMethods.roundNumberOfBytesToNearestWord(javaStrBytes.length);
     final MemoryLocation memory = MemoryBlock.fromLongArray(new long[paddedSizeInWords]);
     final int bytesWritten = UTF8StringMethods.createFromJavaString(
       memory.getBaseObject(),