dynamodb: Guidance for map keys with dots in dynamodb.expression

[zombie-guru]

Please fill out the sections below to help us address your issue.

Version of AWS SDK for Go?

v1.19.15

Version of Go (go version)?

go version go1.12.1 darwin/amd64

What issue did you see?

Not possible with the expression package to delete map keys with . in them.

As an example, I would like to delete the My.Key.With.Dots in a M attribute named MyMap using the expression package. Some code is below. However, the REMOVE statement ends up being:

REMOVE #0.#1.#2.#3.#4

Happy to contribute a possible solution but was wondering if there are any solutions for this.

Steps to reproduce

package main

import (
	"fmt"

	"github.com/aws/aws-sdk-go/aws"
	"github.com/aws/aws-sdk-go/service/dynamodb/expression"
)

func main() {
	expr, err := expression.NewBuilder().
		WithUpdate(expression.Remove(expression.Name("MyMap.My.Key.With.Dots"))).
		Build()
}

[diehlaws]

Thanks for bringing this behavior to our attention @zombie-guru. Unfortunately what you are attempting to do is not possible at this time with the DynamoDB Expression Builder in the AWS SDK for Go. I'll bring this up with the rest of the team during our next sprint to discuss the best way to handle this case - if you'd like to submit a PR to fix this behavior please be sure to read our contribution guidelines before submitting the PR and we'd be happy to review it.

[vitaly-zdanevich]

My related question https://stackoverflow.com/questions/61022482/is-it-possible-to-set-to-map-when-key-contains-dot

About PR: as I understand before PR we need to design some new syntax.

[vitaly-zdanevich]

Please do not ignore this issue :(

[diehlaws]

Hi @vitaly-zdanevich, apologies for the delay in response on this issue. We've had an investigation task for this on our backlog for some time. I've brought it up with the rest of the team and we'll be reprioritizing investigating how to best implement this feature. Once we have more information on its progress I'll update the issue accordingly.

[vitaly-zdanevich]

Because this is also about condition expression.

[odannyc]

Hi, I am also running into this issue. Any updates?

[github-actions]

We have noticed this issue has not received attention in 1 year. We will close this issue for now. If you think this is in error, please feel free to comment and reopen the issue.