aws-samples / serverless-patterns

Serverless patterns. Learn more at the website: https://serverlessland.com/patterns.
https://serverlessland.com
Other
1.48k stars 863 forks source link

New serverless pattern - dynamodb-lambda-elasticache #1735

Closed mansiyd closed 6 months ago

mansiyd commented 7 months ago

Issue #1734, if available:

Description of changes: New serverless pattern submission for implementing CQRS pattern using AWS serverless services

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

mansiyd commented 7 months ago

all review comments have been addressed

mavi888 commented 7 months ago

Getting an error when doing sam deploy

Error: Unable to upload artifact target/dynamodb-lambda-elasticache-1.0-SNAPSHOT.jar referenced by CodeUri parameter of OrderPublisher resource.
Parameter CodeUri of resource OrderPublisher refers to a file or folder that does not exist /serverless-patterns/dynamodb-lambda-elasticache/target/dynamodb-lambda-elasticache-1.0-SNAPSHOT.jar
mansiyd commented 7 months ago

@mavi888 just checking if you have performed a Maven build before doing sam deploy? If not, you would have to run the command "mvn clean install" to build the project which will create the jar file in the target folder, which can then be deployed using sam

mansiyd commented 7 months ago

@mavi888 could you please provide an ETA for the code review, since its open since 3 weeks and is blocking our other tasks that are dependent on this repo. Thanks

mavi888 commented 6 months ago

Just run the maven clean build and Im getting an error

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:compile (default-compile) on project dynamodb-lambda-elasticache: Fatal error compiling: java.lang.NoSuchFieldError: Class com.sun.tools.javac.tree.JCTree$JCImport does not have member field 'com.sun.tools.javac.tree.JCTree qualid' -> [Help 1]

Make sure that all the prerequistes are listed, maybe im missing something.

mansiyd commented 6 months ago

@mavi888 it seems to be an issue with the Java version. the compiler version is 11, as mentioned in the pom. can we have a working session, so we can resolve all issues in a single go?

jbesw commented 6 months ago

Unfortunately we don't have the resources to help troubleshoot this. Patterns should be simple, low-code, IaC frameworks to connect services. I would recommend submitting this as a pattern instead if you are able to fix the issues.

mansiyd commented 6 months ago

Closing this PR, as we have published as a repo instead