Skip to content

fix(schema): change flow_data column type from text to longtext#2590

Merged
Aries-ckt merged 1 commit into
eosphoros-ai:mainfrom
vnicers:fix_db_column_type
Apr 9, 2025
Merged

fix(schema): change flow_data column type from text to longtext#2590
Aries-ckt merged 1 commit into
eosphoros-ai:mainfrom
vnicers:fix_db_column_type

Conversation

@vnicers
Copy link
Copy Markdown
Contributor

@vnicers vnicers commented Apr 7, 2025

  • Update flow_data column type in dbgpt_serve_flow table from text to longtext
  • Modify flow_data column type in v0.7.0.sql and upgrade_to_v0.7.1.sql

Description

When there are too many workflow nodes, saving the workflow will error flow_data data fields are too long

img_v3_02l4_16276396-30ad-4d5d-aa7e-7ae6116fd35p

How Has This Been Tested?

Just change the flow_data field type text to longtext

Snapshots:

N/A

Checklist:

  • My code follows the style guidelines of this project
  • I have already rebased the commits and make the commit message conform to the project standard.
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • Any dependent changes have been merged and published in downstream modules

- Update flow_data column type in dbgpt_serve_flow table from text to longtext
- Modify flow_data column type in v0.7.0.sql and upgrade_to_v0.7.1.sql
@github-actions github-actions Bot added the fix Bug fixes label Apr 7, 2025
@Aries-ckt
Copy link
Copy Markdown
Collaborator

@fangyinc please take a look.

Copy link
Copy Markdown
Member

@fangyinc fangyinc left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Copy Markdown
Collaborator

@Aries-ckt Aries-ckt left a comment

Choose a reason for hiding this comment

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

LGTM.thanks for your contribution

@Aries-ckt Aries-ckt merged commit ae4112c into eosphoros-ai:main Apr 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants