Github user srowen commented on the issue:
https://github.com/apache/spark/pull/17655
This change looks good. You're saying that you think the Avro issue is
another instance of the same issue, just in Avro? if so then, yeah, not sure we
can fix that unfortunately, not directly.
I did one more pass, while we (you) are cleaning up some loose ends, and
spotted two more instances that look questionable to me.
`ResolveTableValuedFunctions`:
```
override def apply(plan: LogicalPlan): LogicalPlan = plan
resolveOperators {
case u: UnresolvedTableValuedFunction if
u.functionArgs.forall(_.resolved) =>
builtinFunctions.get(u.functionName.toLowerCase()) match {
```
I'm guessing this is another instance where the function name should _not_
be locale-sensitive. Not sure how I missed it.
And related to your change here, in `SharedState` line 117:
```
val globalTempDB =
sparkContext.conf.get(GLOBAL_TEMP_DATABASE).toLowerCase
```
should be non-locale-sensitive too?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
