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

ARROW-17697: [Python] Fix Cython warning in types.pxi #14280

Merged
merged 2 commits into from
Oct 3, 2022
Merged

ARROW-17697: [Python] Fix Cython warning in types.pxi #14280

merged 2 commits into from
Oct 3, 2022

Conversation

milesgranger
Copy link
Contributor

@milesgranger milesgranger commented Sep 30, 2022

Will fix ARROW-17697

  • Turn warnings into errors
  • Change signature of DataType.field as others like StructType.field allowed taking int or str.

@github-actions
Copy link

@AlenkaF
Copy link
Member

AlenkaF commented Sep 30, 2022

I think the changes are clear:

  • flag in cmake for Cython compilation that turns warnings to errors,
  • changing signatures to fix for reported warnings in pyarrow/types.pxi when compiling PyArrow.

And just to check my understanding: you used cpdef here because you changed def and we want it to be callable from Python? And the error was happening due to the field method being a cdef function in the DataType superclass?

@milesgranger
Copy link
Contributor Author

Yepsi pepsi, that's right. I have nothing to add to your understanding. :)

@pitrou pitrou merged commit d8d0323 into apache:master Oct 3, 2022
@milesgranger milesgranger deleted the ARROW-17697_cython-warning-in-types branch October 3, 2022 16:21
@ursabot
Copy link

ursabot commented Oct 3, 2022

Benchmark runs are scheduled for baseline = 0aacc28 and contender = d8d0323. d8d0323 is a master commit associated with this PR. Results will be available as each benchmark for each run completes.
Conbench compare runs links:
[Finished ⬇️0.0% ⬆️0.0%] ec2-t3-xlarge-us-east-2
[Failed ⬇️0.56% ⬆️0.0%] test-mac-arm
[Failed ⬇️0.0% ⬆️0.0%] ursa-i9-9960x
[Finished ⬇️0.89% ⬆️0.04%] ursa-thinkcentre-m75q
Buildkite builds:
[Finished] d8d0323f ec2-t3-xlarge-us-east-2
[Failed] d8d0323f test-mac-arm
[Failed] d8d0323f ursa-i9-9960x
[Finished] d8d0323f ursa-thinkcentre-m75q
[Finished] 0aacc28c ec2-t3-xlarge-us-east-2
[Failed] 0aacc28c test-mac-arm
[Failed] 0aacc28c ursa-i9-9960x
[Finished] 0aacc28c ursa-thinkcentre-m75q
Supported benchmarks:
ec2-t3-xlarge-us-east-2: Supported benchmark langs: Python, R. Runs only benchmarks with cloud = True
test-mac-arm: Supported benchmark langs: C++, Python, R
ursa-i9-9960x: Supported benchmark langs: Python, R, JavaScript
ursa-thinkcentre-m75q: Supported benchmark langs: C++, Java

fatemehp pushed a commit to fatemehp/arrow that referenced this pull request Oct 17, 2022
Will fix [ARROW-17697](https://issues.apache.org/jira/browse/ARROW-17697)

- Turn warnings into errors 
- Change signature of DataType.field as others like `StructType.field` allowed taking `int` or `str`.

Authored-by: Miles Granger <miles59923@gmail.com>
Signed-off-by: Antoine Pitrou <antoine@python.org>
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.

None yet

4 participants