Ignore unboxed types for subset of services

awslabs / aws-sdk-kotlin

Multiplatform AWS SDK for Kotlin

Apache License 2.0

398 stars 49 forks source link

Ignore unboxed types for subset of services #261

Closed aajtodd closed 3 years ago

aajtodd commented 3 years ago

We need to implement a customization to pre-process a subset of AWS models such that we apply @boxed trait to all primitive (unboxed) members. Waiting on list of services from Jordon. This ticket is a placeholder for that work.

The rationale is that there are some services that need to be able to send default values and the current behavior w.r.t. boxed vs unboxed types is not consistent. This will be a stop gap, a more permanent fix will come in time.

aajtodd commented 3 years ago

Preliminary list

["AmazonNimbleStudio","AmplifyBackend","ApiGatewayManagementApi","ApiGatewayV2","DataExchange","Greengrass","IoT 1Click Projects","Kafka","Macie2","MediaConnect","MediaConvert","MediaLive","MediaPackage","MediaPackage Vod","MediaTailor","Pinpoint","Pinpoint SMS Voice","ServerlessApplicationRepository","mq","schemas"]

+ EC2

aajtodd commented 3 years ago

There is some talk that this may eventually get up-streamed so that we don't have to handle it. Unsure of timeline at the moment

aajtodd commented 3 years ago

WIP branch: https://github.com/awslabs/aws-sdk-kotlin/tree/box-services

Guidance given from Smithy team was that we can probably just wait for this to be up streamed so that we don't have to maintain it. Going to leave this task alone for now and revisit if it isn't available upstream before alpha launch.