这是indexloc提供的服务,不要输入任何密码
Skip to content

fix: add configurable bar for name truncation for partial files #8641

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

Conversation

theVerySharpFlat
Copy link

What is the purpose of this change?

Currently, rclone truncates paths over a hundred characters to fit a partial suffix. Sometimes, this comes around and bites you when it turns out the ending, say 10 characters, are the only thing distinguishing one file being transferred from another. Thus, I have added this flag to make this limit not hardcoded.

Checklist

  • I have read the contribution guidelines.
  • I have added tests for all changes in this PR if appropriate.
  • I have added documentation for the changes if appropriate.
  • All commit messages are in house style.
  • I'm done, this Pull Request is ready for review :-)

@ncw
Copy link
Member

ncw commented Jun 26, 2025

Rclone adds a hash of the file fingerprint to create a unique suffix. I guess if you have a lot of files with the same size/modtime/hash this could cause a problem.

Perhaps rclone should include the filename in the hash too? That might be an easier fix.

@theVerySharpFlat
Copy link
Author

theVerySharpFlat commented Jun 26, 2025

Rclone adds a hash of the file fingerprint to create a unique suffix. I guess if you have a lot of files with the same size/modtime/hash this could cause a problem.

Perhaps rclone should include the filename in the hash too? That might be an easier fix.

Yeah, our usecase is tens of terrabytes of data spread across tens to hundreds of millions of files where checksumming is a bottleneck, so we have that disabled. We also have very long and similar filenames

I'd rather have the thing fail due to exceeding PATH_MAX instead of writing to the same partial file concurrently

@theVerySharpFlat
Copy link
Author

Perhaps rclone should include the filename in the hash too? That might be an easier fix.

In the fingerprint, yes. I'll sub a PR for that

@theVerySharpFlat
Copy link
Author

closing in favor of #8643

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants