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-2971: [Python] Give some modules in arrow/python more descriptive names #2446

Closed
wants to merge 4 commits into from

Conversation

wesm
Copy link
Member

@wesm wesm commented Aug 18, 2018

Follow-up work to ARROW-2814. Renaming files only

Change-Id: I072d0761c697c8d862df4d5ab561da95ef5db9cb
Change-Id: I350d2535fbae5ff9e5a44184b8c8e621df71e9a1
Change-Id: Ie58ef9953a319f12fe0fc45a1c50834dcdf32375
Change-Id: I4411761f9d1f885a728e51f55ee036e7a5150eb4
@codecov-io
Copy link

Codecov Report

Merging #2446 into master will increase coverage by 1.28%.
The diff coverage is 89.11%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2446      +/-   ##
==========================================
+ Coverage   85.49%   86.78%   +1.28%     
==========================================
  Files         301      241      -60     
  Lines       46273    42748    -3525     
==========================================
- Hits        39563    37097    -2466     
+ Misses       6636     5651     -985     
+ Partials       74        0      -74
Impacted Files Coverage Δ
cpp/src/arrow/python/arrow_to_pandas.cc 88.72% <ø> (ø) ⬆️
cpp/src/arrow/python/python-test.cc 100% <ø> (ø) ⬆️
cpp/src/arrow/python/numpy_to_arrow.cc 94.11% <ø> (ø) ⬆️
cpp/src/arrow/python/deserialize.cc 91.62% <ø> (ø)
cpp/src/arrow/python/decimal.h 100% <ø> (ø) ⬆️
cpp/src/arrow/python/serialize.h 0% <0%> (ø)
cpp/src/arrow/python/python_to_arrow.h 100% <100%> (+100%) ⬆️
cpp/src/arrow/python/serialize.cc 88.88% <88.88%> (ø)
cpp/src/arrow/python/python_to_arrow.cc 89.52% <89.51%> (+0.63%) ⬆️
cpp/src/arrow/compute/kernels/cast.cc 90.82% <0%> (-0.72%) ⬇️
... and 69 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 16bbec8...d13016b. Read the comment docs.

@wesm
Copy link
Member Author

wesm commented Aug 18, 2018

+1

@wesm wesm closed this in 349e6b0 Aug 18, 2018
@wesm wesm deleted the ARROW-2971 branch August 18, 2018 17:45
stephanie-wang pushed a commit to stephanie-wang/arrow that referenced this pull request Aug 29, 2018
…ve names

Follow-up work to ARROW-2814. Renaming files only

Author: Wes McKinney <wesm+git@apache.org>

Closes apache#2446 from wesm/ARROW-2971 and squashes the following commits:

d13016b <Wes McKinney> Add additional header
5a08109 <Wes McKinney> Use only single libarrow_python header that does not include Python.h
a076676 <Wes McKinney> Use public API headers in plasma_op.cc
70ae1ce <Wes McKinney> Give modules in arrow/python more descriptive names
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.

None yet

2 participants