Merge More Main Changes to Azure Storage v12 Branch

Azure / durabletask

Durable Task Framework allows users to write long running persistent workflows in C# using the async/await capabilities.

Apache License 2.0

1.51k stars 290 forks source link

Merge More Main Changes to Azure Storage v12 Branch #930

Closed wsugarman closed 1 year ago

wsugarman commented 1 year ago

Integrate latest changes from main including the new table partition manager into the feature branch
Refactor TablePartitionManager to use the Table class instead of the TableClient directly
Rename TableLease to TablePartitionLease to align with BlobPartitionLease
Some async testing refactors

This change should not be squashed as I will eventually need to merge with main too.

cgillum commented 1 year ago

Adding @nytian to help review this PR and answer Will’s questions.

wsugarman commented 1 year ago

The change looks good to me. I just have a problem about changing TableClient to Table class. Why are we doing this, I assume is it because we cannot apply Azure.Data.Tables packages? Thanks!

Oh, I changed it as all other interactions with Azure Storage, including the existing partition manager, leverage these client wrappers. For example, the BlobPartitionManager used by the LegacyPartitionManager uses the internal Blob type. Probably the biggest advantage is that they are configured with various policies for retrying and logging connection attempts.