You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "Robert Huffman (Jira)" <ji...@apache.org> on 2020/02/16 17:56:00 UTC

[jira] [Created] (DBUTILS-144) RowProcessor.asMap is invalid for multiple computed columns with no aliases

Robert Huffman created DBUTILS-144:
--------------------------------------

             Summary: RowProcessor.asMap is invalid for multiple computed columns with no aliases
                 Key: DBUTILS-144
                 URL: https://issues.apache.org/jira/browse/DBUTILS-144
             Project: Commons DbUtils
          Issue Type: Bug
    Affects Versions: 1.7
         Environment: JDK: OpenJDK 1.8.0_212

JDBC Driver: JTDS 1.2.8.1

Database: SQL Server 2017 (Docker image mcr.microsoft.com/mssql/server:2017-latest)

 
            Reporter: Robert Huffman


At least one database driver (JDTS) returns an empty string for both ResultSet.getColumnName and ResultSet.getColumnLabel if it is not a table column and it has no alias. If there are more than one such column in a result set, then BasicRowProcessor.toMap will put all of them in the map using the empty string as key. This means only the last column added will be retained in the map.

For example, this query will create map with a single entry (the result of the second case statement) using the key "":

{{SELECT}}
{{  CASE WHEN foo = 0 THEN '' ELSE 'FOO' END,}}
{{  CASE WHEN bar = 0 THEN '' ELSE 'BAR' END}}
FROM FooBar

Presumably this is not an issue for most usages. However, for a tool that does something like run canned queries and output the result as JSON, this is a problem. I propose that if both the columnName and columnLabel are null or zero length, the column index (converted to a String) be used as the key.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)