From acc5863fe206c9daa1b15b17036f31214c923afb Mon Sep 17 00:00:00 2001
From: Johan Wikman <johan.wikman@mariadb.com>
Date: Tue, 11 Jun 2019 12:56:16 +0300
Subject: [PATCH] MXS-2442 ':N' accepted as pos. parameter in Oracle mode

---
 query_classifier/qc_sqlite/qc_sqlite.cc | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/query_classifier/qc_sqlite/qc_sqlite.cc b/query_classifier/qc_sqlite/qc_sqlite.cc
index 5d09b5f45..20271885c 100644
--- a/query_classifier/qc_sqlite/qc_sqlite.cc
+++ b/query_classifier/qc_sqlite/qc_sqlite.cc
@@ -878,9 +878,18 @@ public:
                         }
                     }
                 }
-                else if (zToken[0] != '?')
+                else
                 {
-                    MXS_WARNING("%s reported as VARIABLE.", zToken);
+                    // '?' is always accepted as a positional parameter.
+                    if (zToken[0] != '?')
+                    {
+                        // If the mode is Oracle then :N is accepted as well.
+                        if (zToken[0] != ':' || this_thread.sql_mode != QC_SQL_MODE_ORACLE)
+                        {
+                            // Everything else is unexpected, but harmless.
+                            MXS_WARNING("%s reported as VARIABLE.", zToken);
+                        }
+                    }
                 }
             }
             break;