| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
♻️ Refactor(data-process): replace Ray with Celery prefork (#3888) * refactor(data-process): replace Ray with Celery prefork Run parsing in process-local Celery prefork children, keep orchestration and forwarding on thread workers, and remove the Ray runtime, dependency, and deployment surface. Co-authored-by: Codex <noreply@openai.com> Generated-by: gpt-5-codex * fix(data-process): preserve sync parser result metadata * fix(data-process): wait for parser readiness in result backend * fix(data-process): tolerate lightweight worker app doubles * test(model): mock authentication for temporary healthcheck * Revert "test(model): mock authentication for temporary healthcheck" This reverts commit 9aa8d45b8eb9b8248b731957d23d851936e03608. * test(data-process): configure Celery URLs in isolated tests * test(data-process): cover prefork task boundaries * test(data-process): isolate prefork coverage from redis backend * test(data-process): cover lazy prefork branches * fix(data-process): complete celery prefork migration review fixes * test(data-process): align prefork tests with current APIs * fix(data-process): avoid logging user-controlled image paths * test(data-process): isolate router tests from celery bootstrap * test(model): provide auth for temporary healthcheck * test(data-process): stub redis chunk loader in router test * test(data-process): cover parser cancellation and lifecycle helpers * test(data-process): re-export process_sync from tasks module Keep the synchronous parse entry point reachable through the tasks module so coverage and isolation tests can patch one import site. Use a redundant-alias re-export to satisfy F401. * 🧪 Test: restore stubbed sys.modules after app router fixture Problem: the stub_modules fixture in test_data_process_app.py assigned stub modules directly to sys.modules without restoring them, so the stubs leaked into every later test file sharing the same pytest process. Running test_data_process_app.py before test_data_process_service.py failed all 52 service tests in setUp with AttributeError: '_CeleryAppStub' object has no attribute 'reset_mock'. The leak was invisible to CI because run_all_test.py runs each test file in its own subprocess. Fix: replace the six direct sys.modules assignments with monkeypatch.setitem so the real modules are restored when the fixture tears down. Behavior during each test is unchanged. Verification: - pytest test/backend/app/test_data_process_app.py test/backend/services/test_data_process_service.py -> 76 passed (was 24 passed / 52 failed) - each file alone: 24 passed / 52 passed (no regression) - wider combo (app + services + tasks + worker + coverage): app+services now pass together; the remaining failures are a pre-existing module-level mock leak in test_data_process_service.py that exists on develop and is out of scope here. | 10 天前 | |
♻️ Refactor(data-process): replace Ray with Celery prefork (#3888) * refactor(data-process): replace Ray with Celery prefork Run parsing in process-local Celery prefork children, keep orchestration and forwarding on thread workers, and remove the Ray runtime, dependency, and deployment surface. Co-authored-by: Codex <noreply@openai.com> Generated-by: gpt-5-codex * fix(data-process): preserve sync parser result metadata * fix(data-process): wait for parser readiness in result backend * fix(data-process): tolerate lightweight worker app doubles * test(model): mock authentication for temporary healthcheck * Revert "test(model): mock authentication for temporary healthcheck" This reverts commit 9aa8d45b8eb9b8248b731957d23d851936e03608. * test(data-process): configure Celery URLs in isolated tests * test(data-process): cover prefork task boundaries * test(data-process): isolate prefork coverage from redis backend * test(data-process): cover lazy prefork branches * fix(data-process): complete celery prefork migration review fixes * test(data-process): align prefork tests with current APIs * fix(data-process): avoid logging user-controlled image paths * test(data-process): isolate router tests from celery bootstrap * test(model): provide auth for temporary healthcheck * test(data-process): stub redis chunk loader in router test * test(data-process): cover parser cancellation and lifecycle helpers * test(data-process): re-export process_sync from tasks module Keep the synchronous parse entry point reachable through the tasks module so coverage and isolation tests can patch one import site. Use a redundant-alias re-export to satisfy F401. * 🧪 Test: restore stubbed sys.modules after app router fixture Problem: the stub_modules fixture in test_data_process_app.py assigned stub modules directly to sys.modules without restoring them, so the stubs leaked into every later test file sharing the same pytest process. Running test_data_process_app.py before test_data_process_service.py failed all 52 service tests in setUp with AttributeError: '_CeleryAppStub' object has no attribute 'reset_mock'. The leak was invisible to CI because run_all_test.py runs each test file in its own subprocess. Fix: replace the six direct sys.modules assignments with monkeypatch.setitem so the real modules are restored when the fixture tears down. Behavior during each test is unchanged. Verification: - pytest test/backend/app/test_data_process_app.py test/backend/services/test_data_process_service.py -> 76 passed (was 24 passed / 52 failed) - each file alone: 24 passed / 52 passed (no regression) - wider combo (app + services + tasks + worker + coverage): app+services now pass together; the remaining failures are a pre-existing module-level mock leak in test_data_process_service.py that exists on develop and is out of scope here. | 10 天前 | |
♻️ Refactor(data-process): replace Ray with Celery prefork (#3888) * refactor(data-process): replace Ray with Celery prefork Run parsing in process-local Celery prefork children, keep orchestration and forwarding on thread workers, and remove the Ray runtime, dependency, and deployment surface. Co-authored-by: Codex <noreply@openai.com> Generated-by: gpt-5-codex * fix(data-process): preserve sync parser result metadata * fix(data-process): wait for parser readiness in result backend * fix(data-process): tolerate lightweight worker app doubles * test(model): mock authentication for temporary healthcheck * Revert "test(model): mock authentication for temporary healthcheck" This reverts commit 9aa8d45b8eb9b8248b731957d23d851936e03608. * test(data-process): configure Celery URLs in isolated tests * test(data-process): cover prefork task boundaries * test(data-process): isolate prefork coverage from redis backend * test(data-process): cover lazy prefork branches * fix(data-process): complete celery prefork migration review fixes * test(data-process): align prefork tests with current APIs * fix(data-process): avoid logging user-controlled image paths * test(data-process): isolate router tests from celery bootstrap * test(model): provide auth for temporary healthcheck * test(data-process): stub redis chunk loader in router test * test(data-process): cover parser cancellation and lifecycle helpers * test(data-process): re-export process_sync from tasks module Keep the synchronous parse entry point reachable through the tasks module so coverage and isolation tests can patch one import site. Use a redundant-alias re-export to satisfy F401. * 🧪 Test: restore stubbed sys.modules after app router fixture Problem: the stub_modules fixture in test_data_process_app.py assigned stub modules directly to sys.modules without restoring them, so the stubs leaked into every later test file sharing the same pytest process. Running test_data_process_app.py before test_data_process_service.py failed all 52 service tests in setUp with AttributeError: '_CeleryAppStub' object has no attribute 'reset_mock'. The leak was invisible to CI because run_all_test.py runs each test file in its own subprocess. Fix: replace the six direct sys.modules assignments with monkeypatch.setitem so the real modules are restored when the fixture tears down. Behavior during each test is unchanged. Verification: - pytest test/backend/app/test_data_process_app.py test/backend/services/test_data_process_service.py -> 76 passed (was 24 passed / 52 failed) - each file alone: 24 passed / 52 passed (no regression) - wider combo (app + services + tasks + worker + coverage): app+services now pass together; the remaining failures are a pre-existing module-level mock leak in test_data_process_service.py that exists on develop and is out of scope here. | 10 天前 | |
♻️ Refactor(data-process): replace Ray with Celery prefork (#3888) * refactor(data-process): replace Ray with Celery prefork Run parsing in process-local Celery prefork children, keep orchestration and forwarding on thread workers, and remove the Ray runtime, dependency, and deployment surface. Co-authored-by: Codex <noreply@openai.com> Generated-by: gpt-5-codex * fix(data-process): preserve sync parser result metadata * fix(data-process): wait for parser readiness in result backend * fix(data-process): tolerate lightweight worker app doubles * test(model): mock authentication for temporary healthcheck * Revert "test(model): mock authentication for temporary healthcheck" This reverts commit 9aa8d45b8eb9b8248b731957d23d851936e03608. * test(data-process): configure Celery URLs in isolated tests * test(data-process): cover prefork task boundaries * test(data-process): isolate prefork coverage from redis backend * test(data-process): cover lazy prefork branches * fix(data-process): complete celery prefork migration review fixes * test(data-process): align prefork tests with current APIs * fix(data-process): avoid logging user-controlled image paths * test(data-process): isolate router tests from celery bootstrap * test(model): provide auth for temporary healthcheck * test(data-process): stub redis chunk loader in router test * test(data-process): cover parser cancellation and lifecycle helpers * test(data-process): re-export process_sync from tasks module Keep the synchronous parse entry point reachable through the tasks module so coverage and isolation tests can patch one import site. Use a redundant-alias re-export to satisfy F401. * 🧪 Test: restore stubbed sys.modules after app router fixture Problem: the stub_modules fixture in test_data_process_app.py assigned stub modules directly to sys.modules without restoring them, so the stubs leaked into every later test file sharing the same pytest process. Running test_data_process_app.py before test_data_process_service.py failed all 52 service tests in setUp with AttributeError: '_CeleryAppStub' object has no attribute 'reset_mock'. The leak was invisible to CI because run_all_test.py runs each test file in its own subprocess. Fix: replace the six direct sys.modules assignments with monkeypatch.setitem so the real modules are restored when the fixture tears down. Behavior during each test is unchanged. Verification: - pytest test/backend/app/test_data_process_app.py test/backend/services/test_data_process_service.py -> 76 passed (was 24 passed / 52 failed) - each file alone: 24 passed / 52 passed (no regression) - wider combo (app + services + tasks + worker + coverage): app+services now pass together; the remaining failures are a pre-existing module-level mock leak in test_data_process_service.py that exists on develop and is out of scope here. | 10 天前 | |
♻️ Refactor(data-process): replace Ray with Celery prefork (#3888) * refactor(data-process): replace Ray with Celery prefork Run parsing in process-local Celery prefork children, keep orchestration and forwarding on thread workers, and remove the Ray runtime, dependency, and deployment surface. Co-authored-by: Codex <noreply@openai.com> Generated-by: gpt-5-codex * fix(data-process): preserve sync parser result metadata * fix(data-process): wait for parser readiness in result backend * fix(data-process): tolerate lightweight worker app doubles * test(model): mock authentication for temporary healthcheck * Revert "test(model): mock authentication for temporary healthcheck" This reverts commit 9aa8d45b8eb9b8248b731957d23d851936e03608. * test(data-process): configure Celery URLs in isolated tests * test(data-process): cover prefork task boundaries * test(data-process): isolate prefork coverage from redis backend * test(data-process): cover lazy prefork branches * fix(data-process): complete celery prefork migration review fixes * test(data-process): align prefork tests with current APIs * fix(data-process): avoid logging user-controlled image paths * test(data-process): isolate router tests from celery bootstrap * test(model): provide auth for temporary healthcheck * test(data-process): stub redis chunk loader in router test * test(data-process): cover parser cancellation and lifecycle helpers * test(data-process): re-export process_sync from tasks module Keep the synchronous parse entry point reachable through the tasks module so coverage and isolation tests can patch one import site. Use a redundant-alias re-export to satisfy F401. * 🧪 Test: restore stubbed sys.modules after app router fixture Problem: the stub_modules fixture in test_data_process_app.py assigned stub modules directly to sys.modules without restoring them, so the stubs leaked into every later test file sharing the same pytest process. Running test_data_process_app.py before test_data_process_service.py failed all 52 service tests in setUp with AttributeError: '_CeleryAppStub' object has no attribute 'reset_mock'. The leak was invisible to CI because run_all_test.py runs each test file in its own subprocess. Fix: replace the six direct sys.modules assignments with monkeypatch.setitem so the real modules are restored when the fixture tears down. Behavior during each test is unchanged. Verification: - pytest test/backend/app/test_data_process_app.py test/backend/services/test_data_process_service.py -> 76 passed (was 24 passed / 52 failed) - each file alone: 24 passed / 52 passed (no regression) - wider combo (app + services + tasks + worker + coverage): app+services now pass together; the remaining failures are a pre-existing module-level mock leak in test_data_process_service.py that exists on develop and is out of scope here. | 10 天前 | |
♻️ Refactor(data-process): replace Ray with Celery prefork (#3888) * refactor(data-process): replace Ray with Celery prefork Run parsing in process-local Celery prefork children, keep orchestration and forwarding on thread workers, and remove the Ray runtime, dependency, and deployment surface. Co-authored-by: Codex <noreply@openai.com> Generated-by: gpt-5-codex * fix(data-process): preserve sync parser result metadata * fix(data-process): wait for parser readiness in result backend * fix(data-process): tolerate lightweight worker app doubles * test(model): mock authentication for temporary healthcheck * Revert "test(model): mock authentication for temporary healthcheck" This reverts commit 9aa8d45b8eb9b8248b731957d23d851936e03608. * test(data-process): configure Celery URLs in isolated tests * test(data-process): cover prefork task boundaries * test(data-process): isolate prefork coverage from redis backend * test(data-process): cover lazy prefork branches * fix(data-process): complete celery prefork migration review fixes * test(data-process): align prefork tests with current APIs * fix(data-process): avoid logging user-controlled image paths * test(data-process): isolate router tests from celery bootstrap * test(model): provide auth for temporary healthcheck * test(data-process): stub redis chunk loader in router test * test(data-process): cover parser cancellation and lifecycle helpers * test(data-process): re-export process_sync from tasks module Keep the synchronous parse entry point reachable through the tasks module so coverage and isolation tests can patch one import site. Use a redundant-alias re-export to satisfy F401. * 🧪 Test: restore stubbed sys.modules after app router fixture Problem: the stub_modules fixture in test_data_process_app.py assigned stub modules directly to sys.modules without restoring them, so the stubs leaked into every later test file sharing the same pytest process. Running test_data_process_app.py before test_data_process_service.py failed all 52 service tests in setUp with AttributeError: '_CeleryAppStub' object has no attribute 'reset_mock'. The leak was invisible to CI because run_all_test.py runs each test file in its own subprocess. Fix: replace the six direct sys.modules assignments with monkeypatch.setitem so the real modules are restored when the fixture tears down. Behavior during each test is unchanged. Verification: - pytest test/backend/app/test_data_process_app.py test/backend/services/test_data_process_service.py -> 76 passed (was 24 passed / 52 failed) - each file alone: 24 passed / 52 passed (no regression) - wider combo (app + services + tasks + worker + coverage): app+services now pass together; the remaining failures are a pre-existing module-level mock leak in test_data_process_service.py that exists on develop and is out of scope here. | 10 天前 | |
♻️ Refactor(data-process): replace Ray with Celery prefork (#3888) * refactor(data-process): replace Ray with Celery prefork Run parsing in process-local Celery prefork children, keep orchestration and forwarding on thread workers, and remove the Ray runtime, dependency, and deployment surface. Co-authored-by: Codex <noreply@openai.com> Generated-by: gpt-5-codex * fix(data-process): preserve sync parser result metadata * fix(data-process): wait for parser readiness in result backend * fix(data-process): tolerate lightweight worker app doubles * test(model): mock authentication for temporary healthcheck * Revert "test(model): mock authentication for temporary healthcheck" This reverts commit 9aa8d45b8eb9b8248b731957d23d851936e03608. * test(data-process): configure Celery URLs in isolated tests * test(data-process): cover prefork task boundaries * test(data-process): isolate prefork coverage from redis backend * test(data-process): cover lazy prefork branches * fix(data-process): complete celery prefork migration review fixes * test(data-process): align prefork tests with current APIs * fix(data-process): avoid logging user-controlled image paths * test(data-process): isolate router tests from celery bootstrap * test(model): provide auth for temporary healthcheck * test(data-process): stub redis chunk loader in router test * test(data-process): cover parser cancellation and lifecycle helpers * test(data-process): re-export process_sync from tasks module Keep the synchronous parse entry point reachable through the tasks module so coverage and isolation tests can patch one import site. Use a redundant-alias re-export to satisfy F401. * 🧪 Test: restore stubbed sys.modules after app router fixture Problem: the stub_modules fixture in test_data_process_app.py assigned stub modules directly to sys.modules without restoring them, so the stubs leaked into every later test file sharing the same pytest process. Running test_data_process_app.py before test_data_process_service.py failed all 52 service tests in setUp with AttributeError: '_CeleryAppStub' object has no attribute 'reset_mock'. The leak was invisible to CI because run_all_test.py runs each test file in its own subprocess. Fix: replace the six direct sys.modules assignments with monkeypatch.setitem so the real modules are restored when the fixture tears down. Behavior during each test is unchanged. Verification: - pytest test/backend/app/test_data_process_app.py test/backend/services/test_data_process_service.py -> 76 passed (was 24 passed / 52 failed) - each file alone: 24 passed / 52 passed (no regression) - wider combo (app + services + tasks + worker + coverage): app+services now pass together; the remaining failures are a pre-existing module-level mock leak in test_data_process_service.py that exists on develop and is out of scope here. | 10 天前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 10 天前 | ||
| 10 天前 | ||
| 10 天前 | ||
| 10 天前 | ||
| 10 天前 | ||
| 10 天前 | ||
| 10 天前 |