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

docs: Add viztracer command for testing targets #1925

Merged
merged 2 commits into from
Aug 24, 2023

Conversation

mjsqu
Copy link
Contributor

@mjsqu mjsqu commented Aug 23, 2023

When it comes to testing targets it took me a long while to work out I could use the --input switch to pass a file of records to the target rather than trying to work out how to run viztracer on a command containing a pipe 🤯. If the added line can be proven to work by a reviewer and it's OK to be added to the documentation then it would have saved me heaps of time yesterday 😄


📚 Documentation preview 📚: https://meltano-sdk--1925.org.readthedocs.build/en/1925/

When it comes to testing targets it took me a long while to work out I could use the `--input` switch to pass a file of records to the target rather than trying to work out how to run `viztracer` on a command containing a pipe 🤯. If the added line can be proven to work by a reviewer and it's OK to be added to the documentation then it would have saved me heaps of time yesterday 😄
@mjsqu mjsqu requested review from a team as code owners August 23, 2023 22:45
@mjsqu mjsqu changed the title Add viztracer command for testing targets docs: Add viztracer command for testing targets Aug 23, 2023
@mjsqu
Copy link
Contributor Author

mjsqu commented Aug 23, 2023

I also had some success by restricting the output of the command using --include_files - the full command I was running:

poetry run viztracer --include_files ${HOME}/.cache/pypoetry/virtualenvs/meltanolabs-target-snowflake* -o results.html -- target-snowflake --config=ENV --input=messages.json

Copy link
Collaborator

@edgarrmondragon edgarrmondragon left a comment

Choose a reason for hiding this comment

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

If the added line can be proven to work by a reviewer

Confirmed 🙂

Thanks @mjsqu!

@edgarrmondragon edgarrmondragon merged commit f9c2dad into meltano:main Aug 24, 2023
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