From 119881908004802d85fc38a315a04ba23c25b33a Mon Sep 17 00:00:00 2001
From: Tanner Clary <tannerclary@google.com>
Date: Mon, 11 Mar 2024 11:07:28 -0700
Subject: [PATCH] [b/328440720] Cast STRING to FLOAT64 before casting to
 INTEGER

---
 connector/tableau/looker-jdbc/dialect.tdd | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/connector/tableau/looker-jdbc/dialect.tdd b/connector/tableau/looker-jdbc/dialect.tdd
index 3f7bf2174544..fd0ddcdda9f7 100644
--- a/connector/tableau/looker-jdbc/dialect.tdd
+++ b/connector/tableau/looker-jdbc/dialect.tdd
@@ -553,7 +553,11 @@
       <argument type='int' />
     </function>
     <function group='cast' name='INT' return-type='int'>
-      <formula>CAST((%1) AS INTEGER)</formula>
+      <!-- Casting directly to INTEGER errors if there's a decimal, so cast to FLOAT64 first.
+           Casting FLOAT64 to INTEGER returns the nearest integer so something like "19.8"
+           will becoming 20. The alternative is to add extra logic to ensure the whole part
+           remains the same, but this is more complicated (a.k.a. more error prone).  -->
+      <formula>CAST(CAST((%1) AS FLOAT64) AS INTEGER)</formula>
       <argument type='str' />
     </function>
     <function group='cast' name='INT' return-type='int'>