MLE-22953 Merging release/7.2.0 into master but without copyright updates

[rjrudin]

This isolates the files (still a lot of them) that meaningfully changed in the 7.2.0 release. Will follow this up with copyright updates.

[Copilot]

Pull Request Overview

This PR temporarily reverts copyright changes to isolate files with meaningful functional changes. The purpose is to separate copyright updates from actual code modifications, making it easier to identify and review substantive changes.

Key changes include:

• Version updates from 7.0.0 to 7.2.0 across multiple configuration files
• Introduction of new vector utility classes and row template functionality
• Updates to dependency versions (Jackson, SLF4J, Kotlin)
• Modernization of Java code patterns and API usage

Reviewed Changes

Copilot reviewed 218 out of 223 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
pom.xml	Version bump to 7.2.0 and dependency updates
marklogic-client-api/src/main/java/com/marklogic/client/util/VectorUtil.java	New utility class for vector encoding/decoding
marklogic-client-api/src/main/java/com/marklogic/client/row/RowTemplate.java	New template class for simplified Optic API usage
ml-development-tools/build.gradle	Kotlin version update and new task additions
test-app/build.gradle	Dependency version updates and task modernization

Comments suppressed due to low confidence (1)

pom.xml:72

• Jackson version 2.19.0 does not exist. The latest available version in the 2.x series is 2.18.x. This will cause build failures.

      <version>2.19.0</version>

[Copilot]

The semicolon addition changes the behavior from returning 'result' to a statement that doesn't return anything. This breaks the module's intended functionality.

Suggested change

	result;
	return result;

[Copilot]

Removing 'new Integer()' constructor calls is correct as it was deprecated in Java 9, but the replacement should maintain the same Integer type. Consider using Integer.valueOf() if boxed integers are specifically needed.

Suggested change

	listSas.add(Arrays.asList(new Object[] { type_dnsName, passedSas[0] }));
	listSas.add(Arrays.asList(new Object[] { type_ipAddress, passedSas[1] }));
	listSas.add(Arrays.asList(new Object[] { type_dnsName, passedSas[2] }));
	listSas.add(Arrays.asList(new Object[] { type_dnsName, passedSas[3] }));
	listSas.add(Arrays.asList(new Object[] { Integer.valueOf(type_dnsName), passedSas[0] }));
	listSas.add(Arrays.asList(new Object[] { Integer.valueOf(type_ipAddress), passedSas[1] }));
	listSas.add(Arrays.asList(new Object[] { Integer.valueOf(type_dnsName), passedSas[2] }));
	listSas.add(Arrays.asList(new Object[] { Integer.valueOf(type_dnsName), passedSas[3] }));

[Copilot]

Using LaissezFaireSubTypeValidator.instance poses a security risk as it allows deserialization of any class. Consider using a more restrictive SubTypeValidator or validating the specific types that should be allowed.

[Copilot]

The replacement of deprecated newInstance() with getDeclaredConstructor().newInstance() is correct, but this change should handle potential exceptions like NoSuchMethodException and IllegalAccessException that weren't previously thrown.