Skip to content

Initial code update for delete cluster endpoint #184

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
akashb38 wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Initial code update for delete cluster endpoint #184

akashb38 wants to merge 1 commit into from

Conversation

@akashb38
Copy link
Contributor

@akashb38 akashb38 commented Aug 1, 2023 •
edited
Loading

DataPull PR

<High level description of the PR>

Added

  • <detail item of what was added>
  • <describe functionality added>

Changed

  • <detail item of what was changed>
  • _<describe functionality that was changed>
  • _<in particular, describe BACKWARD INCOMPATIBLE changes>

Deleted

  • <detail item of what was removed>

PR Checklist Forms

  • CHANGELOG.md updated
  • Reviewer assigned
  • PR assigned (presumably to submitter)
  • Labels added (enhancement, bug, documentation)

@akashb38
Copy link
Contributor Author

akashb38 commented Aug 1, 2023

Please review the code and provide your feedback on the changes. Thanks

@akashb38 akashb38 closed this Aug 1, 2023
@akashb38 akashb38 reopened this Aug 1, 2023
@akashb38
Copy link
Contributor Author

akashb38 commented Aug 1, 2023

Please review the code and provide your feedback.

srinugajjala
srinugajjala reviewed Apr 23, 2025
View reviewed changes
api/src/main/java/com/homeaway/datapullclient/process/DataPullRequestProcessor.java
}

private void delCluster(String clusterId) {

Copy link
Collaborator

@srinugajjala srinugajjala Apr 23, 2025

Choose a reason for hiding this comment

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

Where is the implementation for this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@srinugajjala srinugajjala srinugajjala left review comments

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@akashb38 @srinugajjala