incubator-devlake icon indicating copy to clipboard operation
incubator-devlake copied to clipboard

[Bug][github] some /pulls/comments of repo get servers error

Open mappjzc opened this issue 3 years ago • 3 comments

Search before asking

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

What happened

image

image image image image

the api request /pulls/comments of apache/incubator-devlake can get the data but the same api for trinodb/trino get the error

It can be succees when we set the since image But if we use long time ago to set the since like then years ago, it will also sent the fail. image

What you expected to happen

Collect data correctly

How to reproduce

collect trinodb/trino suceess.

Anything else

No response

Version

main

Are you willing to submit PR?

  • [ ] Yes I am willing to submit a PR!

Code of Conduct

mappjzc avatar Sep 15 '22 13:09 mappjzc

@keon94 In config ui, we may need more friendly error messages.

mappjzc avatar Sep 16 '22 02:09 mappjzc

Hi, @keon94 , seems like the trino project has some corrupted data in it. We don't have a plan to accommodate the problem at this point. But the message we are seeing doesn't seem friendly, I think you might be interested in taking a look at the issue.

klesh avatar Sep 19 '22 07:09 klesh

@mappjzc @klesh That UI error message you're seeing is the format from before we merged #2940. Could you try again on newest code? It shouldn't look like that anymore. Also, you might want to open your browser inspector to see the error json that comes back because config-ui hasn't adapted the changes yet

keon94 avatar Sep 19 '22 07:09 keon94