Storage - STG102 object level smart access by browndav-msft · Pull Request #48392 · Azure/azure-sdk-for-java

browndav-msft · 2026-03-12T01:18:44Z

No description provided.

…to PathProps

- We needed to add AccessTier smartAccessTier to all constructors in BlobProperties, which is different than what is recommended in BlobPropertiesInternal. Made a deliberate decision to go this route, but will go the other route, if necessessary

@test

@test @RequiredServiceVersion(clazz = DataLakeServiceVersion.class, min = "2026-02-04") public void getInferredTierWhenAssignedSmart() { // Arrange // Create a file in a file system with the smart tiering feature enabled, and set the access tier to smart. // Did this manually to prove feature works fc = premiumDataLakeServiceClient.getFileSystemClient("5333bf800setblobaccesstiersmart26f99030889cd17d8581") .getFileClient("5333bf801setblobaccesstiersmart26f71508c0c760a5e356"); Response<PathProperties> response = fc.getPropertiesWithResponse(null, null, null); HttpHeaders headers = response.getHeaders(); PathProperties properties = response.getValue(); validateBasicHeaders(headers); assertEquals(AccessTier.SMART, properties.getAccessTier()); assertEquals(AccessTier.HOT, properties.getSmartAccessTier()); }

...e/azure-storage-file-datalake/src/test/java/com/azure/storage/file/datalake/FileApiTest.java

…/azure-sdk-for-java into stg102/ObjectLevelSmartAccess

…owndav-msft/azure-sdk-for-java into stg102/ObjectLevelSmartAccess

ibrandes

had a couple more suggestions for further polishing

...orage/azure-storage-blob-batch/src/test/java/com/azure/storage/blob/batch/BatchApiTests.java

ibrandes · 2026-03-16T17:18:00Z

...om/azure/storage/blob/implementation/models/BlobPropertiesInternalConstructorProperties.java

+    @Override
+    public AccessTier getSmartAccessTier() {
+        return null;
+    }
+


this looks weird without context - maybe we should add a comment that says something like "Null because this property is set through BlobPropertiesConstructorProxy"

@ibrandes I had to trace this, but this should return smartAccessTier. It took me a while to figure it out, but I'm going to make the change to the code. The public API remains unchanged, but the service returns smartAccessTier, so this should return a value.

sdk/storage/azure-storage-blob/src/main/java/com/azure/storage/blob/models/BlobProperties.java

ibrandes · 2026-03-16T17:21:44Z

...ge/azure-storage-blob/src/test/java/com/azure/storage/blob/specialized/BlobBaseApiTests.java


+    @RequiredServiceVersion(clazz = BlobServiceVersion.class, min = "2026-02-06")
+    @Test
+    public void startCopyFromURLSmartAccessTier() {


we also need to add these tests to BlobBaseAsyncApiTests

ibrandes · 2026-03-16T17:31:37Z

sdk/storage/azure-storage-blob/src/test/java/com/azure/storage/blob/BlobAsyncApiTests.java

+    @RequiredServiceVersion(clazz = BlobServiceVersion.class, min = "2026-02-06")
+    @Test
+    public void uploadStreamAccessTierSmart() {
+
+        Flux<BlobItem> response
+            = primaryBlobServiceAsyncClient.createBlobContainer(generateContainerName()).flatMapMany(cc -> {
+                BlockBlobAsyncClient bc = cc.getBlobAsyncClient(generateBlobName()).getBlockBlobAsyncClient();
+                return bc.upload(DATA.getDefaultFlux(), DATA.getDefaultData().remaining())
+                    .then(bc.setAccessTierWithResponse(AccessTier.SMART, null, null))
+                    .flatMap(r -> {
+                        HttpHeaders headers = r.getHeaders();
+
+                        assertTrue(r.getStatusCode() == 200 || r.getStatusCode() == 202);
+                        assertNotNull(headers.getValue(X_MS_VERSION));
+                        assertNotNull(headers.getValue(X_MS_REQUEST_ID));
+                        return Mono.empty();
+                    })
+                    .then(bc.getProperties())
+                    .flatMap(r -> {
+                        assertEquals(AccessTier.SMART, r.getAccessTier());
+                        return Mono.empty();
+                    })
+                    .thenMany(cc.listBlobs());
+            });
+
+        StepVerifier.create(response)
+            .assertNext(r -> assertEquals(AccessTier.SMART, r.getProperties().getAccessTier()))
+            .verifyComplete();
+    }
+


this test is a lot more complex than it needs to be - if you use a blobclient instead of a blockblob you can use uploadWithResponse which lets us set the access tier and upload data at the same time:

@RequiredServiceVersion(clazz = BlobServiceVersion.class, min = "2026-02-06") @Test public void uploadStreamAccessTierSmart() { bc = ccAsync.getBlobAsyncClient(generateBlobName()); BlobParallelUploadOptions options = new BlobParallelUploadOptions(DATA.getDefaultFlux()) .setTier(AccessTier.SMART); Mono<Response<BlobProperties>> response = bc.uploadWithResponse(options) .then(bc.getPropertiesWithResponse(null)); StepVerifier.create(response) .assertNext(r -> { assertEquals(AccessTier.SMART, r.getValue().getAccessTier()); assertNotNull(r.getValue().getSmartAccessTier()); }) .verifyComplete(); }

ibrandes · 2026-03-16T17:32:42Z

...ge/azure-storage-file-datalake/src/main/java/com/azure/storage/file/datalake/Transforms.java

+    static final HttpHeaderName X_MS_ACCESS_TIER_INFERRED = HttpHeaderName.fromString("x-ms-access-tier-inferred");
+    static final HttpHeaderName X_MS_SMART_ACCESS_TIER = HttpHeaderName.fromString("x-ms-smart-access-tier");


i added these by mistake - these can be removed

ibrandes · 2026-03-16T17:33:00Z

...re-storage-file-datalake/src/test/java/com/azure/storage/file/datalake/DataLakeTestBase.java

    protected DataLakeFileSystemAsyncClient dataLakeFileSystemAsyncClient;
    protected DataLakeServiceClient primaryDataLakeServiceClient;
    protected DataLakeServiceAsyncClient primaryDataLakeServiceAsyncClient;
+    protected DataLakeServiceClient premiumDataLakeServiceClient;


this can be removed since we aren't adding tests for datalake

browndav-msft added 2 commits March 11, 2026 21:14

generate new api based on swaggerfile

b97fc2e

create base test for new api

b29e9ec

browndav-msft requested review from a team, alzimmermsft, gunjansingh-msft, ibrandes, kyleknap and seanmcc-msft as code owners March 12, 2026 01:18

github-actions bot added the Storage Storage Service (Queues, Blobs, Files) label Mar 12, 2026

browndav-msft and others added 11 commits March 12, 2026 10:43

switch to premiumstorageaccount for smart-tier

50b152e

recording for setblobaccesstiersmart

e8aa8ed

rerecording for setblobaccesstiersmart

216e1a7

updating assets

7b2cc60

add SMART access tier in enum, accessTierInferred and smartAcessTier …

85b2860

…to PathProps

create basic test for smart tier in datalake

2836cba

fix linting in PathProperties

2cb8e5c

changed boolean to Boolean to allow for null

3c5fb20

add premiumDataLakeServiceClient for smart tier

89e946e

browndav-msft commented Mar 13, 2026

View reviewed changes

...e/azure-storage-file-datalake/src/test/java/com/azure/storage/file/datalake/FileApiTest.java Outdated Show resolved Hide resolved

browndav-msft and others added 9 commits March 13, 2026 13:41

add params for smartAccessTier to javadocs

9a5344a

Merge branch 'feature/storage/stg102base' of https://github.com/Azure…

0e3d4db

…/azure-sdk-for-java into stg102/ObjectLevelSmartAccess

Merge branch 'stg102/ObjectLevelSmartAccess' of https://github.com/br…

eba47d8

…owndav-msft/azure-sdk-for-java into stg102/ObjectLevelSmartAccess

removing breaking changes

3be7588

Merge branch 'stg102/ObjectLevelSmartAccess' of https://github.com/br…

c3e54cc

…owndav-msft/azure-sdk-for-java into stg102/ObjectLevelSmartAccess

removing datalake breaking changes

85a4da1

add recordings for BlobBaseApiTests

03c31d9

Merge branch 'stg102/ObjectLevelSmartAccess' of https://github.com/br…

17de56d

…owndav-msft/azure-sdk-for-java into stg102/ObjectLevelSmartAccess

add recordings for BlobApiTests#uploadStreamAccessTierSmart

8f04fa2

browndav-msft added 2 commits March 15, 2026 17:46

add recordings for BlobAsyncApiTests#uploadStreamAccessTierSmart

1c6909f

add recordings for BlobAsyncApiTests#uploadStreamAccessTierSmart

8e5a135

ibrandes changed the title ~~Stg102/object level smart access~~ Storage - STG102 object level smart access Mar 16, 2026

ibrandes reviewed Mar 16, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Storage - STG102 object level smart access#48392

Storage - STG102 object level smart access#48392
browndav-msft wants to merge 24 commits intoAzure:feature/storage/stg102basefrom
browndav-msft:stg102/ObjectLevelSmartAccess

browndav-msft commented Mar 12, 2026

Uh oh!

Uh oh!

ibrandes left a comment

Uh oh!

Uh oh!

ibrandes Mar 16, 2026

Uh oh!

browndav-msft Mar 17, 2026

Uh oh!

Uh oh!

ibrandes Mar 16, 2026

Uh oh!

ibrandes Mar 16, 2026

Uh oh!

ibrandes Mar 16, 2026

Uh oh!

ibrandes Mar 16, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

		static final HttpHeaderName X_MS_ACCESS_TIER_INFERRED = HttpHeaderName.fromString("x-ms-access-tier-inferred");
		static final HttpHeaderName X_MS_SMART_ACCESS_TIER = HttpHeaderName.fromString("x-ms-smart-access-tier");

Conversation

browndav-msft commented Mar 12, 2026

Uh oh!

Uh oh!

ibrandes left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

ibrandes Mar 16, 2026

Choose a reason for hiding this comment

Uh oh!

browndav-msft Mar 17, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

ibrandes Mar 16, 2026

Choose a reason for hiding this comment

Uh oh!

ibrandes Mar 16, 2026

Choose a reason for hiding this comment

Uh oh!

ibrandes Mar 16, 2026

Choose a reason for hiding this comment

Uh oh!

ibrandes Mar 16, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants