From db415a94fde4ee8ff7580e15131cba08d1690150 Mon Sep 17 00:00:00 2001
From: Elon Azoulay <elon.azoulay@gmail.com>
Date: Sun, 8 May 2022 19:31:54 -0700
Subject: [PATCH] Make methods with default access private in
 PinotSegmentPageSource

---
 .../plugin/pinot/PinotSegmentPageSource.java     | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/plugin/trino-pinot/src/main/java/io/trino/plugin/pinot/PinotSegmentPageSource.java b/plugin/trino-pinot/src/main/java/io/trino/plugin/pinot/PinotSegmentPageSource.java
index 004e560f3600..89e3d40b8569 100755
--- a/plugin/trino-pinot/src/main/java/io/trino/plugin/pinot/PinotSegmentPageSource.java
+++ b/plugin/trino-pinot/src/main/java/io/trino/plugin/pinot/PinotSegmentPageSource.java
@@ -317,18 +317,18 @@ private void writeArrayBlock(BlockBuilder blockBuilder, Type columnType, int col
         }
     }
 
-    Type getType(int columnIndex)
+    private Type getType(int columnIndex)
     {
         checkArgument(columnIndex < columnHandles.size(), "Invalid field index");
         return columnHandles.get(columnIndex).getDataType();
     }
 
-    boolean getBoolean(int rowIdx, int columnIndex)
+    private boolean getBoolean(int rowIdx, int columnIndex)
     {
         return currentDataTable.getDataTable().getInt(rowIdx, columnIndex) != 0;
     }
 
-    long getLong(int rowIndex, int columnIndex)
+    private long getLong(int rowIndex, int columnIndex)
     {
         DataSchema.ColumnDataType dataType = currentDataTable.getDataTable().getDataSchema().getColumnDataType(columnIndex);
         // Note columnType in the dataTable could be different from the original columnType in the columnHandle.
@@ -348,7 +348,7 @@ long getLong(int rowIndex, int columnIndex)
         }
     }
 
-    double getDouble(int rowIndex, int columnIndex)
+    private double getDouble(int rowIndex, int columnIndex)
     {
         DataSchema.ColumnDataType dataType = currentDataTable.getDataTable().getDataSchema().getColumnDataType(columnIndex);
         if (dataType.equals(ColumnDataType.FLOAT)) {
@@ -359,7 +359,7 @@ long getLong(int rowIndex, int columnIndex)
         }
     }
 
-    Block getArrayBlock(int rowIndex, int columnIndex)
+    private Block getArrayBlock(int rowIndex, int columnIndex)
     {
         Type trinoType = getType(columnIndex);
         Type elementType = trinoType.getTypeParameters().get(0);
@@ -408,7 +408,7 @@ Block getArrayBlock(int rowIndex, int columnIndex)
         return blockBuilder.build();
     }
 
-    Slice getSlice(int rowIndex, int columnIndex)
+    private Slice getSlice(int rowIndex, int columnIndex)
     {
         Type trinoType = getType(columnIndex);
         if (trinoType instanceof VarcharType) {
@@ -421,7 +421,7 @@ else if (trinoType instanceof VarbinaryType) {
         return Slices.EMPTY_SLICE;
     }
 
-    static byte[] toBytes(String stringValue)
+    private static byte[] toBytes(String stringValue)
     {
         try {
             return Hex.decodeHex(stringValue.toCharArray());
@@ -431,7 +431,7 @@ static byte[] toBytes(String stringValue)
         }
     }
 
-    Slice getUtf8Slice(String value)
+    private Slice getUtf8Slice(String value)
     {
         if (isNullOrEmpty(value)) {
             return Slices.EMPTY_SLICE;