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

feat(supergraph): add --output flag #561

Merged
merged 1 commit into from
Aug 12, 2024

Conversation

aaronArinder
Copy link
Contributor

useful for big, big composition results; spitting those out to stdout can take a non-trivial amount of time, especially if they're using --output on the rover side (meaning stdout is useless apart as a way of communicating the composition result, which can be done faster with this approach)

let me know (via slack) if you'd like more details! this is part of trying to figure out a particular customer's pain

@aaronArinder aaronArinder requested review from a team as code owners August 9, 2024 19:16
  - useful for big, big composition results; spitting those out to
    stdout can take a non-trivial amount of time, especially if they're
    using `--output` on the rover side (meaning stdout is useless apart
    as a way of communicating the composition result, which can be done
    faster with this approach)
@aaronArinder aaronArinder force-pushed the aaron/add-output-flag-to-supergraph-compose branch from 6462774 to 323acac Compare August 12, 2024 14:18
@aaronArinder aaronArinder changed the title fix(supergraph): add --output flag feat(supergraph): add --output flag Aug 12, 2024
@aaronArinder aaronArinder merged commit 28947fc into main Aug 12, 2024
12 checks passed
@aaronArinder aaronArinder deleted the aaron/add-output-flag-to-supergraph-compose branch August 12, 2024 14:46
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