[Issue4484] Support specifying command timeout while using the database loader

[wangyems]

fix issue 4484

-add option to set db command timeout in db loader
-add corresponding test
this pr partially refers to a hanging PR.

[codecov]

Codecov Report

Merging #5288 into master will decrease coverage by 4.16%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #5288      +/-   ##
==========================================
- Coverage   73.68%   69.51%   -4.17%     
==========================================
  Files        1022      779     -243     
  Lines      190320   145811   -44509     
  Branches    20470    18576    -1894     
==========================================
- Hits       140232   101366   -38866     
+ Misses      44557    39273    -5284     
+ Partials     5531     5172     -359     

Flag	Coverage Δ
#Debug	69.51% <100.00%> (-4.17%)	⬇️
#production	69.51% <100.00%> (+0.09%)	⬆️
#test	?

Impacted Files	Coverage Δ
...Data/DataLoadSave/Database/DatabaseLoaderCursor.cs	23.43% <100.00%> (+0.48%)	⬆️
...ft.ML.Data/DataLoadSave/Database/DatabaseSource.cs	100.00% <100.00%> (ø)
....ML.AutoML/PipelineSuggesters/PipelineSuggester.cs	79.83% <0.00%> (-6.73%)	⬇️
src/Microsoft.ML.Maml/MAML.cs	23.78% <0.00%> (-0.98%)	⬇️
src/Microsoft.ML.AutoML/Sweepers/Parameters.cs	84.32% <0.00%> (-0.85%)	⬇️
test/Microsoft.ML.TestFramework/PasteArguments.cs
test/Microsoft.ML.AutoML.Tests/AutoFitTests.cs
test/Microsoft.ML.Benchmarks/Harness/Configs.cs
.../Microsoft.ML.Tests/Transformers/NAReplaceTests.cs
....ML.Tests/Transformers/DateTimeTransformerTests.cs
... and 247 more
#Resolved

[codecov]

Codecov Report

Merging #5288 into master will increase coverage by 0.07%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##           master    #5288      +/-   ##
==========================================
+ Coverage   73.68%   73.75%   +0.07%     
==========================================
  Files        1022     1022              
  Lines      190320   190360      +40     
  Branches    20470    20472       +2     
==========================================
+ Hits       140232   140404     +172     
+ Misses      44557    44439     -118     
+ Partials     5531     5517      -14     

Flag	Coverage Δ
#Debug	73.75% <75.00%> (+0.07%)	⬆️
#production	69.52% <62.50%> (+0.09%)	⬆️
#test	87.62% <81.25%> (-0.01%)	⬇️

Impacted Files	Coverage Δ
...ft.ML.Data/DataLoadSave/Database/DatabaseSource.cs	83.33% <57.14%> (-16.67%)	⬇️
test/Microsoft.ML.Tests/DatabaseLoaderTests.cs	93.33% <81.25%> (-0.75%)	⬇️
...Data/DataLoadSave/Database/DatabaseLoaderCursor.cs	23.07% <100.00%> (+0.12%)	⬆️
....ML.AutoML/PipelineSuggesters/PipelineSuggester.cs	83.19% <0.00%> (-3.37%)	⬇️
src/Microsoft.ML.Maml/MAML.cs	23.78% <0.00%> (-0.98%)	⬇️
src/Microsoft.ML.AutoML/Sweepers/Parameters.cs	84.32% <0.00%> (-0.85%)	⬇️
src/Microsoft.ML.FastTree/FastTreeRanking.cs	55.06% <0.00%> (-0.77%)	⬇️
src/Microsoft.ML.FastTree/FastTreeArguments.cs	85.49% <0.00%> (+0.11%)	⬆️
...StandardTrainers/Standard/LinearModelParameters.cs	66.58% <0.00%> (+0.25%)	⬆️
src/Microsoft.ML.Data/Prediction/Calibrator.cs	80.72% <0.00%> (+0.26%)	⬆️
... and 11 more

[harishsk]

        Contracts.CheckValue(providerFactory, nameof(providerFactory));

What is the default timeout if this constructor is used? #Resolved

---

Refers to: src/Microsoft.ML.Data/DataLoadSave/Database/DatabaseSource.cs:19 in 2406db1. [](commit_id = 2406db1, deletion_comment = False)

[wangyems]

        Contracts.CheckValue(providerFactory, nameof(providerFactory));

What is the default timeout if this constructor is used?

Refers to: src/Microsoft.ML.Data/DataLoadSave/Database/DatabaseSource.cs:19 in 2406db1. [](commit_id = 2406db1, deletion_comment = False)

The default timeout is 30s #Resolved

[frank-dong-ms-zz]

[harishsk]