From 92bd4b6459e88ae7eb809d8e61a7b925c8e72f74 Mon Sep 17 00:00:00 2001
From: Andy Grove <andygrove@nvidia.com>
Date: Tue, 10 Aug 2021 16:59:13 -0600
Subject: [PATCH] fix rowSpan issue

---
 docs/supported_ops.md                                | 12 ------------
 .../scala/com/nvidia/spark/rapids/TypeChecks.scala   |  8 +++-----
 2 files changed, 3 insertions(+), 17 deletions(-)

diff --git a/docs/supported_ops.md b/docs/supported_ops.md
index 3d7040018cd..46e36fb15b8 100644
--- a/docs/supported_ops.md
+++ b/docs/supported_ops.md
@@ -684,8 +684,6 @@ Accelerator supports are described below.
 <td><b>NS</b></td>
 </tr>
 <tr>
-<td></td>
-<td></td>
 <td>leftKeys</td>
 <td>None</td>
 <td>S</td>
@@ -708,8 +706,6 @@ Accelerator supports are described below.
 <td><b>NS</b></td>
 </tr>
 <tr>
-<td></td>
-<td></td>
 <td>rightKeys</td>
 <td>None</td>
 <td>S</td>
@@ -804,8 +800,6 @@ Accelerator supports are described below.
 <td><b>NS</b></td>
 </tr>
 <tr>
-<td></td>
-<td></td>
 <td>leftKeys</td>
 <td>None</td>
 <td>S</td>
@@ -828,8 +822,6 @@ Accelerator supports are described below.
 <td><b>NS</b></td>
 </tr>
 <tr>
-<td></td>
-<td></td>
 <td>rightKeys</td>
 <td>None</td>
 <td>S</td>
@@ -876,8 +868,6 @@ Accelerator supports are described below.
 <td><b>NS</b></td>
 </tr>
 <tr>
-<td></td>
-<td></td>
 <td>leftKeys</td>
 <td>None</td>
 <td>S</td>
@@ -900,8 +890,6 @@ Accelerator supports are described below.
 <td><b>NS</b></td>
 </tr>
 <tr>
-<td></td>
-<td></td>
 <td>rightKeys</td>
 <td>None</td>
 <td>S</td>
diff --git a/sql-plugin/src/main/scala/com/nvidia/spark/rapids/TypeChecks.scala b/sql-plugin/src/main/scala/com/nvidia/spark/rapids/TypeChecks.scala
index 4fc231a217a..a0094353844 100644
--- a/sql-plugin/src/main/scala/com/nvidia/spark/rapids/TypeChecks.scala
+++ b/sql-plugin/src/main/scala/com/nvidia/spark/rapids/TypeChecks.scala
@@ -1559,14 +1559,14 @@ object SupportedOpsDocs {
           nextOutputAt = totalCount + headerEveryNLines
         }
         println("<tr>")
-        val count = if (checks.isDefined) {
+        val rowSpan = if (checks.isDefined) {
           val exprChecks = checks.get.asInstanceOf[ExecChecks]
           1 + exprChecks.extendedChecks.size
         } else {
           1
         }
-        println(s"""<td rowspan="$count">${rule.tag.runtimeClass.getSimpleName}</td>""")
-        println(s"""<td rowspan="$count">${rule.description}</td>""")
+        println(s"""<td rowspan="$rowSpan">${rule.tag.runtimeClass.getSimpleName}</td>""")
+        println(s"""<td rowspan="$rowSpan">${rule.description}</td>""")
         println(s"<td>Input</td>")
         println(s"<td>${rule.notes().getOrElse("None")}</td>")
         if (checks.isDefined) {
@@ -1587,8 +1587,6 @@ object SupportedOpsDocs {
           val exprChecks = checks.get.asInstanceOf[ExecChecks]
           exprChecks.extendedChecks.keys.toSeq.sorted.foreach { ctx =>
             println("<tr>")
-            println(s"<td></td>")
-            println(s"<td></td>")
             println(s"<td>$ctx</td>")
             println(s"<td>${rule.notes().getOrElse("None")}</td>")
             TypeEnum.values.foreach { t =>