-
Notifications
You must be signed in to change notification settings - Fork 430
SCB-874:saga dubbo demo insert sql and readme optimization #276
Conversation
| 2. Enter the saga dubbo demo directory and give permissions to script | ||
| ``` | ||
| cd ./saga-demo/saga-dubbo-demo | ||
| chmod +x saga-dubbo-demo.sh |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I checked the file property of the sh file , it has the executor right.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It has no executor right when first use,or shall we use like bash saga-dubbo-demo.sh?
saga-demo/saga-dubbo-demo/README.md
Outdated
| <artifactId>mysql-connector-java</artifactId> | ||
| </dependency> | ||
| ``` | ||
| 2. remove alpha server's docker image |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The alpha server docker image could be overridden when we regenerate the docker image.
So we don't need to remove the alpha server docker image here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i will change it
| http://${host_address}:8071/serviceInvoke/BExceptionWhenAb | ||
| ``` | ||
| ## Debugging |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please keep the debugging information here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ok,i will change it
| 1. run the following command to create docker images in saga project root folder. | ||
| ``` | ||
| mvn clean package -DskipTests -Pdocker -Pdemo | ||
| mvn clean install -DskipTests -Pdocker -Pdemo |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
package doesn't install the jars into maven directory. We don't need to use install here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
when first use,if you do not use mvn install in saga root project folder,it will got an error when use mvn help:evaluate in saga-demo.sh to get version of docker image.
[INFO] Scanning for projects...
[INFO] mixin-maven-plugin: Merging Mixins
[ERROR] Cannot resolve mixin artifact org.apache.servicecomb.saga:docker-build-config:pom:0.3.0-SNAPSHOT: Error resolving artifact org.apache.servicecomb.saga:docker-build-config:pom:0.3.0-SNAPSHOT -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MavenExecutionException
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK,we can keep the change here.
https://issues.apache.org/jira/projects/SCB/issues/SCB-874?filter=allopenissues