amazon-connect / ai-powered-speech-analytics-for-amazon-connect

The AI Powered Speech Analytics for Amazon Connect solution provides the combination of speech to text transcription, translation into preferred languages, and insights for agents and supervisors all in real-time
https://aws.amazon.com/solutions/implementations/ai-powered-speech-analytics-for-amazon-connect/
Other
55 stars 54 forks source link

lambda.invokeAsync depracated #2

Closed czhao028 closed 4 years ago

czhao028 commented 5 years ago

Although it seems like this code is fairly recently built, the Java SDK says that lambda.invokeAsync is deprecated, which appears in index.js of kvs_trigger. If we're trying to deploy this is a lambda function ourselves (with a few modifications), should we swap out the invokeAsync with invoke(request)? Is there a different version that ran with the CloudFormation template?

czhao028 commented 5 years ago

CloudFormation template mentioned is deployment/AI-powered-speech-analytics-for-amazon-connect.template

efh365 commented 5 years ago

It looks like invokeAsync() is deprecated in the java sdk, but not in the js sdk. Since this is used in the kvs_trigger function, which is js, it should be ok to use. That is what we have used throughout our development and testing.

js: https://docs.aws.amazon.com/AWSJavaScriptSDK/latest/AWS/Lambda.html#invokeAsync-property java: https://docs.aws.amazon.com/AWSJavaSDK/latest/javadoc/com/amazonaws/services/lambda/AWSLambda.html#invokeAsync-com.amazonaws.services.lambda.model.InvokeAsyncRequest-