jonvex commented on code in PR #11237:
URL: https://github.com/apache/hudi/pull/11237#discussion_r1602333216
##########
hudi-spark-datasource/hudi-spark-common/src/main/scala/org/apache/hudi/HoodieBootstrapRelation.scala:
##########
@@ -115,8 +113,8 @@ abstract class BaseHoodieBootstrapRelation(override val
sqlContext: SQLContext,
fileSlices.map { fileSlice =>
val baseFile = fileSlice.getBaseFile.get()
if (baseFile.getBootstrapBaseFile.isPresent) {
- val partitionValues =
getPartitionColumnsAsInternalRowInternal(baseFile.getBootstrapBaseFile.get.getPathInfo,
- bootstrapBasePath, extractPartitionValuesFromPartitionPath =
isPartitioned)
+ val partitionValues =
getPartitionColumnsAsInternalRowInternal(baseFile.getPathInfo,
Review Comment:
No, we are just going to use the regular base path because that is what we
will be doing if we ever do any updates, or compaction, or clustering anyway.
So I think this is a better solution than trying to parse the source table path
correctly.
--
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]
