From c65019e2812971c2d44f0c895be60cadc8cefc33 Mon Sep 17 00:00:00 2001
From: Szymon Matejczyk <smatejczyk@gmail.com>
Date: Fri, 4 Aug 2017 12:05:56 +0200
Subject: [PATCH] Add doc for TestReorderJoins

---
 .../sql/planner/optimizations/TestReorderJoins.java       | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/presto-main/src/test/java/com/facebook/presto/sql/planner/optimizations/TestReorderJoins.java b/presto-main/src/test/java/com/facebook/presto/sql/planner/optimizations/TestReorderJoins.java
index 6da410d5f83a9..2702efb23d57c 100644
--- a/presto-main/src/test/java/com/facebook/presto/sql/planner/optimizations/TestReorderJoins.java
+++ b/presto-main/src/test/java/com/facebook/presto/sql/planner/optimizations/TestReorderJoins.java
@@ -37,6 +37,14 @@
 import static java.util.Objects.requireNonNull;
 import static org.testng.Assert.assertEquals;
 
+/**
+ * This tests verify joins order in plans generated for TPC-H queries using
+ * cost-based optimizer. They are intended to signal that a code change results
+ * in a change of generated plans.
+ *
+ * Most of the plans are currently suboptimal, so the tests results will likely
+ * change. They should be changed with care.
+ */
 public class TestReorderJoins
         extends BasePlanTest
 {