test querying with tz in postgreSQL

Author: pacman82

.github/workflows/test.yml

@@ -18,6 +18,15 @@ jobs:
           ACCEPT_EULA: Y
           SA_PASSWORD: My@Test@Password1
 
+      postgres:
+        image: postgres:13
+        ports:
+          - "5432:5432"
+        env:
+          POSTGRES_DB: test
+          POSTGRES_USER: test
+          POSTGRES_PASSWORD: test
+
     steps:
       - name: Checkout
         uses: actions/checkout@v2

docker-compose.yml

@@ -9,6 +9,15 @@ services:
       - MSSQL_SA_PASSWORD=My@Test@Password1
     command: ["/opt/mssql/bin/sqlservr", "--accept-eula", "--reset-sa-password"]
 
+  postgres:
+    image: postgres:13
+    ports:
+      - "5432:5432"
+    environment:
+      POSTGRES_DB: test
+      POSTGRES_USER: test
+      POSTGRES_PASSWORD: test
+
   dev:
     build: docker/dev
     volumes:
@@ -17,5 +26,5 @@ services:
     # Overrides default command so things don't shut down after the process ends.
     command: sleep infinity
 
-    # Runs app on the same network as the database container, allows "forwardPorts" in devcontainer.json function.
-    network_mode: service:mssql
+    # Allows accessing dbs through the port forwardings. dev container behaves like host for networking
+    network_mode: host

docker/dev/Dockerfile

@@ -10,12 +10,17 @@ RUN curl https://packages.microsoft.com/keys/microsoft.asc | apt-key add -
 RUN curl https://packages.microsoft.com/config/debian/10/prod.list > /etc/apt/sources.list.d/mssql-release.list
 RUN echo msodbcsql17 msodbcsql/ACCEPT_EULA boolean true | debconf-set-selections
 
-# [Optional] Uncomment this section to install additional packages.
+# This section install additional components
 RUN apt-get update && export DEBIAN_FRONTEND=noninteractive \
-    && apt-get -y install --no-install-recommends \
-    build-essential \
-    unixodbc-dev \
-    msodbcsql17
+	&& apt-get -y install --no-install-recommends \
+	build-essential \
+	unixodbc-dev \
+	msodbcsql17 \
+	odbc-postgresql
+
+# Fix PostgreSQL driver paths
+RUN sed --in-place 's/psqlodbca.so/\/usr\/lib\/x86_64-linux-gnu\/odbc\/psqlodbca.so/' /etc/odbcinst.ini
+RUN sed --in-place 's/psqlodbcw.so/\/usr\/lib\/x86_64-linux-gnu\/odbc\/psqlodbcw.so/' /etc/odbcinst.ini
 
 # Setup cargo and install cargo packages required for tests
 USER vscode

src/query/strategy.rs

@@ -1,7 +1,7 @@
 use std::convert::TryInto;
 
 use anyhow::Error;
-use log::{debug, warn};
+use log::{debug, info, warn};
 use odbc_api::{
     buffers::{AnyColumnView, BufferDescription, BufferKind},
     sys::SqlDataType,
@@ -143,6 +143,7 @@ pub fn strategy_from_column_description(
             if db_name == "Microsoft SQL Server" {
                 // -155 is an indication for "Timestamp with timezone" on Microsoft SQL Server. We
                 // give it special treatment so users can sort by time instead lexographically.
+                info!("Detected Timestamp type with time zone. Appyling instant semantics for column {}.", cd.name_to_string()?);
                 timestamp_tz(precision.try_into().unwrap(), repetition)?
             } else {
                 unknown_non_char_type(cd, cursor, index, repetition)?