Re: [PR] Delete database API [pinot]

2024-04-02 Thread via GitHub
gortiz merged PR #12765: URL: https://github.com/apache/pinot/pull/12765 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@pinot.apach

Re: [PR] Delete database API [pinot]

2024-04-02 Thread via GitHub
shounakmk219 commented on code in PR #12765: URL: https://github.com/apache/pinot/pull/12765#discussion_r1548240789 ## pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotDatabaseRestletResource.java: ## @@ -59,4 +67,91 @@ public class PinotDatabaseRest

Re: [PR] Delete database API [pinot]

2024-04-02 Thread via GitHub
gortiz commented on code in PR #12765: URL: https://github.com/apache/pinot/pull/12765#discussion_r1548226272 ## pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotDatabaseRestletResource.java: ## @@ -59,4 +67,91 @@ public class PinotDatabaseRestletRes

Re: [PR] Delete database API [pinot]

2024-04-02 Thread via GitHub
shounakmk219 commented on code in PR #12765: URL: https://github.com/apache/pinot/pull/12765#discussion_r1547716136 ## pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotDatabaseRestletResource.java: ## @@ -59,4 +67,75 @@ public class PinotDatabaseRest

Re: [PR] Delete database API [pinot]

2024-04-02 Thread via GitHub
gortiz commented on code in PR #12765: URL: https://github.com/apache/pinot/pull/12765#discussion_r1547450666 ## pinot-controller/src/main/java/org/apache/pinot/controller/api/resources/PinotDatabaseRestletResource.java: ## @@ -59,4 +67,75 @@ public class PinotDatabaseRestletRes

Re: [PR] Delete database API [pinot]

2024-04-01 Thread via GitHub
codecov-commenter commented on PR #12765: URL: https://github.com/apache/pinot/pull/12765#issuecomment-2029994316 ## [Codecov](https://app.codecov.io/gh/apache/pinot/pull/12765?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&u

[PR] Delete database API [pinot]

2024-04-01 Thread via GitHub
shounakmk219 opened a new pull request, #12765: URL: https://github.com/apache/pinot/pull/12765 # Description API to delete the given database which effectively deletes all the tables under the given database. The API has a sync behaviour which means partial database deletion is possi