tanishqgandhi1908 commented on code in PR #6502:
URL: https://github.com/apache/texera/pull/6502#discussion_r3640471179
##########
common/workflow-core/src/main/scala/org/apache/texera/amber/core/storage/FileResolver.scala:
##########
@@ -75,9 +75,19 @@ object FileResolver {
filePath.toUri
}
+ // The resource-type segment that identifies a dataset logical path. The
prefix selects the
+ // resource kind (and thus the backing table) during resolution, so it is
REQUIRED: a path
+ // without it is not a dataset path. Other resource types (e.g. models) get
their own prefix.
+ val DATASET_PATH_PREFIX = "datasets"
+
/**
* Parses a dataset file path and extracts its components.
- * Expected format: /ownerEmail/datasetName/versionName/fileRelativePath
+ *
+ * Expected format:
/datasets/ownerEmail/datasetName/versionName/fileRelativePath
+ *
+ * The leading [[DATASET_PATH_PREFIX]] segment is required — it identifies
the path as a
Review Comment:
Trimmed the comment to necessary text
--
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]
