You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by "ArafatKhan2198 (via GitHub)" <gi...@apache.org> on 2024/10/20 14:21:34 UTC
[PR] HDDS-11561. Refactor Open Key Search Endpoint and Consolidate with OmDBInsightEndpoint Using StartPrefix Parameter. [ozone]
ArafatKhan2198 opened a new pull request, #7336:
URL: https://github.com/apache/ozone/pull/7336
## What changes were proposed in this pull request?
This pull request introduces the following changes:
1. Addition of startPrefix to `/open` endpoint:
The `startPrefix` parameter has been added to the `/open` endpoint to enable filtering of open keys based on a specific prefix. This change mirrors the behaviour implemented for the /deletePending endpoint and improves search efficiency for open keys.
2. Consolidation of Search Functionality:
The search functionality for open keys, previously handled in the `OmDBInsightsSearchEndpoint`, has been fully integrated into the `/open` endpoint in `OMDBInsightEndpoint`. The `OmDBInsightsSearchEndpoint` file has been removed as it is no longer needed.
3. Test Class Renaming:
The test class `TestOmDBSearchEndpoint` has been renamed to `TestOpenKeysSearchEndpoint` to better reflect its purpose, as it now only tests the open keys search functionality.
## What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-11561
## How was this patch tested?
- Unit tests were updated to reflect the changes made to the /open endpoint.
- The renamed TestOpenKeysSearchEndpoint class has been run to verify the correctness of the open key search functionality.
- Manual testing was performed to ensure that both /open and /deletePending endpoints handle startPrefix correctly and return expected results.
--
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: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
Re: [PR] HDDS-11561. Refactor Open Key Search Endpoint and Consolidate with OmDBInsightEndpoint Using StartPrefix Parameter. [ozone]
Posted by "ArafatKhan2198 (via GitHub)" <gi...@apache.org>.
ArafatKhan2198 commented on PR #7336:
URL: https://github.com/apache/ozone/pull/7336#issuecomment-2425010277
@devmadhuu @sumitagrawl please take a look.
--
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: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
Re: [PR] HDDS-11561. Refactor Open Key Search Endpoint and Consolidate with OmDBInsightEndpoint Using StartPrefix Parameter. [ozone]
Posted by "nandakumar131 (via GitHub)" <gi...@apache.org>.
nandakumar131 commented on PR #7336:
URL: https://github.com/apache/ozone/pull/7336#issuecomment-2480372448
@ArafatKhan2198, can you please resolve the merge conflict?
--
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: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
Re: [PR] HDDS-11561. Refactor Open Key Search Endpoint and Consolidate with OmDBInsightEndpoint Using StartPrefix Parameter. [ozone]
Posted by "devmadhuu (via GitHub)" <gi...@apache.org>.
devmadhuu merged PR #7336:
URL: https://github.com/apache/ozone/pull/7336
--
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: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org