Remove dj.kill and dj.kill_quick#1348
Merged
dimitri-yatsenko merged 2 commits intomasterfrom Jan 26, 2026
Merged
Conversation
Remove database connection management functions that don't belong in a data pipeline library. Closes #1347 Reasoning: - Outside DataJoint's scope (data pipelines, not DB administration) - Better tools exist (native SQL, CLI tools, GUIs, cloud consoles) - Implementation had issues (interactive prompts, SQL injection risk) - MySQL-only, wouldn't work with PostgreSQL backend Users can use native database tools instead: - MySQL: SHOW PROCESSLIST; KILL <pid>; - PostgreSQL: SELECT * FROM pg_stat_activity; SELECT pg_terminate_backend(pid); Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Dead code - class was defined but never instantiated anywhere. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
ttngu207
approved these changes
Jan 26, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Remove
dj.kill()anddj.kill_quick()— database administration utilities that don't belong in a data pipeline library.Closes #1347
Reasoning
Outside DataJoint's scope — DataJoint is for defining schemas and managing scientific data pipelines, not database administration
Better alternatives exist
SHOW PROCESSLIST; KILL <pid>;(MySQL)mysqladmin processlist,mysqladmin killImplementation had issues
input()prompts — unusable in scripts/automationChanges
src/datajoint/admin.pykillfrom__all__and_lazy_modulesin__init__.pytest_lazy_imports.pyMigration
Users should use native database tools:
Test plan
🤖 Generated with Claude Code