aicam commented on code in PR #6502:
URL: https://github.com/apache/texera/pull/6502#discussion_r3609165397
##########
common/workflow-core/src/main/scala/org/apache/texera/amber/core/storage/FileResolver.scala:
##########
@@ -75,9 +75,18 @@ object FileResolver {
filePath.toUri
}
+ private val RESOURCE_TYPE_PREFIXES = Set("datasets")
+
/**
* Parses a dataset file path and extracts its components.
- * Expected format: /ownerEmail/datasetName/versionName/fileRelativePath
+ *
+ * Two path shapes are accepted:
+ * - Prefixed:
/datasets/ownerEmail/datasetName/versionName/fileRelativePath
Review Comment:
In the new design, we will have a new table named "model", in this regard,
an unprefixed path can not be resolved because the prefix indicate the table,
please update implementation.
--
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]
