search

databricks / koalas

Koalas: pandas API on Apache Spark
Apache License 2.0
3.32k stars 356 forks source link

Make SeriesType, DataFrameType, and ScalarType hold the dtypes as well. #2123

Closed ueshin closed 3 years ago

ueshin commented 3 years ago

After the return type inference, SeriesType, DataFrameType, and ScalarType hold the dtypes as well as Spark DataTypes.

codecov-io commented 3 years ago

Codecov Report

Merging #2123 (3ad837f) into master (ed0bd49) will increase coverage by 0.00%. The diff coverage is 89.74%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2123   +/-   ##
=======================================
  Coverage   95.37%   95.37%           
=======================================
  Files          60       60           
  Lines       13587    13591    +4     
=======================================
+ Hits        12958    12962    +4     
  Misses        629      629
Impacted Files Coverage Δ
databricks/koalas/typedef/typehints.py 95.40% <86.36%> (+0.58%) :arrow_up:
databricks/koalas/groupby.py 94.31% <87.50%> (-0.12%) :arrow_down:
databricks/koalas/accessors.py 93.50% <100.00%> (+0.03%) :arrow_up:
databricks/koalas/frame.py 96.52% <100.00%> (+<0.01%) :arrow_up:
databricks/koalas/series.py 96.92% <100.00%> (ø)
databricks/koalas/utils.py 95.66% <0.00%> (-0.09%) :arrow_down:

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 ed0bd49...3ad837f. Read the comment docs.

xinrong-meng commented 3 years ago

Looks great! Thank you!

ueshin commented 3 years ago

Thanks! merging.