tanishqgandhi1908 opened a new pull request, #6502:
URL: https://github.com/apache/texera/pull/6502

   ### What changes were proposed in this PR?
   
   Adds an explicit resource-type prefix to asset logical file paths, changing 
the format from 
   
   `/<owner>/<name>/<version>/<file>` to
   `/datasets/<owner>/<name>/<version>/<file>`.
   
   Paths are namespaced under a resource-type segment so different resource 
kinds can be distinguished from a single logical path. The resolver strips a 
recognized prefix and parses paths without one exactly as before, so there is 
**no behavior
   change** for existing assets. A leading segment that isn't a recognized type 
is left untouched (so an unregistered segment like `models` is not stripped 
yet).
   
   - **Path resolver:** strips a known resource-type prefix in one place before 
parsing; unprefixed (legacy) paths still parse for backward compatibility.
   - **File tree:** the tree is rooted at a resource-type node, with 
owner/name/version nested underneath; owner-email lookup and empty-tree 
handling are hardened.
   - **Frontend:** file-tree relative-path handling and the selection modal 
account for the prefix; an unused client-side path parser with no callers is 
removed.
   - **Python file API:** strips the prefix on parse and re-emits it when 
requesting a presigned download URL.
   
   ### Any related issues, documentation, discussions?
   Closes #6495.
   
   ### How was this PR tested?
   New and updated unit tests, all passing locally:
   - Resolver: an unprefixed path resolves identically to the prefixed path; an 
unregistered prefix is not stripped; a too-few-segments path is rejected.
   - File tree: the tree is rooted at the resource-type node with 
owner/name/version nested underneath.
   - Frontend: relative-path extraction accounts for the new leading segment.
   - Python file API: the prefix is stripped on parse and re-emitted on presign.
   
   ### Was this PR authored or co-authored using generative AI tooling?
   Generated-by: Claude Code (Claude Opus 4.8)


-- 
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]

Reply via email to