-
Notifications
You must be signed in to change notification settings - Fork 8.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
“Test cases cannot achieve project readme functionality”Problem Solvi… #3430
base: 2022.x
Are you sure you want to change the base?
Conversation
…ng,Reason: When the file contents of the nacos file are being initialized, it has not been executed correctly and the configuration file is incorrect. Solution: It was discovered that only data source-related configurations exist in the file in nacos. Then I put other configuration content in the front position and re-executed the script. It was found that the content was successfully loaded. The consumer method name in the previously configured "bindings" was inconsistent with the actual code, and there was no configuration for"function.Fixes alibabagh-3427
pullBatchSize: 4 | ||
datasource: | ||
url: jdbc:mysql://integrated-mysql:3306/integrated_praise?useSSL=false&characterEncoding=utf8 |
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.
Try to keep as few changes as possible, this part seems unnecessary to modify.
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.
This part of the change should be necessary, because the content of the Nacos configuration file after initialization was inconsistent with the content of the configuration file when it was not changed before. However, once the position is replaced, the content will be consistent after execution. I don't know why this is the case, but it can solve the problem. You can try it out
Hello, I have some questions. As you described, I changed the configuration file to the configuration you provided, and then ran the project for testing. The result I got is still the same. Is there any other steps required? Also, I can make sure my example environment is fine url: Response
Configuration filesspring:
cloud:
stream:
function:
definition: consumer;
bindings:
consumer-in-0:
destination: PRAISE-TOPIC-01
content-type: application/json
group: praise-consumer-group-PRAISE-TOPIC-01
rocketmq:
binder:
name-server: rocketmq:9876
bindings:
consumer-in-0:
consumer:
pullInterval: 4000
pullBatchSize: 4
datasource:
url: jdbc:mysql://integrated-mysql:3306/integrated_praise?useSSL=false @China-MBAI |
You need to check if the configuration file on Nacos is consistent with my modifications. If it is inconsistent, please replace it with my configuration in the "integrated-consumer.yaml" file and then execute the "sh nacos-config-quick.sh" command in the "integrated-example/config-init/scripts/nacos-config-quick.sh" folder. After the command is executed, the configuration on Nacos will change, and then restart the project.After modifying the configuration file, you can also follow the instructions document "local-deployment.md" to execute step by step again |
of course, I made sure my config file got update and I restarted project. |
|
Have you ever executed this command:sh nacos-config-quick.sh |
yeay, I will try to do again in later. |
okay, got it. I tested locally and passed. LGTM! @China-MBAI |
“Test cases cannot achieve project readme functionality”Problem Solving,Reason: When the file contents of the nacos file are being initialized, it has not been executed correctly and the configuration file is incorrect. Solution: It was discovered that only data source-related configurations exist in the file in nacos. Then I put other configuration content in the front position and re-executed the script. It was found that the content was successfully loaded. The consumer method name in the previously configured "bindings" was inconsistent with the actual code, and there was no configuration for"function.Fixes gh-3427