amazon-archives / sql-jdbc

🔍 Open Distro for Elasticsearch JDBC Driver
Apache License 2.0
111 stars 49 forks source link

Accept null 'columnNamePattern' on DatabaseMetaData.getColumns() - #11 #12

Closed tbrugz closed 5 years ago

tbrugz commented 5 years ago

Issue #, if available: #11

Description of changes:

Changes constructor of DatabaseMetaDataImpl.ColumnMetadataStatement to allow null on columnNamePattern parameter

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

arsen-es commented 5 years ago

Thank you for the change. Could you please also include a test which covers the null argument code path?

tbrugz commented 5 years ago

added tests for null and not null columnNamePattern argument