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

Glue View L2 Construct #463

Closed
1 of 11 tasks
JeromeGuyon opened this issue Nov 4, 2022 · 1 comment
Closed
1 of 11 tasks

Glue View L2 Construct #463

JeromeGuyon opened this issue Nov 4, 2022 · 1 comment
Labels
l2-request request for new L2 construct management/tracking status/stale The RFC did not get any significant enough progress or tracking and has become stale.

Comments

@JeromeGuyon
Copy link

JeromeGuyon commented Nov 4, 2022

Description

Supports Views as a L2 construct in aws-glue cdk library

It allows to create Presto Views directly from the SQL Statement.

With :

  • that columns that must be a subset of declared columns in SQL Statement, but no sush checks. No checks on column typing consistency with SQL Statement.
  • No checks on the SQL Statement.

Example of usage :

declare const myDatabase: glue.Database;

new glue.View(this, 'MyView', {
  database: myDatabase,
    tableName: 'my_view',
    columns: [{
      name: 'col',
      type: glue.Schema.STRING,
    }],
    sql: 'select "1" as col',
  });

The Glue View is a special Glue Table, with readonly access on data and should share the maximum of attributes/members for Table L2 Construct.

Draft PR:

Draft PR : aws/aws-cdk#22459

Roles

Role User
Proposed by @JeromeGuyon
Author(s) @JeromeGuyon
API Bar Raiser
Stakeholders

See RFC Process for details

Workflow

  • Tracking issue created (label: status/proposed)
  • API bar raiser assigned (ping us at #aws-cdk-rfcs if needed)
  • Kick off meeting
  • RFC pull request submitted (label: status/review)
  • Community reach out (via Slack and/or Twitter)
  • API signed-off (label api-approved applied to pull request)
  • Final comments period (label: status/final-comments-period)
  • Approved and merged (label: status/approved)
  • Execution plan submitted (label: status/planning)
  • Plan approved and merged (label: status/implementing)
  • Implementation complete (label: status/done)

Author is responsible to progress the RFC according to this checklist, and
apply the relevant labels to this issue so that the RFC table in README gets
updated.

@JeromeGuyon JeromeGuyon changed the title Glue View L2 Glue View L2 Construct Nov 7, 2022
@evgenyka evgenyka added l2-request request for new L2 construct bar-raiser/needed labels Aug 10, 2023
@mrgrain
Copy link
Contributor

mrgrain commented Nov 1, 2023

Marking this RFCs as stale since there has been little recent activity and it is not currently close to getting accepted as-is. We appreciate the effort that has gone into this proposal. Marking an RFCs as stale is not a one-way door. If you have made substantial changes to the proposal, please open a new issue/RFC. You might also consider raising a PR to aws/aws-cdk directly or self-publishing to Construct Hub.

@mrgrain mrgrain closed this as not planned Won't fix, can't repro, duplicate, stale Nov 1, 2023
@mrgrain mrgrain added status/stale The RFC did not get any significant enough progress or tracking and has become stale. and removed status/proposed Newly proposed RFC bar-raiser/needed labels Nov 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
l2-request request for new L2 construct management/tracking status/stale The RFC did not get any significant enough progress or tracking and has become stale.
Projects
None yet
Development

No branches or pull requests

3 participants