Patch release for SQL 1.2.1

ballerina/Ballerina.toml

@@ -1,18 +1,18 @@
 [package]
 org = "ballerina"
 name = "sql"
-version = "1.2.0"
+version = "1.2.1"
 authors = ["Ballerina"]
 keywords = ["database", "client", "network", "SQL", "RDBMS"]
 repository = "https://github.com/ballerina-platform/module-ballerina-sql"
 license = ["Apache-2.0"]
 distribution = "slbeta6"
 
 [[platform.java11.dependency]]
-path = "../native/build/libs/sql-native-1.2.0.jar"
+path = "../native/build/libs/sql-native-1.2.1.jar"
 
 [[platform.java11.dependency]]
-path = "../test-utils/build/libs/sql-test-utils-1.2.0.jar"
+path = "../test-utils/build/libs/sql-test-utils-1.2.1.jar"
 scope = "testOnly"
 
 [[platform.java11.dependency]]

ballerina/CompilerPlugin.toml

@@ -3,4 +3,4 @@ id = "sql-compiler-plugin"
 class = "io.ballerina.stdlib.sql.compiler.SQLCompilerPlugin"
 
 [[dependency]]
-path = "../compiler-plugin/build/libs/sql-compiler-plugin-1.2.0.jar"
+path = "../compiler-plugin/build/libs/sql-compiler-plugin-1.2.1.jar"

ballerina/Dependencies.toml

@@ -277,7 +277,7 @@ dependencies = [
 [[package]]
 org = "ballerina"
 name = "sql"
-version = "1.2.0"
+version = "1.2.1"
 dependencies = [
 	{org = "ballerina", name = "file"},
 	{org = "ballerina", name = "io"},

changelog.md

@@ -8,6 +8,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Added
 
+### Changed
+
+## [1.2.1] - 2021-12-13
+
+### Added
+
+### Changed
+- [Fix Compiler plugin crash when variable is passed for `sql:connectionPool`](https://github.com/ballerina-platform/ballerina-standard-library/issues/2536)
+
+## [1.2.0] - 2021-12-13
+
 ### Changed
 - [Fix queryRow method to use module's provided resultParameterProcessor](https://github.com/ballerina-platform/ballerina-standard-library/issues/2466)
 

compiler-plugin-tests/src/test/java/io/ballerina/stdlib/sql/compiler/CompilerPluginTest.java

@@ -140,4 +140,17 @@ public void testOutParameterHint() {
         });
 
     }
+
+    @Test
+    public void testConnectionPoolWithVariable() {
+        Package currentPackage = loadPackage("sample4");
+        PackageCompilation compilation = currentPackage.getCompilation();
+        DiagnosticResult diagnosticResult = compilation.diagnosticResult();
+        List<Diagnostic> errorDiagnosticsList = diagnosticResult.diagnostics().stream()
+                .filter(r -> r.diagnosticInfo().severity().equals(DiagnosticSeverity.ERROR))
+                .collect(Collectors.toList());
+        long availableErrors = errorDiagnosticsList.size();
+
+        Assert.assertEquals(availableErrors, 0);
+    }
 }

compiler-plugin-tests/src/test/resources/diagnostics/sample4/Ballerina.toml

@@ -0,0 +1,4 @@
+[package]
+org = "sql_test"
+name = "sample4"
+version = "0.1.0"

compiler-plugin-tests/src/test/resources/diagnostics/sample4/main.bal

@@ -0,0 +1,32 @@
+// Copyright (c) 2021, WSO2 Inc. (http://www.wso2.org) All Rights Reserved.
+//
+// WSO2 Inc. licenses this file to you under the Apache License,
+// Version 2.0 (the "License"); you may not use this file except
+// in compliance with the License.
+// You may obtain a copy of the License at
+//
+// http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing,
+// software distributed under the License is distributed on an
+// "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+// KIND, either express or implied.  See the License for the
+// specific language governing permissions and limitations
+// under the License.
+
+import ballerina/sql;
+
+public function main() {
+
+    int connectionNum = 5;
+    int connectionNumInvalid = -5;
+
+    int|sql:ConnectionPool pool = {
+        maxOpenConnections: connectionNum
+    };
+
+    sql:ConnectionPool|int poolInvalid = {
+        maxOpenConnections: connectionNumInvalid
+    };
+
+}

compiler-plugin/src/main/java/io/ballerina/stdlib/sql/compiler/analyzer/ConnectionPoolConfigAnalyzer.java

@@ -99,7 +99,7 @@ public void perform(SyntaxNodeAnalysisContext ctx) {
                 ExpressionNode valueNode = ((SpecificFieldNode) field).valueExpr().get();
                 switch (name) {
                     case Constants.ConnectionPool.MAX_OPEN_CONNECTIONS:
-                        int maxOpenConnections = Integer.parseInt(getTerminalNodeValue(valueNode));
+                        int maxOpenConnections = Integer.parseInt(getTerminalNodeValue(valueNode, "1"));
                         if (maxOpenConnections < 1) {
                             DiagnosticInfo diagnosticInfo = new DiagnosticInfo(SQL_101.getCode(), SQL_101.getMessage(),
                                     SQL_101.getSeverity());
@@ -110,7 +110,7 @@ public void perform(SyntaxNodeAnalysisContext ctx) {
                         }
                         break;
                     case Constants.ConnectionPool.MIN_IDLE_CONNECTIONS:
-                        int minIdleConnection = Integer.parseInt(getTerminalNodeValue(valueNode));
+                        int minIdleConnection = Integer.parseInt(getTerminalNodeValue(valueNode, "0"));
                         if (minIdleConnection < 0) {
                             DiagnosticInfo diagnosticInfo = new DiagnosticInfo(SQL_102.getCode(), SQL_102.getMessage(),
                                     SQL_102.getSeverity());
@@ -120,7 +120,7 @@ public void perform(SyntaxNodeAnalysisContext ctx) {
                         }
                         break;
                     case Constants.ConnectionPool.MAX_CONNECTION_LIFE_TIME:
-                        float maxConnectionTime = Float.parseFloat(getTerminalNodeValue(valueNode));
+                        float maxConnectionTime = Float.parseFloat(getTerminalNodeValue(valueNode, "30"));
                         if (maxConnectionTime < 30) {
                             DiagnosticInfo diagnosticInfo = new DiagnosticInfo(SQL_103.getCode(), SQL_103.getMessage(),
                                     SQL_103.getSeverity());
@@ -137,15 +137,16 @@ public void perform(SyntaxNodeAnalysisContext ctx) {
         }
     }
 
-    private String getTerminalNodeValue(Node valueNode) {
-        String value;
+    private String getTerminalNodeValue(Node valueNode, String defaultValue) {
+        String value = defaultValue;
         if (valueNode instanceof BasicLiteralNode) {
             value = ((BasicLiteralNode) valueNode).literalToken().text();
-        } else {
+        } else if (valueNode instanceof UnaryExpressionNode) {
             UnaryExpressionNode unaryExpressionNode = (UnaryExpressionNode) valueNode;
             value = unaryExpressionNode.unaryOperator() +
                     ((BasicLiteralNode) unaryExpressionNode.expression()).literalToken().text();
         }
+        // Currently we cannot process values from variables, this needs code flow analysis
         return value.replaceAll(UNNECESSARY_CHARS_REGEX, "");
     }
 

gradle.properties

@@ -1,5 +1,5 @@
 group=io.ballerina.stdlib
-version=1.2.0
+version=1.2.1
 
 puppycrawlCheckstyleVersion=8.18
 hikkariLibVersion=3.3.1