-
Notifications
You must be signed in to change notification settings - Fork 302
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
evalv3: stack overflow with matchN
#3443
Comments
Note: this issue also applies to
|
matchN
matchN
FTR: fix upcoming for the original problem, but this will still fail:
so leaving issue open. |
cueckoo
pushed a commit
that referenced
this issue
Oct 8, 2024
This is needed to pass various tests. Bugs still remains, but the main point is that it unblocks a next phase in structure sharing. Issue #2854 Issue #2850 Issue #3165 Fixes #3410 Fixes #3420 Issue #3443 Signed-off-by: Marcel van Lohuizen <[email protected]> Change-Id: Ibd2a41e25e07bd37899620af6bd9665435d68e8a Reviewed-on: https://review.gerrithub.io/c/cue-lang/cue/+/1202212 TryBot-Result: CUEcueckoo <[email protected]> Reviewed-by: Matthew Sackman <[email protected]> Unity-Result: CUE porcuepine <[email protected]>
vanhtuan0409
pushed a commit
to anduintransaction/cue
that referenced
this issue
Oct 15, 2024
This is needed to pass various tests. Bugs still remains, but the main point is that it unblocks a next phase in structure sharing. Issue cue-lang#2854 Issue cue-lang#2850 Issue cue-lang#3165 Fixes cue-lang#3410 Fixes cue-lang#3420 Issue cue-lang#3443 Signed-off-by: Marcel van Lohuizen <[email protected]> Change-Id: Ibd2a41e25e07bd37899620af6bd9665435d68e8a Reviewed-on: https://review.gerrithub.io/c/cue-lang/cue/+/1202212 TryBot-Result: CUEcueckoo <[email protected]> Reviewed-by: Matthew Sackman <[email protected]> Unity-Result: CUE porcuepine <[email protected]>
vanhtuan0409
pushed a commit
to anduintransaction/cue
that referenced
this issue
Oct 15, 2024
The key change here is in context.go, where we now also pass the CloseInfo asociated with the validator. This ensures that cycles are tracked across validator calls. To accomplish this, we need to record the CloseInfo for validators as well. For completeness sake, we also track the Environment, which may give us some additional contextual information that we can use in error messages down the line. This avoids a hang in cue-lang#3443. It still is a bit sloppy about whether something should be considered a cycle or not. One could argue either way. Fixes cue-lang#3443 Signed-off-by: Marcel van Lohuizen <[email protected]> Change-Id: Id6c7f2ae48d07586e969af2353bb34b23771f977 Reviewed-on: https://review.gerrithub.io/c/cue-lang/cue/+/1202289 TryBot-Result: CUEcueckoo <[email protected]> Unity-Result: CUE porcuepine <[email protected]> Reviewed-by: Matthew Sackman <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What version of CUE are you using (
cue version
)?Does this issue reproduce with the latest stable release?
N/A
What did you do?
What did you expect to see?
Passing test.
What did you see instead?
The text was updated successfully, but these errors were encountered: