aws / aws/aws-lambda-java-libs

How about some Builders in lambda-java-events

Aberta
#566 0 comentários 0 reações 0 responsáveis Ver no GitHub
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

Abrir o 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

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.