Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug][Github: Extract Users] avatar_url data too long error #8250

Open
3 tasks done
krishnaku opened this issue Dec 20, 2024 · 1 comment
Open
3 tasks done

[Bug][Github: Extract Users] avatar_url data too long error #8250

krishnaku opened this issue Dec 20, 2024 · 1 comment
Labels
component/plugins This issue or PR relates to plugins severity/p1 This bug affects functionality or significantly affect ux type/bug This issue is a bug

Comments

@krishnaku
Copy link

Search before asking

  • I had searched in the issues and found no similar issues.

What happened

DevLake: v1.0.1@e061ef2

I ran into an import error while trying to process the Github repo for Devlake.

Tracked it down to the loading of the table _tool_github_accounts from _raw_github_account

I resolved it by changing the type of the avatar_url column from VARCHAR(255) to TEXT.
Repeated the same for table accounts.

I was able to work around this issue with this fix, but reporting anyway.

logging (3).tar.gz
Screen Shot 2024-12-19 at 8 39 43 PM

What do you expect to happen

Script should run without errors.

How to reproduce

  1. Create a Github connection.
  2. Connect to apache/incubator-devlake project.
  3. Set up project with default settings with this connection.
  4. Collect data.

Anything else

No response

Version

v1.0.1@e061ef2

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@krishnaku krishnaku added the type/bug This issue is a bug label Dec 20, 2024
Copy link

dosubot bot commented Dec 20, 2024

Hey @krishnaku! 👋 I'm here to help you with any bugs, questions, or contributions. I'll assist you while we wait for a human maintainer.

The issue you're encountering is due to the avatar_url field being defined as VARCHAR(255), which is causing a "data too long" error when the URL exceeds this length. This field is indeed defined with a maximum length of 255 characters in the codebase [1].

You've resolved the issue by changing the column type to TEXT, which allows for longer URLs. This is a practical workaround, but it might be worth considering if this change should be made in the main codebase to prevent similar issues for other users.

If you're willing to submit a pull request, you could propose this change to the repository maintainers. It would be helpful to include your findings and the rationale for changing the column type to TEXT to accommodate longer URLs.

To continue talking to Dosu, mention @dosu.


Help Dosu learn! Give it feedback: Great Response | Irrelevant Answer | Incorrect Sources | Too Verbose | Hallucination | Other

@dosubot dosubot bot added component/plugins This issue or PR relates to plugins severity/p1 This bug affects functionality or significantly affect ux labels Dec 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/plugins This issue or PR relates to plugins severity/p1 This bug affects functionality or significantly affect ux type/bug This issue is a bug
Projects
None yet
Development

No branches or pull requests

1 participant