enable build step timeout also some code cleanup (dotnet#5122)

* enable build step timeout also some code cleanup * clean up unnecessary file * sync to latest for submodule
xxpniu · May 14, 2020 · a2406f6 · a2406f6
1 parent 8e5f7b4
commit a2406f6
Show file tree

Hide file tree

Showing 8 changed files with 11 additions and 312 deletions.
diff --git a/build/ci/job-template.yml b/build/ci/job-template.yml
@@ -57,6 +57,7 @@ jobs:
         displayName: Set LD_LIBRARY_PATH for Ubuntu and CentOS to locate Native shared library in current running path
     - script: ${{ parameters.buildScript }} -$(_configuration) -buildArch=${{ parameters.architecture }}
       displayName: Build
+      timeoutInMinutes: 20
     - ${{ if eq(parameters.pool.name, 'Hosted macOS') }}:
       - task: Bash@3
         inputs:

diff --git a/test/Microsoft.ML.Functional.Tests/Evaluation.cs b/test/Microsoft.ML.Functional.Tests/Evaluation.cs
@@ -221,7 +221,7 @@ public void TrainAndEvaluateRankingWithOptions()
         /// <summary>
         /// Train and Evaluate: Recommendation.
         /// </summary>
-        [MatrixFactorizationFact]
+        [Fact]
         public void TrainAndEvaluateRecommendation()
         {
             var mlContext = new MLContext(seed: 1);

diff --git a/test/Microsoft.ML.TestFrameworkCommon/Attributes/MatrixFactorizationFactAttribute.cs b/test/Microsoft.ML.TestFrameworkCommon/Attributes/MatrixFactorizationFactAttribute.cs
diff --git a/test/Microsoft.ML.TestFrameworkCommon/Attributes/RetryFactAttribute.cs b/test/Microsoft.ML.TestFrameworkCommon/Attributes/RetryFactAttribute.cs
diff --git a/test/Microsoft.ML.TestFrameworkCommon/DelayedMessageBus.cs b/test/Microsoft.ML.TestFrameworkCommon/DelayedMessageBus.cs
diff --git a/test/Microsoft.ML.TestFrameworkCommon/RetryFactDiscoverer.cs b/test/Microsoft.ML.TestFrameworkCommon/RetryFactDiscoverer.cs
diff --git a/test/Microsoft.ML.TestFrameworkCommon/RetryTestCase.cs b/test/Microsoft.ML.TestFrameworkCommon/RetryTestCase.cs