Skip to content

Added encryption option to perf tests#29703

Merged
rickle-msft merged 2 commits intoAzure:mainfrom
rickle-msft:encryptionV2Perf
Jun 30, 2022
Merged

Added encryption option to perf tests#29703
rickle-msft merged 2 commits intoAzure:mainfrom
rickle-msft:encryptionV2Perf

Conversation

@rickle-msft
Copy link
Contributor

No description provided.

@ghost ghost added the Storage Storage Service (Queues, Blobs, Files) label Jun 29, 2022
@azure-sdk
Copy link
Collaborator

azure-sdk commented Jun 29, 2022

API change check

APIView has identified API level changes in this PR and created following API reviews.

azure-storage-blob-cryptography

Copy link
Contributor

@kasobol-msft kasobol-msft left a comment

Choose a reason for hiding this comment

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

LGTM

public class BlobPerfStressOptions extends StoragePerfStressOptions {

@Parameter(names = { "--encryption-version" })
private String encryptionVersion;
Copy link
Contributor

Choose a reason for hiding this comment

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

this can be enum, JCommander should be able to parse it.

See


for reference.

Make sure to default to null.

@rickle-msft rickle-msft merged commit 8d85737 into Azure:main Jun 30, 2022
kasobol-msft added a commit that referenced this pull request Jul 5, 2022
mssfang pushed a commit to mssfang/azure-sdk-for-java that referenced this pull request Jul 6, 2022
mssfang pushed a commit that referenced this pull request Jul 6, 2022
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.

3 participants