Skip to content

Conversation

@rjrudin
Copy link
Contributor

@rjrudin rjrudin commented Dec 11, 2025

Should have renamed this in a prior release.

Copilot AI review requested due to automatic review settings December 11, 2025 18:29
@github-actions
Copy link

github-actions bot commented Dec 11, 2025

Copyright Validation Results
Total: 2 | Passed: 2 | Failed: 0 | Skipped: 0 | at: 2025-12-11 18:32:30 UTC | commit: f0d4fdd

✅ Valid Files

  • marklogic-client-api/src/main/java/com/marklogic/client/DatabaseClientBuilder.java
  • marklogic-client-api/src/main/java/com/marklogic/client/impl/DatabaseClientPropertySource.java

✅ All files have valid copyright headers!

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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_CLOUD as the new constant and deprecated AUTH_TYPE_MARKLOGIC_CLOUD
  • Updated all internal usages to reference AUTH_TYPE_CLOUD instead 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";
Copy link

Copilot AI Dec 11, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Suggested change
public final static String AUTH_TYPE_MARKLOGIC_CLOUD = "cloud";
public final static String AUTH_TYPE_MARKLOGIC_CLOUD = AUTH_TYPE_CLOUD;

Copilot uses AI. Check for mistakes.
Should have renamed this in a prior release.
@rjrudin rjrudin merged commit 8c2f2eb into develop Dec 11, 2025
3 checks passed
@rjrudin rjrudin deleted the feature/deprecate branch December 11, 2025 18:55
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