[
https://issues.apache.org/jira/browse/DRILL-8147?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17542829#comment-17542829
]
James Turton edited comment on DRILL-8147 at 5/27/22 8:33 AM:
--------------------------------------------------------------
Hi [~pseudomo], doubled backslashes will also work here. If we change to
prevent the use of backslash as an escape character then users will no longer
be able to enter paths containing backticks or Unicode chars that are not on
their keyboards. Not that one would expect much of a need for that. [~cgivre],
any thoughts about whether this should change or remain as it is?
{code:java}
SELECT * FROM dfs.`c:\\AnyFolder\\udontlikewindo.csv`;{code}
CC [~luoc]
was (Author: dzamo):
Hi [~pseudomo], doubled backslashes will also work here. If we change to
prevent the use of backslash as an escape character then users will no longer
be able to enter paths containing backticks or Unicode chars that are not on
their keyboards. Not that one would expect much of a need for that. [~cgivre],
any thoughts about whether this should change or remain as it is?
{code:java}
SELECT * FROM dfs.`c:\\AnyFolder\\udontlikewindo.csv`;{code}
> Drill fails to read files that begin with 'u' on Windows
> --------------------------------------------------------
>
> Key: DRILL-8147
> URL: https://issues.apache.org/jira/browse/DRILL-8147
> Project: Apache Drill
> Issue Type: Bug
> Components: Storage - Text & CSV
> Reporter: pseudomo
> Priority: Major
>
> It is very easy to reproduce on Windows.
> Just try to run this select. It is not necessary to actually have this folder
> of file.
> SELECT * FROM dfs.`c:\AnyFolder{*}\u{*}dontlikewindo.csv`;
> SQL Error: VALIDATION ERROR: Unable to parse unicode value: dont
> The workaround is to replace "\" with "/" which will work on Windows 10.
> Stack trace:
> {{Caused by: java.lang.IllegalArgumentException: Unable to parse unicode
> value: dont }}
> {{ at
> org.apache.commons.lang3.text.translate.UnicodeUnescaper.translate(UnicodeUnescaper.java:59)
> }}
> {{ at
> org.apache.commons.lang3.text.translate.AggregateTranslator.translate(AggregateTranslator.java:55)
> }}
> {{ at
> org.apache.commons.lang3.text.translate.CharSequenceTranslator.translate(CharSequenceTranslator.java:90)
> }}
> {{ at
> org.apache.commons.lang3.text.translate.CharSequenceTranslator.translate(CharSequenceTranslator.java:64)
> }}
> {{ at
> org.apache.commons.lang3.StringEscapeUtils.unescapeJava(StringEscapeUtils.java:526)
> }}
> {{ at
> org.apache.drill.common.util.DrillStringUtils.unescapeJava(DrillStringUtils.java:55)
> }}
> {{ at
> org.apache.drill.exec.store.dfs.FileSelection.create(FileSelection.java:277)
> }}
> {{ at
> org.apache.drill.exec.store.dfs.WorkspaceSchemaFactory$WorkspaceSchema$FileSelectionInspector.getFileSelection(WorkspaceSchemaFactory.java:860)
> }}
> {{ at
> org.apache.drill.exec.store.dfs.WorkspaceSchemaFactory$WorkspaceSchema$FileSelectionInspector.<init>(WorkspaceSchemaFactory.java:847)
> }}
> {{ at
> org.apache.drill.exec.store.dfs.WorkspaceSchemaFactory$WorkspaceSchema.create(WorkspaceSchemaFactory.java:600)
> }}
> {{ at
> org.apache.drill.exec.store.dfs.WorkspaceSchemaFactory$WorkspaceSchema.create(WorkspaceSchemaFactory.java:295)
> }}
> {{ at
> org.apache.drill.exec.planner.sql.ExpandingConcurrentMap.getNewEntry(ExpandingConcurrentMap.java:96)
> }}
> {{ at
> org.apache.drill.exec.planner.sql.ExpandingConcurrentMap.get(ExpandingConcurrentMap.java:90)
> }}
> {{ at
> org.apache.drill.exec.store.dfs.WorkspaceSchemaFactory$WorkspaceSchema.getTable(WorkspaceSchemaFactory.java:452)
> }}
> {{ at
> org.apache.drill.exec.store.dfs.FileSystemSchemaFactory$FileSystemSchema.getTable(FileSystemSchemaFactory.java:127)
> }}
> {{ at
> org.apache.calcite.jdbc.SimpleCalciteSchema.getImplicitTable(SimpleCalciteSchema.java:83)
> }}
> {{ at
> org.apache.calcite.jdbc.CalciteSchema.getTable(CalciteSchema.java:289) }}
> {{ at
> org.apache.calcite.sql.validate.EmptyScope.resolve_(EmptyScope.java:143) }}
> {{ at
> org.apache.calcite.sql.validate.EmptyScope.resolveTable(EmptyScope.java:99) }}
> {{ at
> org.apache.calcite.sql.validate.DelegatingScope.resolveTable(DelegatingScope.java:203)
> }}
> {{ at
> org.apache.calcite.sql.validate.IdentifierNamespace.resolveImpl(IdentifierNamespace.java:105)
> }}
> {{ at
> org.apache.calcite.sql.validate.IdentifierNamespace.validateImpl(IdentifierNamespace.java:177)
> }}
> {{ at
> org.apache.calcite.sql.validate.AbstractNamespace.validate(AbstractNamespace.java:84)
> }}
> {{ at
> org.apache.calcite.sql.validate.SqlValidatorImpl.validateNamespace(SqlValidatorImpl.java:1009)
> }}
> {{ at
> org.apache.calcite.sql.validate.SqlValidatorImpl.validateQuery(SqlValidatorImpl.java:969)
> }}
> {{ at
> org.apache.calcite.sql.validate.SqlValidatorImpl.validateFrom(SqlValidatorImpl.java:3129)
> }}
> {{ at
> org.apache.drill.exec.planner.sql.conversion.DrillValidator.validateFrom(DrillValidator.java:80)
> }}
> {{ at
> org.apache.calcite.sql.validate.SqlValidatorImpl.validateFrom(SqlValidatorImpl.java:3111)
> }}
> {{ at
> org.apache.drill.exec.planner.sql.conversion.DrillValidator.validateFrom(DrillValidator.java:80)
> }}
> {{ at
> org.apache.calcite.sql.validate.SqlValidatorImpl.validateSelect(SqlValidatorImpl.java:3383)
> }}
> {{ at
> org.apache.calcite.sql.validate.SelectNamespace.validateImpl(SelectNamespace.java:60)
> }}
> {{ at
> org.apache.calcite.sql.validate.AbstractNamespace.validate(AbstractNamespace.java:84)
> }}
> {{ at
> org.apache.calcite.sql.validate.SqlValidatorImpl.validateNamespace(SqlValidatorImpl.java:1009)
> }}
> {{ at
> org.apache.calcite.sql.validate.SqlValidatorImpl.validateQuery(SqlValidatorImpl.java:969)
> }}
> {{ at org.apache.calcite.sql.SqlSelect.validate(SqlSelect.java:216) }}
> {{ at
> org.apache.calcite.sql.validate.SqlValidatorImpl.validateScopedExpression(SqlValidatorImpl.java:944)
> }}
> {{ at
> org.apache.calcite.sql.validate.SqlValidatorImpl.validate(SqlValidatorImpl.java:651)
> }}
> {{ at
> org.apache.drill.exec.planner.sql.conversion.SqlConverter.validate(SqlConverter.java:190)
> }}
> {{ at
> org.apache.drill.exec.planner.sql.handlers.DefaultSqlHandler.validateNode(DefaultSqlHandler.java:647)
> }}
> {{ at
> org.apache.drill.exec.planner.sql.handlers.DefaultSqlHandler.validateAndConvert(DefaultSqlHandler.java:195)
> }}
> {{ at
> org.apache.drill.exec.planner.sql.handlers.DefaultSqlHandler.getPlan(DefaultSqlHandler.java:169)
> }}
> {{ at
> org.apache.drill.exec.planner.sql.DrillSqlWorker.getQueryPlan(DrillSqlWorker.java:283)
> }}
> {{ at
> org.apache.drill.exec.planner.sql.DrillSqlWorker.getPhysicalPlan(DrillSqlWorker.java:163)
> }}
> {{ at
> org.apache.drill.exec.planner.sql.DrillSqlWorker.convertPlan(DrillSqlWorker.java:140)
> }}
> {{ at
> org.apache.drill.exec.planner.sql.DrillSqlWorker.getPlan(DrillSqlWorker.java:93)
> }}
> {{ at org.apache.drill.exec.work.foreman.Foreman.runSQL(Foreman.java:593)
> }}
> {{ at org.apache.drill.exec.work.foreman.Foreman.run(Foreman.java:274) }}
> {{ at .......(:0) }}
> {{Caused by: java.lang.NumberFormatException: For input string: "dont" under
> radix 16 }}
> {{ at .......(:0) }}
> {{ at
> org.apache.commons.lang3.text.translate.UnicodeUnescaper.translate(UnicodeUnescaper.java:56)
> }}
> {{ ... 47 more}}
>
>
>
--
This message was sent by Atlassian Jira
(v8.20.7#820007)
