Skip to content
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

Release Python GIL during slow load I/O #8066

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Release Python GIL during slow load I/O

0dd171c
Select commit
Loading
Failed to load commit list.
Open

Release Python GIL during slow load I/O #8066

Release Python GIL during slow load I/O
0dd171c
Select commit
Loading
Failed to load commit list.
DCO / DCO required action Apr 11, 2024 in 0s

DCO

There is one commit incorrectly signed off. This means that the author of this commit failed to include a Signed-off-by line in the commit message.

To avoid having PRs blocked in the future, always include Signed-off-by: Author Name <[email protected]> in every commit message. You can also do this automatically by using the -s flag (i.e., git commit -s).

Here is how to fix the problem so that this code can be merged.


Preferred method: Commit author adds a DCO remediation commit

A DCO Remediation Commit contains special text in the commit message that applies a missing Signed-off-by line in a subsequent commit. The primary benefit of this method is that the project’s history does not change, and there is no risk of breaking someone else’s work.

These authors can unblock this PR by adding a new commit to this branch with the following text in their commit message:

Andrew Bolin <[email protected]>

DCO Remediation Commit for Andrew Bolin <[email protected]>

I, Andrew Bolin <[email protected]>, hereby add my Signed-off-by to this commit: 0dd171c1ba08b08bbdcf80992013d732fb007c89

Signed-off-by: Andrew Bolin <[email protected]>

Please note: You should avoid adding empty commits (i.e., git commit -s --allow-empty), because these will be discarded if someone rebases the branch / repo.


Alternate method: An employer adds a DCO Remediation Commit

If the contents of this commit were contributed on behalf of a third party (generally, the author’s employer), an authorized individual may add a Third-Party DCO Remediation Commit. This may be necessary if the original author is unavailable to add their own DCO Remediation Commit to this branch.

If you are about to add a Third-Party DCO Remediation Commit under DCO section (b) or (c), be sure you are authorized by your employer to take this action. Generally speaking, maintainers and other project contributors cannot sign off on behalf of project contributors, unless there is some relationship which permits this action. It is your responsibility to verify this.

This PR can be unblocked by an authorized third party adding one or more new commits to this branch with the following text in the commit message. Replace YOUR_COMPANY with your company name, YOUR_NAME with the name of the authorized representative, and YOUR_EMAIL with the representative’s email address.

For the sake of clarity, please use a separate commit per author:

On behalf of Andrew Bolin <[email protected]>

Third-Party DCO Remediation Commit for Andrew Bolin <[email protected]>

On behalf of Andrew Bolin <[email protected]>, I, YOUR_NAME <YOUR_EMAIL>, hereby add my Signed-off-by to this commit: 0dd171c1ba08b08bbdcf80992013d732fb007c89

Signed-off-by: YOUR_NAME <YOUR_EMAIL>

Least preferred method: Rebase the branch

If you have a local git environment and meet the criteria below, one option is to rebase the branch and add your Signed-off-by lines in the new commits. Please note that if others have already begun work based upon the commits in this branch, this solution will rewrite history and may cause serious issues for collaborators (described in the git documentation under "The Perils of Rebasing").

You should only do this if:

  • You are the only author of the commits in this branch
  • You are absolutely certain nobody else is doing any work based upon this branch
  • There are no empty commits in the branch (for example, a DCO Remediation Commit which was added using --allow-empty)

To add your Signed-off-by line to every commit in this branch:

  1. Ensure you have a local copy of your branch by checking out the pull request locally via command line.
  2. In your local branch, run: git rebase HEAD~1 --signoff
  3. Force push your changes to overwrite the branch: git push --force-with-lease origin pyxrt-release-gil

Summary

Commit sha: 0dd171c, Author: Andrew Bolin, Committer: Andrew Bolin; The sign-off is missing.