UTF8 Decoding depends on customer content type about sagemaker-inference-toolkit HOT 2 OPEN

@ericangelokim,

What do you recommend?

I think when this was written it was to mimc this line in the SageMaker containers SDK functionality: https://github.com/aws/sagemaker-containers/blob/master/src/sagemaker_containers/_worker.py#L196

Can you reference an issue as an example?

Feel free to ping me if it is private.

Thanks!

from sagemaker-inference-toolkit.

I think you need to relinquish this validation to the customer. For example (personal case), this line fails if another encoding is sent but the customer sets content_type to be CSV. There isn't a good way to respond to the client with the root cause of the error.

I am working on another PR that adds error handling back to client, so we can work on that first and see if this situation changes.

from sagemaker-inference-toolkit.