fix #5767 issue with DataFrame Merge method

[asmirnov82]

Fixes #5767

Fix issue related to DataFrame Merge method with Algorithm.Inner, in case when right dataframe has more rows. that left

[dnfadmin]

All CLA requirements met.

[codecov]

Codecov Report

Merging #5768 (c527c21) into main (ebc431f) will decrease coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #5768      +/-   ##
==========================================
- Coverage   68.38%   68.33%   -0.06%     
==========================================
  Files        1131     1131              
  Lines      241019   241027       +8     
  Branches    25024    25026       +2     
==========================================
- Hits       164822   164699     -123     
- Misses      69714    69831     +117     
- Partials     6483     6497      +14     

Flag	Coverage Δ
Debug	68.33% <100.00%> (-0.06%)	⬇️
production	62.96% <100.00%> (-0.07%)	⬇️
test	89.20% <100.00%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
src/Microsoft.Data.Analysis/DataFrame.Join.cs	97.88% <100.00%> (-0.03%)	⬇️
...st/Microsoft.Data.Analysis.Tests/DataFrameTests.cs	99.94% <100.00%> (+<0.01%)	⬆️
...osoft.ML.KMeansClustering/KMeansPlusPlusTrainer.cs	83.60% <0.00%> (-7.16%)	⬇️
src/Microsoft.ML.FastTree/Training/StepSearch.cs	57.42% <0.00%> (-4.96%)	⬇️
src/Microsoft.ML.Data/Training/TrainerUtils.cs	65.86% <0.00%> (-3.82%)	⬇️
...crosoft.ML.StandardTrainers/Standard/SdcaBinary.cs	85.49% <0.00%> (-3.08%)	⬇️
src/Microsoft.ML.Sweeper/AsyncSweeper.cs	71.42% <0.00%> (-1.37%)	⬇️
...crosoft.ML.StandardTrainers/Optimizer/Optimizer.cs	71.96% <0.00%> (-1.16%)	⬇️
...oft.ML.StandardTrainers/Standard/SdcaMulticlass.cs	91.46% <0.00%> (-1.03%)	⬇️
src/Microsoft.ML.Data/Utils/LossFunctions.cs	66.83% <0.00%> (-0.52%)	⬇️
... and 4 more

[pgovind]

Yup, this LGTM!

[pgovind]

Thank you for the PR @asmirnov82 ! Really appreciate the work here!

[asmirnov82]

You are welcome @pgovind. Do I need to click "Close PR" to merge it or you will merge it by yourself?

[pgovind]

Do I need to click "Close PR" to merge it or you will merge it by yourself?

I'm not sure how the permissions are set up in this repo. In this particular case, I'll do it because we're working on fixing the CI in this repo

[asmirnov82]

Thanks!

[pgovind]

@asmirnov82, I believe CI is fixed in the repo now. Can you rebase your PR on the latest dotnet/main?

Edit: Never mind, I was able to do it within GitHub!

[pgovind]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 2 pipeline(s).