From 646d51ee976107487b2fccc2ea321b3a5b56523c Mon Sep 17 00:00:00 2001
From: Piotr Findeisen <piotr.findeisen@gmail.com>
Date: Sun, 4 Oct 2020 15:02:52 +0200
Subject: [PATCH] Mark testWrittenStats test as @Flaky

---
 presto-testing/pom.xml                                | 11 +++++------
 .../testing/AbstractTestDistributedQueries.java       |  2 ++
 2 files changed, 7 insertions(+), 6 deletions(-)

diff --git a/presto-testing/pom.xml b/presto-testing/pom.xml
index 528c8d878018..270203896341 100644
--- a/presto-testing/pom.xml
+++ b/presto-testing/pom.xml
@@ -47,6 +47,11 @@
             <artifactId>presto-spi</artifactId>
         </dependency>
 
+        <dependency>
+            <groupId>io.prestosql</groupId>
+            <artifactId>presto-testng-services</artifactId>
+        </dependency>
+
         <dependency>
             <groupId>io.prestosql</groupId>
             <artifactId>presto-tpch</artifactId>
@@ -143,12 +148,6 @@
         </dependency>
 
         <!-- used by tests but also needed transitively -->
-        <dependency>
-            <groupId>io.prestosql</groupId>
-            <artifactId>presto-testng-services</artifactId>
-            <scope>runtime</scope>
-        </dependency>
-
         <dependency>
             <groupId>io.airlift</groupId>
             <artifactId>log-manager</artifactId>
diff --git a/presto-testing/src/main/java/io/prestosql/testing/AbstractTestDistributedQueries.java b/presto-testing/src/main/java/io/prestosql/testing/AbstractTestDistributedQueries.java
index fb150a2a4651..8cd435a6e491 100644
--- a/presto-testing/src/main/java/io/prestosql/testing/AbstractTestDistributedQueries.java
+++ b/presto-testing/src/main/java/io/prestosql/testing/AbstractTestDistributedQueries.java
@@ -26,6 +26,7 @@
 import io.prestosql.server.BasicQueryInfo;
 import io.prestosql.spi.security.Identity;
 import io.prestosql.testing.sql.TestTable;
+import io.prestosql.testng.services.Flaky;
 import org.intellij.lang.annotations.Language;
 import org.testng.annotations.DataProvider;
 import org.testng.annotations.Test;
@@ -1254,6 +1255,7 @@ public void testViewFunctionAccessControl()
     }
 
     @Test
+    @Flaky(issue = "https://github.com/prestosql/presto/issues/5172", match = "AssertionError: expected \\[.*\\] but found \\[.*\\]")
     public void testWrittenStats()
     {
         String tableName = "test_written_stats_" + randomTableSuffix();