Skip to content

Fix python actions#243

Open
sofia-tekdatum wants to merge 11 commits intomainfrom
fix_python_actions
Open

Fix python actions#243
sofia-tekdatum wants to merge 11 commits intomainfrom
fix_python_actions

Conversation

@sofia-tekdatum
Copy link
Collaborator

@sofia-tekdatum sofia-tekdatum commented Feb 5, 2026

Fix two remaining python github actions

ARM64 macOS 14 Python 3
Before
After

AMD64 Windows 2022 Python 3.13
Before
After

Both are now failing with an unrelated Panda test and the unrelated s3 dataset failure

@github-actions
Copy link

github-actions bot commented Feb 5, 2026

Thanks for opening a pull request!

If this is not a minor PR. Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose

Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project.

Then could you also rename the pull request title in the following format?

GH-${GITHUB_ISSUE_ID}: [${COMPONENT}] ${SUMMARY}

or

MINOR: [${COMPONENT}] ${SUMMARY}

See also:

Comment on lines +140 to +148
ExternalDBPAEncryptorAdapterFactory() = default;
ExternalDBPAEncryptorAdapterFactory(const ExternalDBPAEncryptorAdapterFactory&) =
delete;
ExternalDBPAEncryptorAdapterFactory& operator=(
const ExternalDBPAEncryptorAdapterFactory&) = delete;
ExternalDBPAEncryptorAdapterFactory(ExternalDBPAEncryptorAdapterFactory&&) = default;
ExternalDBPAEncryptorAdapterFactory& operator=(ExternalDBPAEncryptorAdapterFactory&&) =
default;

Copy link
Collaborator

Choose a reason for hiding this comment

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

why? (again, just curiosity)

Copy link
Collaborator

Choose a reason for hiding this comment

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

(and maybe a comment is granted in the code?)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants