Enable AWS Lambda context object compatibility

[ksatzke]

KNIX python function handlers have a context object just like AWS Lambda python and Java function handlers have, but some of the properties of the corresponding AWS context object are still missing in the KNIX implementation.

This implementation task will be performed in the feature/AWS_API_compatibility branch, targeting full Lambda object compatibility for KNIX.

[Istemi Ekin Akkus]

From the changes I saw, it looks like we also need to update the API object for the Java functions in KNIX.

Are there other changes/additions that need to be done? If not, I can take the modified Python API and apply it for Java functions.

[ksatzke]

Hi Ekin, as you can see in the implementation a lot of the emulated context object properties are hardcoded. One thing that comes into mind immediately is a proper support of function versioning/aliasing - which still needs to be added to KNIX. BR Klaus From: Istemi Ekin Akkus <[e-mail]> Sent: Thursday, September 24, 2020 3:53 PM To: knix-microfunctions/knix <[e-mail]> Cc: Satzke, Klaus (Nokia - DE/Stuttgart) <[e-mail]>; Assign <[e-mail]> Subject: Re: [knix-microfunctions/knix] Enable AWS Lambda context object compatibility (#80) From the changes I saw, it looks like we also need to update the API object for the Java functions in KNIX. Are there other changes/additions that need to be done? If not, I can take the modified Python API and apply it for Java functions. — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub<#80 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ADFTJENP7M46JII5SXL7YA3SHNFLLANCNFSM4PVSC4IA>.

[Istemi Ekin Akkus]

I meant something like this: https://docs.aws.amazon.com/en_us/lambda/latest/dg/java-context.html

[ksatzke]

Hi Ekin, when looking at https://docs.aws.amazon.com/en_us/lambda/latest/dg/python-context.html for comparison the context object for Java looks quite different, e.g. in terms of use method names. Plus, there are no properties in the context object for Java at all. Could it be handled with a simple case distinction in MicrofunctionsAPI.py? I thinks I’ve made changes only to this file. BR Klaus From: Istemi Ekin Akkus <[e-mail]> Sent: Thursday, September 24, 2020 5:05 PM To: knix-microfunctions/knix <[e-mail]> Cc: Satzke, Klaus (Nokia - DE/Stuttgart) <[e-mail]>; Assign <[e-mail]> Subject: Re: [knix-microfunctions/knix] Enable AWS Lambda context object compatibility (#80) I meant something like this: https://docs.aws.amazon.com/en_us/lambda/latest/dg/java-context.html — You are receiving this because you were assigned. Reply to this email directly, view it on GitHub<#80 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ADFTJEMBPBEXDEVFPPKDS4TSHNNYVANCNFSM4PVSC4IA>.

[Istemi Ekin Akkus]

I added the modifications necessary for the Java worker, but it needs testing.

[Istemi Ekin Akkus]

Added Java context test as well.

I am not sure whether we are going to support different versions etc. as intended, but at this point, we wouldn't have to worry about existing code breaking due to migration to knix.

I think we can merge this.

@ksatzke : Could you open a PR please?

[ksatzke]

Hi Ekin, thanks for this – OK, will do. BR Klaus From: Istemi Ekin Akkus <[e-mail]> Sent: Friday, September 25, 2020 11:07 AM To: knix-microfunctions/knix <[e-mail]> Cc: Satzke, Klaus (Nokia - DE/Stuttgart) <[e-mail]>; Mention <[e-mail]> Subject: Re: [knix-microfunctions/knix] Enable AWS Lambda context object compatibility (#80) Added Java context test as well. I am not sure whether we are going to support different versions etc. as intended, but at this point, we wouldn't have to worry about existing code breaking due to migration to knix. I think we can merge this. @ksatzke<https://github.com/ksatzke> : Could you open a PR please? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#80 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ADFTJEPMHDSC2UJ55J4V2ELSHRMTNANCNFSM4PVSC4IA>.