#811 Fix PR suggestions (Thanks @coderabbitai).

Author: yruslan

cobol-parser/src/main/scala/za/co/absa/cobrix/cobol/reader/VarLenNestedReader.scala

@@ -44,6 +44,7 @@ class VarLenNestedReader[T: ClassTag](copybookContents: Seq[String],
                                       handler: RecordHandler[T]) extends VarLenReader with Logging with Serializable {
   private val DEFAULT_INDEX_SIZE_COMPRESSED_FILES_MB = 1024
   private val DEFAULT_FS_INDEX_SIZE_MULTIPLIER = 4
+  private val DEFAULT_MAX_FS_BASED_SPLIT_SIZE_MB = 256
 
   protected val cobolSchema: CobolSchema = loadCopyBook(copybookContents)
 
@@ -221,10 +222,10 @@ class VarLenNestedReader[T: ClassTag](copybookContents: Seq[String],
     if (isCompressed) {
       readerProperties.inputSplitSizeCompressedMB.orElse(Some(DEFAULT_INDEX_SIZE_COMPRESSED_FILES_MB))
     } else {
-      val defaultIndexSizeBofFsBlock = readerProperties.fsDefaultBlockSize.map { size =>
-        Math.min(size * DEFAULT_FS_INDEX_SIZE_MULTIPLIER, 256)
+      val defaultIndexSizeBasedOnFsBlock = readerProperties.fsDefaultBlockSize.map { size =>
+        Math.min(size * DEFAULT_FS_INDEX_SIZE_MULTIPLIER, DEFAULT_MAX_FS_BASED_SPLIT_SIZE_MB)
       }
-      readerProperties.inputSplitSizeMB.orElse(defaultIndexSizeBofFsBlock)
+      readerProperties.inputSplitSizeMB.orElse(defaultIndexSizeBasedOnFsBlock)
     }
   }
 

spark-cobol/src/main/scala/za/co/absa/cobrix/spark/cobol/utils/LRUCache.scala

@@ -19,7 +19,7 @@ package za.co.absa.cobrix.spark.cobol.utils
 import scala.collection.JavaConverters._
 
 class LRUCache[K,V](maxSize: Int, loadFactor: Float = 0.75f) {
-  private val cache = new java.util.LinkedHashMap[K, V](16, loadFactor, true) {
+  private val cache = new java.util.LinkedHashMap[K, V](Math.min(maxSize, 128), loadFactor, true) {
     override def removeEldestEntry(eldest: java.util.Map.Entry[K, V]): Boolean = size() > maxSize
   }
 

spark-cobol/src/test/scala/za/co/absa/cobrix/spark/cobol/source/integration/Test37RecordLengthMappingSpec.scala

@@ -201,7 +201,7 @@ class Test37RecordLengthMappingSpec extends AnyWordSpec with SparkTestBase with
           case (k, v) => (k.fileName, v)
         }
 
-        assert(indexCacheSimplified.get(pathNameAsCached) != null)
+        assert(indexCacheSimplified.contains(pathNameAsCached))
         assert(indexCacheSimplified(pathNameAsCached).length == 2)
 
         assert(actualInitial == expected)