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

Internal error: merge job timed out after xxx seconds #5831

Closed
bokuweb opened this issue Feb 17, 2018 · 14 comments
Closed

Internal error: merge job timed out after xxx seconds #5831

bokuweb opened this issue Feb 17, 2018 · 14 comments

Comments

@bokuweb
Copy link

bokuweb commented Feb 17, 2018

2018-02-17 12 43 41

Internal error: merge job timed out after xxx seconds error occurred. But other error looks correct.
This error appears v0.63.0~, but v0.62.0 works fine in my project (I can not provide this project because it is private...). I can not know how to reproduce this error.

Can I set merge timeout value to avoid timeout?

@osmestad
Copy link

We had the same issue, you can set 'merge_timeout' either in your .flowconfig or as a command line flag. That helped for us :-)

@bokuweb
Copy link
Author

bokuweb commented Mar 14, 2018

@osmestad Thanks :) I'll try it!

@IanLondon
Copy link

@bokuweb did setting a higher timeout fix the problem for you? I'm having the same issue.

@bokuweb
Copy link
Author

bokuweb commented Apr 4, 2018

@osmestad Hi :). I increased timeout, but it did not solve it..

@IanLondon
Copy link

For me, I found out the problem was that flow was getting stuck inferring the type of an accumulator in a reduce operation. It took a long time using trial & error to narrow it down, but once I added a type annotation to the accumulator in the reduce callback, the merge operation completed in a reasonable time. Hope that helps.

@osmestad
Copy link

osmestad commented Apr 5, 2018

For us I suspect the types may have had a circular dependency, but I have not tried to reduce a test case of it (simplifying the code some seemed to help the speed, was only happening on the build server for us so painful to isolate). Looking at the Flow logs might also be useful, as that gives a bit more info than the console output :-)

@rostislav-simonik
Copy link
Contributor

I've the same problem. Is there any profiling option that I can use to narrow area of issue. I've large project and it took 34 seconds before. Now it takes ~ 1500 seconds.

@frontendphil
Copy link

I'm usually not a fan of "me too" posts, but well me too! Today the flow check stopped working for no apparent reason. I've already added type definitions to the recursive methods I've found but that didn't help. If I use flow check-contents it works but takes forever... I'm out of ideas on how I can fix that.

@kogai
Copy link

kogai commented Nov 12, 2018

The same error occurred at 0.86 👀

I don't know it's worth or not, server log of Flow like below.

(8650) merge_strict_job THROWS: [35] `A file path which Flow reported to merge job timeout occurred`
      `blah, blah, blah...`
Flow_error.EMergeTimeout(105.011678934)
Raised by primitive operation at file "src/typing/flow_js.ml", line 11123, characters 21-55
Called from file "src/typing/flow_js.ml", line 8504, characters 4-28

@kogai
Copy link

kogai commented Dec 3, 2018

Still causing at 0.87 👀
I haven't idea whether it causes by a bug of Flow or any mistake in my code 🤔

@Maggie199
Copy link

Still causing at 0.93

@paperkotter
Copy link

Reinstalling packages helped me in my case

@ericsoco
Copy link

One workaround is to add the offending path/file to [untyped] in .flowconfig, particularly if the issue is coming from within node_modules (otherwise, if in your source, you should try to fix the error!).

[untyped]
.*/node_modules/package-with-merge-timeout/filename.js

You can also add merge_timeout=N under [options] to make testing the fix a bit quicker. Set N to a value that is long enough that flow should complete without the the merge job timeout, but short enough you're not waiting forever. YMMV depending on size of your codebase.

@SamChou19815
Copy link
Contributor

There is not enough information to repro, and the issue is likely fixed a while ago.

@SamChou19815 SamChou19815 closed this as not planned Won't fix, can't repro, duplicate, stale Feb 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests