Skip to content

Increase maximum queue size to 20_000#12218

Closed
tomquist wants to merge 1 commit into
babel:mainfrom
tomquist:fix-infinite-cycle-detected
Closed

Increase maximum queue size to 20_000#12218
tomquist wants to merge 1 commit into
babel:mainfrom
tomquist:fix-infinite-cycle-detected

Conversation

@tomquist

@tomquist tomquist commented Oct 19, 2020

Copy link
Copy Markdown

Due to our tests we hit the "Infinite cycle detected" error on seemingly random changes. It seems to be related to the queue size exceeding a hard-coded limit. Not sure if 20000 is a good limit but 10000 already seemed to be kind of arbitrary.

Unfortunately we hit the  maximum queue size
with our code base even though there’s no infinite
cycle.
@babel-bot

Copy link
Copy Markdown
Collaborator

Build successful! You can test your changes in the REPL here: https://babeljs.io/repl/build/30509/

@codesandbox-ci

Copy link
Copy Markdown

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 1991c3e:

Sandbox Source
babel-repl-custom-plugin Configuration
babel-plugin-multi-config Configuration

@JLHwung

JLHwung commented Oct 19, 2020

Copy link
Copy Markdown
Contributor

Can you share a reproduction? I doubt that it is an internal Babel bugs.

@nicolo-ribaudo

Copy link
Copy Markdown
Member

Closing since you didn't reply to #12218 (comment), so I assume it's ok now!

@github-actions github-actions Bot added the outdated A closed issue/PR that is archived due to age. Recommended to make a new issue label May 27, 2021
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators May 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

outdated A closed issue/PR that is archived due to age. Recommended to make a new issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants