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

chore(sources): Add SourceContext #4685

Closed
wants to merge 2 commits into from
Closed

Commits on Oct 21, 2020

  1. chore(sources): Add SourceContext

    To inject the `Resolver` similar to the `TransformContext`. I need the
    `Resolver` in the new `aws_s3` source. I figured I'd submit this change
    separately since it was easy to pull out and I wanted thoughts sooner
    than later.
    
    Alternatively, it seems like the resolver is static now so we could drop
    it from `SinkContext` and `TransformContext`. I wanted some confirmation
    on that front though. I'm happy to reverse this and remove injection of
    the `Resolver`.
    
    Signed-off-by: Jesse Szwedko <jesse@szwedko.me>
    jszwedko committed Oct 21, 2020
    Configuration menu
    Copy the full SHA
    e9a701f View commit details
    Browse the repository at this point in the history

Commits on Oct 22, 2020

  1. Add SourceContext to tests

    Signed-off-by: Jesse Szwedko <jesse@szwedko.me>
    jszwedko committed Oct 22, 2020
    Configuration menu
    Copy the full SHA
    ed7ebfb View commit details
    Browse the repository at this point in the history