[API Review] Auto-gen layer queue

• CorsRule getMaxAgeInSeconds() why not milliseconds and return long

• Should these classes (e.g. EqueuedMessage) be made immutable? Remove setter and final member data.

• Rename Logging to LogData?

• isIncludeAPIs() in Metrics return boolean instead of Boolean

• setName in QueueItem rename to setQueueName

• QueueMessage class is not used anywhere, so moved to implementation.

• getMaxResults() in QueueItem return int instead of Integer?

• Extract StorageException to common module.

• setLogging in StorageServiceProperties whether can make it clear to differentiate from client-side logging.

• The necessity of StorageServiceStats as it only has one property field in class.

Reference: https://apiview.azurewebsites.net/Assemblies/Review/2cefcabd89fe4b008f29c7300ba1c3c2

[sima-zhu]

Regarding: CorsRule getMaxAgeInSeconds() why not milliseconds and return long

and

Regarding: The necessity of StorageServiceStats as it only has one property field in class.

Refers Rick’s response, there are no action item for SDK.

Update: Oct/15: This is no action, see above comments

[sima-zhu]

Regarding: setLogging in StorageServiceProperties whether can make it clear to differentiate from client-side logging.

This will be implicitly addressed once we decide on this.

Update: Oct/15: Resolved in the PR https://github.com/Azure/azure-sdk-for-java/pull/5868