Skip to content

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 43bacc9

Browse filesBrowse files
NivinCSNivin
authored and
Nivin
committed
Enable mixed-case support for Oracle
1 parent 37e9ca9 commit 43bacc9
Copy full SHA for 43bacc9

File tree

2 files changed

+10
-0
lines changed
Filter options

2 files changed

+10
-0
lines changed

‎presto-docs/src/main/sphinx/connector/oracle.rst

Copy file name to clipboardExpand all lines: presto-docs/src/main/sphinx/connector/oracle.rst
+4
Original file line numberDiff line numberDiff line change
@@ -68,6 +68,10 @@ Property Name Description
6868

6969
``case-insensitive-name-matching.cache-ttl`` Duration for which remote dataset and table names will be
7070
cached. Set to ``0ms`` to disable the cache. ``1m``
71+
72+
``case-sensitive-name-matching`` Enable case sensitive identifier support for schema and table ``false``
73+
names for the connector. When disabled, names are matched
74+
case-insensitively using lowercase normalization.
7175
================================================== ==================================================================== ===========
7276

7377
Querying Oracle

‎presto-oracle/src/main/java/com/facebook/presto/plugin/oracle/OracleClient.java

Copy file name to clipboardExpand all lines: presto-oracle/src/main/java/com/facebook/presto/plugin/oracle/OracleClient.java
+6
Original file line numberDiff line numberDiff line change
@@ -167,4 +167,10 @@ public Optional<ReadMapping> toPrestoType(ConnectorSession session, JdbcTypeHand
167167
}
168168
return super.toPrestoType(session, typeHandle);
169169
}
170+
171+
@Override
172+
public String normalizeIdentifier(ConnectorSession session, String identifier)
173+
{
174+
return caseSensitiveNameMatchingEnabled ? identifier : identifier.toLowerCase(ENGLISH);
175+
}
170176
}

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.