[ 
https://issues.apache.org/jira/browse/DRILL-8328?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17613155#comment-17613155
 ] 

ASF GitHub Bot commented on DRILL-8328:
---------------------------------------

jnturton commented on code in PR #2668:
URL: https://github.com/apache/drill/pull/2668#discussion_r985373328


##########
contrib/storage-http/src/main/java/org/apache/drill/exec/store/http/util/SimpleHttp.java:
##########
@@ -870,6 +928,22 @@ public static HttpStoragePlugin 
getStoragePlugin(DrillbitContext context, String
     }
   }
 
+  public static String addBackTicksToAliasName(String plugin) {

Review Comment:
   The identifier quoting character in Drill is configurable and not 
necessarily a backtick. See  
https://drill.apache.org/docs/lexical-structure/#identifier-quotes. Is there 
any existing code for unwrapping and wrapping identifiers like this that could 
be reused?





> HTTP UDF Not Resolving Storage Aliases
> --------------------------------------
>
>                 Key: DRILL-8328
>                 URL: https://issues.apache.org/jira/browse/DRILL-8328
>             Project: Apache Drill
>          Issue Type: Bug
>          Components: Storage - HTTP
>    Affects Versions: 1.20.0
>            Reporter: Charles Givre
>            Assignee: Charles Givre
>            Priority: Blocker
>             Fix For: 1.20.3
>
>
> The http_request function currently does not resolve plugin aliases 
> correctly.  This PR fixes that issue. 



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to