Skip to content

feat(python-notebook-migration): add database tables for Notebook Migration tool#5055

Open
zyratlo wants to merge 6 commits into
apache:mainfrom
zyratlo:migration-tool-database-tables
Open

feat(python-notebook-migration): add database tables for Notebook Migration tool#5055
zyratlo wants to merge 6 commits into
apache:mainfrom
zyratlo:migration-tool-database-tables

Conversation

@zyratlo
Copy link
Copy Markdown
Contributor

@zyratlo zyratlo commented May 13, 2026

What changes were proposed in this PR?

This PR adds two new tables to the database, notebook and workflow_notebook_mapping. These tables will be used in the new Python Notebook Migration tool to store the user-uploaded notebook and the generated mapping between the notebook and workflow.

Any related issues, documentation, discussions?

Closes #5054
The parent issue is #4301

Schema

image

How was this PR tested?

New tables were manually confirmed to be created successfully and usable.

Was this PR authored or co-authored using generative AI tooling?

No

@github-actions github-actions Bot added the ddl-change Changes to the TexeraDB DDL label May 13, 2026
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 43.14%. Comparing base (538ae1d) to head (4387ad0).

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #5055      +/-   ##
============================================
- Coverage     43.17%   43.14%   -0.03%     
+ Complexity     2213     2211       -2     
============================================
  Files          1045     1045              
  Lines         40363    40355       -8     
  Branches       4268     4268              
============================================
- Hits          17425    17413      -12     
  Misses        21865    21865              
- Partials       1073     1077       +4     
Flag Coverage Δ *Carryforward flag
access-control-service 39.53% <ø> (ø)
agent-service 33.72% <ø> (ø) Carriedforward from 13c4836
amber 43.79% <ø> (-0.02%) ⬇️
computing-unit-managing-service 0.00% <ø> (ø)
config-service 0.00% <ø> (ø)
file-service 32.18% <ø> (ø)
frontend 34.07% <ø> (ø) Carriedforward from 13c4836
python 90.48% <ø> (-0.02%) ⬇️ Carriedforward from 13c4836
workflow-compiling-service 56.81% <ø> (ø)

*This pull request uses carry forward flags. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@mengw15 mengw15 self-requested a review May 13, 2026 20:47
@mengw15
Copy link
Copy Markdown
Contributor

mengw15 commented May 18, 2026

Please check this pr: #4401, for database changes

@zyratlo
Copy link
Copy Markdown
Contributor Author

zyratlo commented May 18, 2026

Please check this pr: #4401, for database changes

I have addressed this issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ddl-change Changes to the TexeraDB DDL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Notebook Migration] 1. Add Database Tables

3 participants