-
Notifications
You must be signed in to change notification settings - Fork 86
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
Fix test artifact upload path incorrectly #240
Fix test artifact upload path incorrectly #240
Conversation
The commit information has been completed. I'll be happy to help you check if there are any further changes needed. @tuhaihe |
Hey @Zhangbaowen-Hashdata: I appreciate your effort, but the maximum limit for the commit body length is 72 characters instead of 50. Could you amend it again? |
In order to make it easier for users to quickly error logs, we have copied the entire directory to a fixed location and thoroughly tested it without encountering any issues.
3e3afb6
to
c691eef
Compare
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.
LGTM
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.
LGTM.
After this PR is merged, we can close #118.
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.
LGTM
In order to make it easier for users to quickly locate error logs, we have copied the entire directory to a fixed location and thoroughly tested it without encountering any issues.
closes: #ISSUE_Number
Change logs
Describe your change clearly, including what problem is being solved or what feature is being added.
If it has some breaking backward or forward compatibility, please clary.
Why are the changes needed?
Describe why the changes are necessary.
Does this PR introduce any user-facing change?
If yes, please clarify the previous behavior and the change this PR proposes.
How was this patch tested?
Please detail how the changes were tested, including manual tests and any relevant unit or integration tests.
Contributor's Checklist
Here are some reminders and checklists before/when submitting your pull request, please check them:
make installcheck
make -C src/test installcheck-cbdb-parallel