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

feat: move partition consolidator and add LocalAggregator API #1071

Merged
merged 2 commits into from
Jun 4, 2021

Conversation

imatiach-msft
Copy link
Contributor

@imatiach-msft imatiach-msft commented Jun 4, 2021

  • move PartitionConsolidator API to stages directory
  • add LocalAggregator API (to be used by LightGBM)

@imatiach-msft
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@imatiach-msft imatiach-msft force-pushed the ilmat/move-part-cons branch from cfb06d5 to 46a2e04 Compare June 4, 2021 15:16
@imatiach-msft imatiach-msft changed the title move partition consolidator and add LocalAggregator API feat: move partition consolidator and add LocalAggregator API Jun 4, 2021
@imatiach-msft
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

mhamilton723
mhamilton723 previously approved these changes Jun 4, 2021
@codecov
Copy link

codecov bot commented Jun 4, 2021

Codecov Report

Merging #1071 (96a2de8) into master (2a716c1) will decrease coverage by 0.09%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1071      +/-   ##
==========================================
- Coverage   84.99%   84.90%   -0.10%     
==========================================
  Files         206      206              
  Lines        9751     9751              
  Branches      549      550       +1     
==========================================
- Hits         8288     8279       -9     
- Misses       1463     1472       +9     
Impacted Files Coverage Δ
...rosoft/ml/spark/stages/PartitionConsolidator.scala 95.74% <ø> (ø)
...ala/org/apache/spark/ml/param/DataFrameParam.scala 69.56% <0.00%> (-17.40%) ⬇️
...a/com/microsoft/ml/spark/io/http/HTTPClients.scala 73.33% <0.00%> (-10.00%) ⬇️

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 2a716c1...96a2de8. Read the comment docs.

@imatiach-msft
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@imatiach-msft imatiach-msft merged commit 36ee274 into microsoft:master Jun 4, 2021
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.

2 participants