Yicong-Huang opened a new issue, #5429:
URL: https://github.com/apache/texera/issues/5429

   ### Task Summary
   
   `PythonWorkflowWorker` (in 
`amber/src/main/scala/org/apache/texera/amber/engine/architecture/pythonworker/`)
 calls `PveManager.getPythonBin(cuid, pveName)` to resolve the Python 
interpreter path for a workflow's Python venv. `PveManager` lives under 
`amber/src/main/scala/org/apache/texera/web/resource/pythonvirtualenvironment/` 
— i.e. the engine reaches into the web layer for a pure utility computation.
   
   The `getPythonBin` body is just regex validation, path construction, and a 
file-existence check. No web concerns. Move it (with its companion bits — 
`SafePveName.pattern`, `VenvRoot`, `pythonBinPath`) to 
`common/config/src/main/scala/org/apache/texera/amber/util/PveResolver.scala` 
so both engine and web depend on a shared util instead of engine depending on a 
web resource.
   
   `PveManager`'s REST surface stays in web; its body delegates to the new 
`PveResolver`.
   
   Part of #5424.
   
   ### Task Type
   
   - [x] Refactor / Cleanup


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