MLE-25959 Deprecating AUTH_TYPE_MARKLOGIC_CLOUD#1860
Conversation
|
Copyright Validation Results ✅ Valid Files
✅ All files have valid copyright headers! |
There was a problem hiding this comment.
Pull request overview
This PR deprecates the AUTH_TYPE_MARKLOGIC_CLOUD constant in favor of the shorter AUTH_TYPE_CLOUD constant. The change maintains backward compatibility by keeping the deprecated constant while updating internal references to use the new naming convention.
Key changes:
- Introduced
AUTH_TYPE_CLOUDas the new constant and deprecatedAUTH_TYPE_MARKLOGIC_CLOUD - Updated all internal usages to reference
AUTH_TYPE_CLOUDinstead of the deprecated constant
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| DatabaseClientBuilder.java | Introduced new AUTH_TYPE_CLOUD constant, deprecated AUTH_TYPE_MARKLOGIC_CLOUD, and updated methods to use the new constant |
| DatabaseClientPropertySource.java | Updated switch case and conditional logic to reference AUTH_TYPE_CLOUD instead of deprecated constant |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| * @deprecated as of 8.1.0, use AUTH_TYPE_CLOUD instead | ||
| */ | ||
| @Deprecated | ||
| public final static String AUTH_TYPE_MARKLOGIC_CLOUD = "cloud"; |
There was a problem hiding this comment.
The deprecated constant AUTH_TYPE_MARKLOGIC_CLOUD should be updated to point to the new constant AUTH_TYPE_CLOUD to ensure consistency. Consider changing its value from \"cloud\" to reference AUTH_TYPE_CLOUD directly (e.g., AUTH_TYPE_MARKLOGIC_CLOUD = AUTH_TYPE_CLOUD). This ensures both constants remain synchronized if the underlying value ever needs to change.
| public final static String AUTH_TYPE_MARKLOGIC_CLOUD = "cloud"; | |
| public final static String AUTH_TYPE_MARKLOGIC_CLOUD = AUTH_TYPE_CLOUD; |
Should have renamed this in a prior release.
62392ef to
f0d4fdd
Compare
Should have renamed this in a prior release.