alexeykudinkin commented on code in PR #6264:
URL: https://github.com/apache/hudi/pull/6264#discussion_r941962972
##########
hudi-spark-datasource/hudi-spark/src/main/scala/org/apache/spark/sql/hudi/analysis/HoodieAnalysis.scala:
##########
@@ -473,26 +473,25 @@ case class HoodieResolveReferences(sparkSession:
SparkSession) extends Rule[Logi
"version expression is not supported for time travel")
}
- val tableIdentifier =
sparkAdapter.getCatalystPlanUtils.toTableIdentifier(plan)
Review Comment:
Alright, so after taking a deeper look this seems to be a piece of code that
is problematic -- `toTableIdentifier` assumes that the name is 2 name-parts,
when it can be 3.
What i think we need to do is simply make sure we handle it properly here. I
don't believe we need to change anything around how the catalogs/tables are
resolved actually.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
