Skip to content

Fork rebase - #657

Draft
mgrunbauer wants to merge 7 commits into
masterfrom
fork-rebase
Draft

Fork rebase#657
mgrunbauer wants to merge 7 commits into
masterfrom
fork-rebase

Conversation

@mgrunbauer

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread python/python_fastapi_server/fork_server_supervisor.py
Comment thread adagucserverEC/fork_server.cpp
Comment thread adagucserverEC/fork_server.cpp
Comment thread adagucserverEC/fork_server.cpp Outdated
Comment thread adagucserverEC/fork_server.cpp
Comment thread adagucserverEC/fork_server.cpp Outdated
Comment thread adagucserverEC/fork_server.cpp Outdated
Comment thread adagucserverEC/fork_server.cpp Outdated
Comment thread adagucserverEC/fork_server.cpp
Comment thread adagucserverEC/fork_server.cpp Outdated
Comment thread adagucserverEC/fork_server.cpp Outdated
Comment thread adagucserverEC/fork_server.h Outdated
Comment thread adagucserverEC/fork_server.h Outdated
Comment thread doc/Environment_Variables.md Outdated
time_t now = time(NULL);
for (auto it = child_procs.begin(); it != child_procs.end(); ++it) {
if (difftime(now, it->second.forked_at) > MAX_CHILD_PROC_TIMEOUT) {
kill(it->first, SIGKILL);

@mgrunbauer mgrunbauer Mar 4, 2026

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We kill process with SIGKILL, but we should send process timeout status code 34 to python.

Comment thread adagucserverEC/fork_server.cpp Outdated
Comment thread adagucserverEC/fork_server.cpp
Comment thread adagucserverEC/fork_server.cpp Outdated
Comment thread python/lib/adaguc/CGIRunner.py Outdated
Comment thread python/lib/adaguc/CGIRunner.py Outdated
Comment thread python/python_fastapi_server/fork_server_supervisor.py Outdated
Comment thread adagucserverEC/fork_server.cpp Outdated
@mgrunbauer mgrunbauer mentioned this pull request Mar 5, 2026
7 tasks
@mgrunbauer
mgrunbauer force-pushed the fork-rebase branch 4 times, most recently from f839ed7 to 474742f Compare April 20, 2026 15:50
@mgrunbauer
mgrunbauer force-pushed the fork-rebase branch 2 times, most recently from ac95870 to 02c5d0e Compare July 7, 2026 19:14
Comment thread adagucserverEC/fork_server.cpp Outdated

if __name__ == "__main__":
testadaguc()
# testadaguc()

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is this disabled

self.env = os.environ.copy()
adaguc_env = runAdaguc().getAdagucEnv()
self.env.update({k: str(v) for k, v in adaguc_env.items()})
self.env["ADAGUC_CONFIG"] = f"{self.env.get('ADAGUC_PATH')}/python/lib/adaguc/adaguc-server-config-python-postgres.xml"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should this not be ADAGUC_CONFIG?

Comment thread python/lib/adaguc/CGIRunner.py Outdated
Comment thread adagucserverEC/fork_server.cpp Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants