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

sql: remove InternalExecutor from EvalCtx #71246

Merged

Commits on Nov 24, 2021

  1. sql: refactor InternalExecutor on EvalCtx

    Minor refactor to remove duplicate InternalExecutor definition
    on EvalCtx and in the tree package.
    
    We already have 2 interface declarations for InternalExecutor
    in sql and sqlutil, this refactor hopefully makes the dependencies
    a little more clear.
    
    For builtins, we now use an InternalExecutorFactory where
    we must pass in the txn, session data and desc.Collections
    to use the IE.
    
    Release note: None
    RichardJCai committed Nov 24, 2021
    Configuration menu
    Copy the full SHA
    203c7f4 View commit details
    Browse the repository at this point in the history