Skip to content

Stg105/photon on by default - #50477

Draft
browndav-msft wants to merge 6 commits into
Azure:feature/storage/stg105basefrom
browndav-msft:stg105/photonOnByDefault
Draft

browndav-msft wants to merge 6 commits into
Azure:feature/storage/stg105basefrom
browndav-msft:stg105/photonOnByDefault

Conversation

@browndav-msft

Copy link
Copy Markdown
Member

Summary

  • Changes the default Blob listing response format from XML to Apache Arrow.
  • Treats unset and AUTO formats as Arrow while preserving explicit XML.
  • Centralizes ListBlobsOptions copying and default application across sync/async flat and hierarchical listings.
  • Preserves options when pagination overrides the page size.
  • Adds request-level tests verifying the generated Accept header for all listing paths (null, unset, ARROW, AUTO, and XML`)
  • Removes the obsolete error-response-body deserialization test.

Why

Blob listing should use Apache Arrow by default.

Testing

  • Added 16 cases covering null, unset, ARROW, AUTO, and XML` across sync/async flat and hierarchical listings.
  • Ran targeted tests and Checkstyle successfully.

@github-actions github-actions Bot added the Storage Storage Service (Queues, Blobs, Files) label Sep 15, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
34 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@browndav-msft

Copy link
Copy Markdown
Member Author

Isabelle (@ibrandes) tests that are failing in CI right now aren't for the arrow enablement. Those have to do with issues with the service not supporting the features. So I don't know why this is happening. But this should be good now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Storage Storage Service (Queues, Blobs, Files)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant