Skip to content

Probe component #251

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

Closed
wants to merge 2 commits into from
Closed

Conversation

olivierauverlot
Copy link
Contributor

A debug component that displays a value (constant, variable, parameter value, result of an SQL function, e.g.). Usefull to validate data flow or function behaviour. Each value (except NULL value) is surrounded by double quotes and followed by datatype (number, string, boolean, e.g.).

@lovasoa
Copy link
Collaborator

lovasoa commented Mar 4, 2024

Is there something that you don't like in the existing debug component?

@olivierauverlot
Copy link
Contributor Author

Not really the same finality. The debug component is practical to display and control parameters passed to a component. The Probe component is more generic and her work is just to display a value with additional informations (name, datatype, string content, NULL value, etc.). It could be used to print a HTTP parameter from request, the content of a variable, the value of an environment variable, the result of a fonction, etc.

@lovasoa
Copy link
Collaborator

lovasoa commented Mar 4, 2024

I'm not sure I see the difference with just

select 'debug' as component, my_value

where my_value can be an HTTP parameter from request, the contents of a variable, the value of an environment variable, the result of a function, etc.

This dumps the value as json, so you can also see its type (including whether it is null).

Maybe the main issue is in the documentation of the debug component that isn't clear enough and doesn't give enough examples?

@olivierauverlot
Copy link
Contributor Author

Yes, it may be necessary to modify the documentation for the debug component. The first sentence is too restrictive: "Display all the parameters passed to the component".

@lovasoa lovasoa closed this in c0d4b82 Mar 5, 2024
lovasoa added a commit that referenced this pull request Mar 5, 2024
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