-
Notifications
You must be signed in to change notification settings - Fork 59
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
change MAX_TARABLE_SYMLINK_PATH_LENGTH
to 200
#350
base: main
Are you sure you want to change the base?
Conversation
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.
Hiiii, @shmiwy welcome!馃帄 Thanks for taking the effort to make our project better! 馃檶 Keep making such awesome contributions!
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
Please fix CI failures. |
Symlink targets currently have a length limit of 100, which may not be sufficient in many test cases, so we change it to 200 here.
Symlink targets currently have a length limit of 100, which may not be sufficient in many test cases, so we change it to 200 here.
Change logs
change MAX_TARABLE_SYMLINK_PATH_LENGTH to 200
Why are the changes needed?
In many test cases, we found that the 100 limit was not enough.
Does this PR introduce any user-facing change?
No
How was this patch tested?
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
cloudberrydb/dev
team for review and approval when your PR is ready馃コ