[TD-12223]<fix>fix bug long time convert timestamp when negative value (#9267)

* fix bug long time convert timestamp when negative value

* add test description

src/connector/jdbc/src/main/java/com/taosdata/jdbc/TSDBResultSetRowData.java

@@ -443,16 +443,29 @@ public class TSDBResultSetRowData {
             case 0: {
                 milliseconds = ts;
                 fracNanoseconds = (int) (ts * 1_000_000 % 1_000_000_000);
+                fracNanoseconds = fracNanoseconds < 0 ? 1_000_000_000 + fracNanoseconds : fracNanoseconds;
                 break;
             }
             case 1: {
                 milliseconds = ts / 1_000;
                 fracNanoseconds = (int) (ts * 1_000 % 1_000_000_000);
+                if (fracNanoseconds < 0) {
+                    if (milliseconds == 0 ){
+                        milliseconds = -1;
+                    }
+                    fracNanoseconds += 1_000_000_000;
+                }
                 break;
             }
             case 2: {
                 milliseconds = ts / 1_000_000;
                 fracNanoseconds = (int) (ts % 1_000_000_000);
+                if (fracNanoseconds < 0) {
+                    if (milliseconds == 0 ){
+                        milliseconds = -1;
+                    }
+                    fracNanoseconds += 1_000_000_000;
+                }
                 break;
             }
             default: {

src/connector/jdbc/src/test/java/com/taosdata/jdbc/cases/DatetimeBefore1970Test.java

 package com.taosdata.jdbc.cases;
 
+import com.taosdata.jdbc.annotation.CatalogRunner;
+import com.taosdata.jdbc.annotation.Description;
+import com.taosdata.jdbc.annotation.TestTarget;
 import com.taosdata.jdbc.utils.TimestampUtil;
 import org.junit.*;
+import org.junit.runner.RunWith;
 
 import java.sql.*;
 
+@RunWith(CatalogRunner.class)
+@TestTarget(alias = "negative value convert to timestamp", author = "huolibo", version = "2.0.37")
 public class DatetimeBefore1970Test {
 
     private static final String host = "127.0.0.1";
     private Connection conn;
 
     @Test
-    public void test() throws SQLException {
+    @Description("millisecond")
+    public void msTest() throws SQLException {
+        conn = createEnvironment("ms");
+        long now = System.currentTimeMillis();
         try (Statement stmt = conn.createStatement()) {
             // given
+            // before
+            stmt.executeUpdate("insert into weather(ts) values('1969-12-31 00:00:00.001')");
             stmt.executeUpdate("insert into weather(ts) values('1969-12-31 23:59:59.999')");
+            // zero
             stmt.executeUpdate("insert into weather(ts) values('1970-01-01 00:00:00.000')");
-            stmt.executeUpdate("insert into weather(ts) values('1970-01-01 08:00:00.000')");
-            stmt.executeUpdate("insert into weather(ts) values('1970-01-01 07:59:59.999')");
+            //after
+            stmt.executeUpdate("insert into weather(ts) values('1970-01-01 00:00:00.001')");
+            stmt.executeUpdate("insert into weather(ts) values(" + now + ")");
             ResultSet rs = stmt.executeQuery("select * from weather order by ts asc");
             ResultSetMetaData metaData = rs.getMetaData();
             Assert.assertEquals(2, metaData.getColumnCount());
@@ -26,44 +39,221 @@ public class DatetimeBefore1970Test {
             rs.next();
             // then
             Timestamp ts = rs.getTimestamp("ts");
-            Assert.assertEquals("1969-12-31 23:59:59.999", TimestampUtil.longToDatetime(ts.getTime()));
+            Assert.assertEquals(-24 * 60 * 60 * 1000 + 1, ts.getTime());
 
             // when
             rs.next();
             // then
             ts = rs.getTimestamp("ts");
-            Assert.assertEquals("1970-01-01 00:00:00.000", TimestampUtil.longToDatetime(ts.getTime()));
+            Assert.assertEquals(-1, ts.getTime());
 
             // when
             rs.next();
             // then
             ts = rs.getTimestamp("ts");
-            Assert.assertEquals("1970-01-01 08:00:00.000", TimestampUtil.longToDatetime(ts.getTime()));
+            Assert.assertEquals(0, ts.getTime());
+
+            // when
+            rs.next();
+            // then
+            ts = rs.getTimestamp("ts");
+            Assert.assertEquals(1, ts.getTime());
+
+            // when
+            rs.next();
+            // then
+            ts = rs.getTimestamp("ts");
+            Assert.assertEquals(now, ts.getTime());
+        }
+    }
+
+    @Test
+    @Description("microsecond")
+    public void usTest() throws SQLException {
+        conn = createEnvironment("us");
+        long now = System.currentTimeMillis();
+        try (Statement stmt = conn.createStatement()) {
+            // given
+            stmt.executeUpdate("insert into weather(ts) values('1969-12-31 00:00:00.000001')");
+            stmt.executeUpdate("insert into weather(ts) values('1969-12-31 23:59:59.999999')");
+            stmt.executeUpdate("insert into weather(ts) values('1970-01-01 00:00:00.000000')");
+            stmt.executeUpdate("insert into weather(ts) values('1970-01-01 00:00:00.000001')");
+            stmt.executeUpdate("insert into weather(ts) values(" + now + ")");
+            ResultSet rs = stmt.executeQuery("select * from weather order by ts asc");
+            ResultSetMetaData metaData = rs.getMetaData();
+            Assert.assertEquals(2, metaData.getColumnCount());
+
+            // when
+            rs.next();
+            // then
+            Timestamp ts = rs.getTimestamp("ts");
+            Assert.assertEquals(-24 * 60 * 60 * 1000, ts.getTime());
+            Assert.assertEquals(1_000, ts.getNanos());
+
+            // when
+            rs.next();
+            // then
+            ts = rs.getTimestamp("ts");
+            Assert.assertEquals(-1, ts.getTime());
+            Assert.assertEquals(999_999_000, ts.getNanos());
+
+            // when
+            rs.next();
+            // then
+            ts = rs.getTimestamp("ts");
+            Assert.assertEquals(0, ts.getTime());
+
+            // when
+            rs.next();
+            // then
+            ts = rs.getTimestamp("ts");
+            Assert.assertEquals(0, ts.getTime());
+            Assert.assertEquals(1_000, ts.getNanos());
+
+            // when
+            rs.next();
+            // then
+            ts = rs.getTimestamp("ts");
+            String s = String.valueOf(now);
+            Assert.assertEquals(Long.parseLong(s.substring(0, s.length() - 3)), ts.getTime());
+            Assert.assertEquals(Long.parseLong(s.substring(s.length() - 6) + "000"), ts.getNanos());
+        }
+    }
+
+    @Test
+    @Description("nanosecond")
+    public void nanoTest() throws SQLException {
+        conn = createEnvironment("ns");
+        long now = System.currentTimeMillis() * 1000_000L + System.nanoTime() % 1000_000L;
+        try (Statement stmt = conn.createStatement()) {
+            // given
+            stmt.executeUpdate("insert into weather(ts) values('1969-12-31 00:00:00.000000123')");
+            stmt.executeUpdate("insert into weather(ts) values('1969-12-31 23:59:59.999999999')");
+            stmt.executeUpdate("insert into weather(ts) values('1970-01-01 00:00:00.000')");
+            stmt.executeUpdate("insert into weather(ts) values('1970-01-01 00:00:00.000000001')");
+            stmt.executeUpdate("insert into weather(ts) values(" + now + ")");
+            ResultSet rs = stmt.executeQuery("select * from weather order by ts asc");
+            ResultSetMetaData metaData = rs.getMetaData();
+            Assert.assertEquals(2, metaData.getColumnCount());
+
+            // when
+            rs.next();
+            // then
+            Timestamp ts = rs.getTimestamp("ts");
+            Assert.assertEquals(-24 * 60 * 60 * 1_000, ts.getTime());
+            Assert.assertEquals(123, ts.getNanos());
+
+            // when
+            rs.next();
+            // then
+            ts = rs.getTimestamp("ts");
+            Assert.assertEquals(-1, ts.getTime());
+            Assert.assertEquals(999999999, ts.getNanos());
+
+            // when
+            rs.next();
+            // then
+            ts = rs.getTimestamp("ts");
+            Assert.assertEquals(0, ts.getNanos());
+
+            // when
+            rs.next();
+            // then
+            ts = rs.getTimestamp("ts");
+            Assert.assertEquals(1, ts.getNanos());
+
+            // when
+            rs.next();
+            // then
+            ts = rs.getTimestamp("ts");
+            String s = String.valueOf(now);
+            Assert.assertEquals(Long.parseLong(s.substring(0, s.length() - 6)), ts.getTime());
+            Assert.assertEquals(Long.parseLong(s.substring(s.length() - 9)), ts.getNanos());
+        }
+    }
+
+    @Test
+    @Ignore
+    @Description("nanosecond convert timestamp when timezone is asia shanghai")
+    public void asiaShanghaiTest() throws SQLException {
+        conn = createEnvironment("ns");
+        long now = System.currentTimeMillis() * 1000_000L + System.nanoTime() % 1000_000L;
+        try (Statement stmt = conn.createStatement()) {
+            // given
+            stmt.executeUpdate("insert into weather(ts) values('1969-12-31 00:00:00.000000123')");
+            stmt.executeUpdate("insert into weather(ts) values('1969-12-31 23:59:59.999999999')");
+            stmt.executeUpdate("insert into weather(ts) values('1970-01-01 00:00:00.000')");
+            stmt.executeUpdate("insert into weather(ts) values('1970-01-01 00:00:00.000000001')");
+            stmt.executeUpdate("insert into weather(ts) values(" + now + ")");
+            ResultSet rs = stmt.executeQuery("select * from weather order by ts asc");
+            ResultSetMetaData metaData = rs.getMetaData();
+            Assert.assertEquals(2, metaData.getColumnCount());
+
+            // when
+            rs.next();
+            // then
+            Timestamp ts = rs.getTimestamp("ts");
+            Assert.assertEquals("1969-12-31 08:00:00.000", TimestampUtil.longToDatetime(ts.getTime()));
+            Assert.assertEquals(123, ts.getNanos());
 
             // when
             rs.next();
             // then
             ts = rs.getTimestamp("ts");
             Assert.assertEquals("1970-01-01 07:59:59.999", TimestampUtil.longToDatetime(ts.getTime()));
+            Assert.assertEquals(999999999, ts.getNanos());
+
+            // when
+            rs.next();
+            // then
+            ts = rs.getTimestamp("ts");
+            Assert.assertEquals("1970-01-01 08:00:00.000", TimestampUtil.longToDatetime(ts.getTime()));
+            Assert.assertEquals(0, ts.getNanos());
+
+            // when
+            rs.next();
+            // then
+            ts = rs.getTimestamp("ts");
+            Assert.assertEquals("1970-01-01 08:00:00.000", TimestampUtil.longToDatetime(ts.getTime()));
+            Assert.assertEquals(1, ts.getNanos());
+
+            // when
+            rs.next();
+            // then
+            ts = rs.getTimestamp("ts");
+            String s = String.valueOf(now);
+            Assert.assertEquals(Long.parseLong(s.substring(0, s.length() - 6)), ts.getTime());
+            Assert.assertEquals(Long.parseLong(s.substring(s.length() - 9)), ts.getNanos());
         }
     }
 
-    @Before
-    public void before() throws SQLException {
-        conn = DriverManager.getConnection("jdbc:TAOS://" + host + ":6030/?user=root&password=taosdata");
+    private Connection createEnvironment(String precision) throws SQLException {
+        String url = "jdbc:TAOS://" + host + ":6030/?user=root&password=taosdata&timezone=UTC";
+        String createSql = "create database if not exists test_timestamp keep 36500";
+        if (!isEmpty(precision)) {
+            createSql += " precision '" + precision + "'";
+        }
+        conn = DriverManager.getConnection(url);
         Statement stmt = conn.createStatement();
         stmt.execute("drop database if exists test_timestamp");
-        stmt.execute("create database if not exists test_timestamp keep 36500");
+        stmt.execute(createSql);
         stmt.execute("use test_timestamp");
         stmt.execute("create table weather(ts timestamp,f1 float)");
         stmt.close();
+        return conn;
+    }
+
+    private boolean isEmpty(String string) {
+        return null == string || string.trim().equals("");
     }
 
     @After
     public void after() throws SQLException {
-        Statement stmt = conn.createStatement();
-        stmt.execute("drop database if exists test_timestamp");
-        if (conn != null)
+        if (conn != null) {
+            Statement stmt = conn.createStatement();
+            stmt.execute("drop database if exists test_timestamp");
+            stmt.close();
             conn.close();
+        }
     }
 }