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.

Sign up for GitHub

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

Jump to bottom

pkg: Add support for user specified target cols in IMPORT INTO #39023

Merged
merged 3 commits into from
Jul 29, 2019
Merged

pkg: Add support for user specified target cols in IMPORT INTO #39023

merged 3 commits into from
Jul 29, 2019

Commits on Jul 29, 2019

  1. distsqlpb: Adds target cols to the IMPORT job and processor protos 

    This change adds the proto definitions required to
plumb information about the target columns to be imported into,
through the import job and ReadImport processor.

Release note: None
    @adityamaru27
    adityamaru27 committed Jul 29, 2019
    Configuration menu
    Copy the full SHA
    08b9469 View commit details
    Browse the repository at this point in the history

  2. sql: Plumbing target column logic for IMPORT INTO 

    This change adds the ability for users to specify a subset
of the columns of an existing table to import into, from a
CSV data source.

TODO: Add the grammar which supports IMPORT INTO without any
target columns specified, in which case we import data for all
columns.

Release note: None
    @adityamaru27
    adityamaru27 committed Jul 29, 2019
    Configuration menu
    Copy the full SHA
    796a57e View commit details
    Browse the repository at this point in the history

  3. importccl: Added tests for IMPORT INTO with user specified target cols. 

    This change adds unit tests for importing into an existing table while
specifying a subset of all visible columns as targets.

Release note: None
    @adityamaru27
    adityamaru27 committed Jul 29, 2019
    Configuration menu
    Copy the full SHA
    91aa7c4 View commit details
    Browse the repository at this point in the history