aws / aws/aws-lambda-java-libs
How about some Builders in lambda-java-events
- Linguagem predominante
- Java
- Estrelas
- 548
- Forks
- 241
- Merge médio
- 2d 5min
- PRs com merge (30d)
- 11
Descrição
The core AWS Java/Kotlin libraries provide builders for just about every object. Builders abound! In the `aws-lambda-java-events` library we get sad old POJOs.
```Kotlin
val messageAttribute = SQSEvent.MessageAttribute()
messageAttribute.stringValue = "So Verbose!"
val sqsMessage = SQSEvent.SQSMessage()
sqsMessage.body = objectMapper.writeValueAsString(DummyMessage(name = "Bob"))
sqsMessage.messageAttributes["So-Much-Typing"] = messageAttribute // <<-- whoops, error, messageAttributes is null
```
I guess I could do this, but its not winning any awards:
```Kotlin
val messageAttribute = SQSEvent.MessageAttribute()
messageAttribute.stringValue = "Still have to do this on two lines"
val sqsMessage = SQSEvent.SQSMessage()
sqsMessage.body = objectMapper.writeValueAsString(DummyMessage(name = "Bob"))
sqsMessage.messageAttributes = mapOf("So-Much-Typing" to messageAttribute)
```
I'm using `SQSEvent` as an example because that's what I was working with today. But same goes for `SNSEvent`, `S3Event`, `ScheduleEvent`, `LexEvent`...
I see that `KafkaEvent` uses Lombok. I'd take that.
Thanks.
Guia de contribuição
Direção de pesquisa
Comece pelas classes SQSEvent e compare a construção atual de POJO com o ponto de entrada KafkaEvent baseado em Lombok. Revise as classes correspondentes SNSEvent, S3Event, ScheduleEvent e LexEvent; em seguida, defina uma abordagem consistente de Builder para todos os tipos de evento e verifique se objetos aninhados, como messageAttributes, podem ser construídos sem inicialização com null.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- aws, java, kotlin
- Domínio
- backend, cloud
- Tipo de issue
- Funcionalidade
- Dificuldade
- 5/5
- Tempo estimado
- Mais de uma semana
- Status de atividade
- Estagnada
- Clareza
- Razoavelmente clara
- Facilidade para iniciantes
- 35/100