Skip to content

fix: Fix long message response error#2051

Merged
csunny merged 1 commit into
eosphoros-ai:mainfrom
noahc1510:main
Oct 16, 2024
Merged

fix: Fix long message response error#2051
csunny merged 1 commit into
eosphoros-ai:mainfrom
noahc1510:main

Conversation

@noahc1510
Copy link
Copy Markdown
Contributor

Description

When I chat with a large database, the LLM will response a large content. The pymysql will feedback this error message: Data too long for column 'message_detail' at row 1.
This PR change the type of message_detail from text to longtext.

How Has This Been Tested?

Reproduct that case I wrote before, the bug has been fix.

Snapshots:

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

@github-actions github-actions Bot added the fix Bug fixes label Sep 30, 2024
Copy link
Copy Markdown
Collaborator

@csunny csunny left a comment

Choose a reason for hiding this comment

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

@noahc1510 thank you for your contribution, it's useful.

@csunny csunny changed the title fix: Fix when get too long response, message cannot add to mysql data… fix: Fix long message response error Oct 16, 2024
Copy link
Copy Markdown
Collaborator

@yhjun1026 yhjun1026 left a comment

Choose a reason for hiding this comment

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

r+

@csunny csunny merged commit 4cde891 into eosphoros-ai:main Oct 16, 2024
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