Convert CtrlData to record

Converts the CtrlData class to use Java 17 records, reducing boilerplate
and improving memory layout. Updates all getter method calls to use
record accessors.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: oschwald

dev-bin/release-wip.sh

@@ -0,0 +1,130 @@
+#!/bin/bash
+
+set -eu -o pipefail
+
+changelog=$(cat CHANGELOG.md)
+
+regex='
+([0-9]+\.[0-9]+\.[0-9]+[a-zA-Z0-9\-]*) \(([0-9]{4}-[0-9]{2}-[0-9]{2})\)
+-*
+
+((.|
+)*)
+'
+
+if [[ ! $changelog =~ $regex ]]; then
+      echo "Could not find date line in change log!"
+      exit 1
+fi
+
+version="${BASH_REMATCH[1]}"
+date="${BASH_REMATCH[2]}"
+notes="$(echo "${BASH_REMATCH[3]}" | sed -n -e '/^[0-9]\+\.[0-9]\+\.[0-9]\+/,$!p')"
+
+if [[ "$date" !=  $(date +"%Y-%m-%d") ]]; then
+    echo "$date is not today!"
+    exit 1
+fi
+
+tag="v$version"
+
+# if [ -n "$(git status --porcelain)" ]; then
+#     echo ". is not clean." >&2
+#     exit 1
+# fi
+
+# if [ ! -d .gh-pages ]; then
+#     echo "Checking out gh-pages in .gh-pages"
+#     git clone -b gh-pages git@github.com:maxmind/MaxMind-DB-Reader-java.git .gh-pages
+#     pushd .gh-pages
+# else
+#     echo "Updating .gh-pages"
+#     pushd .gh-pages
+#     git pull
+# fi
+
+# if [ -n "$(git status --porcelain)" ]; then
+#     echo ".gh-pages is not clean" >&2
+#     exit 1
+# fi
+
+# popd
+
+# mvn versions:display-plugin-updates
+# mvn versions:display-dependency-updates
+
+# read -r -n 1 -p "Continue given above dependencies? (y/n) " should_continue
+
+# if [ "$should_continue" != "y" ]; then
+#     echo "Aborting"
+#     exit 1
+# fi
+
+# mvn test
+
+# read -r -n 1 -p "Continue given above tests? (y/n) " should_continue
+
+# if [ "$should_continue" != "y" ]; then
+#     echo "Aborting"
+#     exit 1
+# fi
+
+# page=.gh-pages/index.md
+# cat <<EOF > $page
+# ---
+# layout: default
+# title: MaxMind DB Java API
+# language: java
+# version: $tag
+# ---
+
+# EOF
+
+# mvn versions:set -DnewVersion="$version"
+
+# perl -pi -e "s/(?<=<version>)[^<]*/$version/" README.md
+# perl -pi -e "s/(?<=com\.maxmind\.db\:maxmind-db\:)\d+\.\d+\.\d+([\w\-]+)?/$version/" README.md
+
+# cat README.md >> $page
+
+# git diff
+
+# read -r -n 1 -p "Commit changes? " should_commit
+# if [ "$should_commit" != "y" ]; then
+#     echo "Aborting"
+#     exit 1
+# fi
+# git add README.md pom.xml
+# git commit -m "Preparing for $version"
+
+# mvn clean deploy
+
+rm -fr ".gh-pages/doc/$tag"
+cp -r target/reports/apidocs ".gh-pages/doc/$tag"
+rm .gh-pages/doc/latest
+ln -fs "$tag" .gh-pages/doc/latest
+
+pushd .gh-pages
+
+git add doc/
+git commit -m "Updated for $tag" -a
+
+echo "Release notes for $version:
+
+$notes
+
+"
+read -r -n 1 -p "Push to origin? " should_push
+
+if [ "$should_push" != "y" ]; then
+    echo "Aborting"
+    exit 1
+fi
+
+git push
+
+popd
+
+git push
+
+gh release create --target "$(git branch --show-current)" -t "$version" -n "$notes" "$tag"

java-17-upgrade-plan.md

@@ -0,0 +1,78 @@
+# Comprehensive Plan for Dropping Java 11 Support (ENG-2665)
+
+## 1. Update Build Configuration
+- **pom.xml:149-153**: Update maven-compiler-plugin from Java 11 to Java 17
+  - Change `<release>11</release>` to `<release>17</release>`
+  - Change `<source>11</source>` to `<source>17</source>` 
+  - Change `<target>11</target>` to `<target>17</target>`
+
+## 2. Update CI/CD Configuration
+- **.github/workflows/test.yml:12**: Remove Java 11 from test matrix
+  - Change `version: [ 11, 17, 21, 22 ]` to `version: [ 17, 21, 22 ]`
+
+## 3. Update Documentation
+- **README.md:210**: Change "This API requires Java 11 or greater" to "This API requires Java 17 or greater"
+
+## 4. Version Bump (Breaking Change)
+- **pom.xml:6**: Bump to major version 4.0.0
+- **README.md:19,32**: Update version in Maven and Gradle examples to 4.0.0
+- **CHANGELOG.md**: Add entry for 4.0.0 documenting the breaking change
+
+## 5. Java 17 Code Modernizations
+
+### 5.1 Convert Data Classes to Records
+These immutable data classes are perfect candidates for records:
+- **CtrlData.java**: Convert to record (4 final fields with getters)
+- **CacheKey.java**: Convert to record with custom equals/hashCode if needed
+- **DatabaseRecord.java**: Convert to record (2 final fields)
+- **CachedConstructor.java**: Convert to record (4 final fields)
+
+### 5.2 Reflection & Performance Improvements
+- **Consider using MethodHandles**: Replace reflection-based constructor invocation with MethodHandles for better performance
+  - In `Decoder.java:437`: Convert `constructor.newInstance(parameters)` to MethodHandle
+  - Cache MethodHandles alongside constructors for repeated invocations
+- **Add @Stable annotations**: For frequently accessed final fields to enable JVM optimizations
+
+### 5.3 Pattern Matching for instanceof
+- **Decoder.java:161,322**: Use pattern matching to eliminate casts
+- **Networks.java:99**: Simplify instanceof checks
+- **Reader.java:302**: Use pattern matching
+
+### 5.4 Switch Expressions
+Convert switch statements that return values:
+- **Decoder.java:123,156,263,503,558**: Convert to switch expressions
+- **Reader.java:364**: Convert to switch expression
+
+### 5.5 Collection Factory Methods
+- Replace `Arrays.asList()` with `List.of()` for immutable lists
+- Replace array initializations with collection factories where appropriate
+
+### 5.6 Text Blocks
+- **Decoder.java:448-450**: Use text blocks for multi-line error messages
+
+## 6. Security & Safety Enhancements
+
+### 6.1 Reflection Security
+- Add security checks for reflection operations
+- Consider adding SecurityManager checks if needed
+- Document security implications of reflection usage
+
+### 6.2 Type Safety
+- Use sealed classes for Type enum if appropriate
+- Add stronger type checking with generics improvements
+
+## 7. Testing & Validation
+- Run full test suite with Java 17, 21, and 22
+- Verify checkstyle compliance
+- Performance benchmarking to ensure improvements
+- Security audit of reflection usage
+
+## Benefits of These Changes:
+1. **Performance**: MethodHandles are faster than reflection for repeated invocations
+2. **Memory**: Records have better memory layout and reduced boilerplate
+3. **Safety**: Pattern matching eliminates ClassCastException risks
+4. **Readability**: Switch expressions and records make code more concise
+5. **Maintainability**: Less boilerplate code to maintain
+6. **Security**: Better control over reflection operations
+
+This is a breaking change warranting version 4.0.0.