Skip to content

Chore: log rendered jinja right before parsing it to improve debugging#5043

Merged
georgesittas merged 1 commit intomainfrom
jo/log_rendered_jinja_before_parsing
Jul 28, 2025
Merged

Chore: log rendered jinja right before parsing it to improve debugging#5043
georgesittas merged 1 commit intomainfrom
jo/log_rendered_jinja_before_parsing

Conversation

@georgesittas
Copy link
Contributor

There have been quite a few occasions in the wild where the parser fails to process a rendered Jinja string, while at the same time displaying a limited query context in the exception, making debugging difficult. This PR debug-logs the rendered Jinja string right before it is parsed.

@georgesittas georgesittas requested review from a team and izeigerman July 28, 2025 11:38
@georgesittas georgesittas force-pushed the jo/log_rendered_jinja_before_parsing branch from c3a4b7c to a2a7bf4 Compare July 28, 2025 11:41
Copy link
Contributor

@benfdking benfdking left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@georgesittas georgesittas force-pushed the jo/log_rendered_jinja_before_parsing branch from a2a7bf4 to e4acb3d Compare July 28, 2025 11:52
@georgesittas georgesittas merged commit 57d2e9f into main Jul 28, 2025
27 checks passed
@georgesittas georgesittas deleted the jo/log_rendered_jinja_before_parsing branch July 28, 2025 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants