Skip to content

Conversation

@HyukjinKwon
Copy link
Member

Currently, Koalas' statics functions such as count, sum, min, max, etc. returns a pandas Series.

Resolves #1332

@unittest.skip(
"PySpark does not allow object type. "
"Should we support these case by casting to string? or just disallow?"
)
Copy link
Member Author

Choose a reason for hiding this comment

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

I will make a followup after this PR.

@codecov-io
Copy link

codecov-io commented Mar 10, 2020

Codecov Report

Merging #1333 into master will increase coverage by 0.05%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1333      +/-   ##
==========================================
+ Coverage    95.2%   95.26%   +0.05%     
==========================================
  Files          34       34              
  Lines        7445     7448       +3     
==========================================
+ Hits         7088     7095       +7     
+ Misses        357      353       -4
Impacted Files Coverage Δ
databricks/koalas/generic.py 96.99% <ø> (ø) ⬆️
databricks/koalas/frame.py 96.71% <100%> (ø) ⬆️
databricks/koalas/__init__.py 93.61% <0%> (+8.51%) ⬆️

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 03b3ecb...3615ad0. Read the comment docs.

@HyukjinKwon HyukjinKwon requested a review from ueshin March 11, 2020 00:31
@HyukjinKwon HyukjinKwon force-pushed the return_series_stats branch from 9d9f577 to ce8e26b Compare March 11, 2020 07:27
@HyukjinKwon HyukjinKwon force-pushed the return_series_stats branch from ce8e26b to 0bcf606 Compare March 11, 2020 07:49
@HyukjinKwon
Copy link
Member Author

I discussed offline with @ueshin. I am going to merge this.

@HyukjinKwon HyukjinKwon merged commit f0af49c into databricks:master Mar 11, 2020
@HyukjinKwon HyukjinKwon deleted the return_series_stats branch September 11, 2020 07:52
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.

Return type consistency in docs

3 participants