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

Create a mode that does not generate intermediate files in SftpUpload. #457

Merged
merged 1 commit into from
Jul 23, 2024

Conversation

nissy0409240
Copy link
Collaborator

Brief

Create a mode that does not generate intermediate files in SftpUpload.

Points to Check

  • Is there any discomfort in the corresponding parts and contents?

Test

Confirmed

I was able to transfer files using the modified SftpUpload class.
I was able to transfer files by adding the put_intermediation option.
I was able to transfer files by specifying None for the put_intermediation option.
The contents of the file did not change before and after the transfer.
I confirmed the above.

Review Limit

  • As soon as possible.

@nissy0409240 nissy0409240 self-assigned this Jul 22, 2024

_logger = logging.getLogger(__name__)

if _logger.isEnabledFor(logging.DEBUG):
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I removed it because it was only used for debugging.

@@ -40,6 +40,7 @@ python-gnupg = "==0.4.8"
openpyxl = "==3.0.9"
pyminizip = "==0.2.5"
psycopg2 = "==2.9.1"
numpy = "<2"
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I also made some changes to this p-r to pass the unit test.
If there are any conflicts, I will deal with them.

@yasuhiro-ohba yasuhiro-ohba merged commit 3706f28 into master Jul 23, 2024
5 checks passed
@yasuhiro-ohba yasuhiro-ohba deleted the 456 branch July 23, 2024 16:08
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.

Create a mode that does not generate intermediate files in SftpUpload.
2 participants