Skip to content

Add bulk uploader from on-prem fileserver with S3 API#235

Open
dmoore247 wants to merge 6 commits into
mainfrom
feature/mover
Open

Add bulk uploader from on-prem fileserver with S3 API#235
dmoore247 wants to merge 6 commits into
mainfrom
feature/mover

Conversation

@dmoore247

Copy link
Copy Markdown
Collaborator

Parallel copy from S3 api to /Volumes

/security-review:

  All input sources (config.yaml, manifest Delta table, secrets scope) are operator-controlled and 
  qualify as trusted under the precedent that notebook vulnerabilities require a concrete attack
  path from untrusted input. No such path exists in the added code. 

Parallel copy from S3 api to /Volumes
@dmoore247 dmoore247 added the enhancement New feature or request label Jun 26, 2026
dmoore247 and others added 5 commits June 25, 2026 21:53
Replaces the static NOTEBOOK_PATH config entry with a runtime lookup via
dbruntime get_context().notebookPath, so the coordinator submits workers
against the actual notebook location without requiring per-user config.

Co-authored-by: Isaac
- 01-setup-MANIFEST: fix NameError (`len(cfg)` -> `len(_cfg)`) and use
  the MANIFEST_TABLE config var instead of a hardcoded table name.
- Large File Copy Benchmark v6: clear hardcoded `worker_pool_id`
  default and current value, and drop the workspace-specific
  experimentId from notebook metadata.

Co-authored-by: Isaac
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant