Skip to content

Conversation

@kushagraThapar
Copy link
Member

@kushagraThapar kushagraThapar commented Mar 24, 2021

  • Removed Beta tag from handle, and readMany APIs

* @return set of regions contacted for this request
*/
@Beta(value = Beta.SinceVersion.V4_9_0, warningText = Beta.PREVIEW_SUBJECT_TO_CHANGE_WARNING)
public Set<URI> getRegionsContacted() {
Copy link
Contributor

@moderakh moderakh Mar 24, 2021

Choose a reason for hiding this comment

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

can we postpone GAing the getRegionsContacted() API for now.
I suspect we may need to return a different type. perhaps a Set<URL> or Set<String>

Copy link
Member

Choose a reason for hiding this comment

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

Also, the feedResponseDiagnostics doesnt return all the values for this and currently just null. So its worth waiting to fix that part too for GA

Copy link
Member Author

Choose a reason for hiding this comment

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

@moderakh - makes sense, reverted this API back to Beta.

Copy link
Contributor

@moderakh moderakh left a comment

Choose a reason for hiding this comment

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

I am signing off pending one comment on the getRegionsContacted().

"can we postpone GAing the getRegionsContacted() API for now.
I suspect we may need to return a different type. perhaps a Set<URL> or Set<String>"

@kushagraThapar kushagraThapar changed the title Removed Beta tag from handle, getRegionsContacted, and readMany APIs Removed Beta tag from handle, and readMany APIs Mar 24, 2021
@kushagraThapar kushagraThapar merged commit 4802792 into Azure:master Mar 24, 2021
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-java that referenced this pull request Aug 5, 2022
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-java that referenced this pull request Aug 5, 2022
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-java that referenced this pull request Aug 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants