Jesse Rosenberger
Jesse Rosenberger
As an update, we are **still** waiting on an upstream dependency (one which depends on `opentelemetry-rust`, itself) to actually move this along — https://github.com/tokio-rs/tracing-opentelemetry/pull/9.
This is great! Given my comments on #43, do you know which of these makes the more substantial improvement? I'm pretty sure we could make _tremendous_ improvements by switching to...
Some notes: - The aws deps are all actually at 1.0 now, but all the actual breaking changes seemed to happen in pre-1.0 versions. - The `serde_yaml` does some major...
As to context about this new mechanism, we've introduced https://github.com/apollographql/router/pull/5031 and https://github.com/apollographql/router/pull/5030 and this is the first output from that.
Tests will fail on this PR until we actually apply what @bonnici tried to apply in https://github.com/apollographql/router/pull/5026 but for whatever reason haven't been able to pass because of a failing...
We **may** have only cached it previously because it was going through the `router-bridge`.
@nicholascioli Did you possibly fix this recently?
The `--debug` flag is only relevant during development with the `meteor` command. The built bundle (for production, which is a Node.js app, and no longer aware of the `meteor` command),...
Sorry, I may have closed this prematurely, and it seems that `--debug` on `meteor build` probably does something different than what I was thinking you were trying to do. Are...
@smyrick That test isn't required. It's the other failing tests that are problematic. Will look soon.