Azure / Azure/azure-sdk-for-python

Clean up AnalyzeTextAsyncLROPoller logic before GA/next beta

Aperta
#42,935 0 commenti 0 reazioni 1 assegnatario Rivendicata da @amber-Chen-86 Vedi su GitHub
Client Cognitive - Language Cognitive - Text Analytics
Lingua principale
Python
Stelle
5.6k
Fork
3.4k
Merge medio
2g
PR unite (30g)
217

Descrizione

During PR review (https://github.com/Azure/azure-sdk-for-python/pull/42685), reviewers flagged the complexity of the patched poller used to expose details in AnalyzeTextAsyncLROPoller (and the sync peer).
It should be relatively straightforward to patch a poller to expose the `details` property, and we should reduce paging complexity while preserving `ItemPaged/AsyncItemPaged` behavior.

Another refactor(PR: https://github.com/Azure/azure-sdk-for-python/pull/42928): Once the LRO finishes, should any actions still be “in progress”? If terminal LRO ⇒ in_progress should be 0, we likely shouldn’t surface this on the result page.

We’ll aim to simplify in the next beta or GA.

Guida per i contributori

Apri la guida per i contributori

Valutazione

Questa issue non è ancora stata valutata.

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.