Skip to content

Conversation

@rjrudin
Copy link
Contributor

@rjrudin rjrudin commented Dec 1, 2025

This is just scratching the surface - next step will be to modify GenericFileLoader to accept a DatabaseClientSupplier instead of a DatabaseClient. That will allow the commands for loading modules, data, and schemas to only construct a client when necessary instead of eagerly.

@github-actions
Copy link

github-actions bot commented Dec 1, 2025

Copyright Validation Results
Total: 4 | Passed: 4 | Failed: 0 | Skipped: 0 | at: 2025-12-01 21:05:20 UTC | commit: b10f2b0

✅ Valid Files

  • ml-app-deployer/src/main/java/com/marklogic/appdeployer/command/modules/LoadModulesCommand.java
  • ml-app-deployer/src/main/java/com/marklogic/appdeployer/command/plugins/InstallPluginsCommand.java
  • ml-app-deployer/src/main/java/com/marklogic/appdeployer/command/security/InsertCertificateHostsTemplateCommand.java
  • ml-javaclient-util/src/main/java/com/marklogic/client/ext/helper/DatabaseClientSupplier.java

✅ All files have valid copyright headers!

@rjrudin rjrudin force-pushed the feature/25666-initial-lazy branch from 8667b44 to 05a7674 Compare December 1, 2025 21:04
This is just scratching the surface - next step will be to modify GenericFileLoader to accept a DatabaseClientSupplier instead of a DatabaseClient. That will allow the commands for loading modules, data, and schemas to only construct a client when necessary instead of eagerly.
@rjrudin rjrudin force-pushed the feature/25666-initial-lazy branch from 05a7674 to b10f2b0 Compare December 1, 2025 21:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants