[FIX] Empty lines or tabs inside SQL query of view definition will trigger redeployment of databricks_sql_table resource #3865

catalog/resource_sql_table.go

@@ -5,6 +5,7 @@ import (
 	"fmt"
 	"log"
 	"reflect"
+	"regexp"
 	"slices"
 	"strings"
 	"time"
@@ -369,6 +370,7 @@ func (ti *SqlTableInfo) diff(oldti *SqlTableInfo) ([]string, error) {
 
 	if ti.TableType == "VIEW" {
 		// View only attributes
+		formatViewDefinition(ti)
 		if ti.ViewDefinition != oldti.ViewDefinition {
 			statements = append(statements, fmt.Sprintf("ALTER VIEW %s AS %s", ti.SQLFullName(), ti.ViewDefinition))
 		}
@@ -408,6 +410,19 @@ func (ti *SqlTableInfo) diff(oldti *SqlTableInfo) ([]string, error) {
 	return statements, nil
 }
 
+// formatViewDefinition removes empty lines and changes tabs to 4 spaces
+// in order to compare view definitions correctly
+func formatViewDefinition(ti *SqlTableInfo) {
+
+	// remove empty lines
+	// 1\n\n\n2 => 1\n2
+	ti.ViewDefinition = regexp.MustCompile(`[\r\n]+`).ReplaceAllString(ti.ViewDefinition, "\n")
+
+	// change tab to 4 spaces
+	// 1\t2 => 1    2
+	ti.ViewDefinition = strings.ReplaceAll(ti.ViewDefinition, "\t", "    ")
+}
+
 func (ti *SqlTableInfo) updateTable(oldti *SqlTableInfo) error {
 	statements, err := ti.diff(oldti)
 	if err != nil {

catalog/resource_sql_table_test.go

@@ -1710,3 +1710,17 @@ func TestSqlTablesAPI_getTable_OptionsAndParametersProcessedCorrectly(t *testing
 		})
 	}
 }
+
+func TestFormatViewDefinition(t *testing.T) {
+	ti := &SqlTableInfo{
+		ViewDefinition: "SELECT\n\n\n\t*\nFROM\n\tmy_table\nWHERE\n\tcolumn = 'value'",
+	}
+
+	expected := "SELECT\n    *\nFROM\n    my_table\nWHERE\n    column = 'value'"
+
+	formatViewDefinition(ti)
+
+	if ti.ViewDefinition != expected {
+		t.Errorf("Expected view definition: %s, but got: %s", expected, ti.ViewDefinition)
+	}
+}