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

Update API / FE Catalog Model #1491

Closed
cgardens opened this issue Jan 4, 2021 · 4 comments
Closed

Update API / FE Catalog Model #1491

cgardens opened this issue Jan 4, 2021 · 4 comments
Assignees
@ChristopheDuong @cgardens
Labels
two-third type/enhancement New feature or request
Milestone
2021-02-19

Comments

@cgardens
Copy link
Contributor

cgardens commented Jan 4, 2021

Tell us about the problem you're trying to solve

  • The API / FE assume that the data in catalogs is structured as table / column (see StandardDataSchema). The BE data model supports any schema that can be described as JsonSchema (see AirbyteCatalog).
  • When we convert between the two we lose information.

Describe the solution you’d like

  • The API / FE should should support handling any schema the backend can handle.
@cgardens cgardens added the type/enhancement New feature or request label Jan 4, 2021
@cgardens cgardens modified the milestone: v0.10.0 Jan 4, 2021
@sherifnada sherifnada mentioned this issue Jan 29, 2021
Closed
@cgardens cgardens added this to the 2021/02/05 milestone Feb 1, 2021
@cgardens
Copy link
Contributor Author

cgardens commented Feb 1, 2021

I am going to work through getting the interface defined with artem, and then chris will take over after that.

#1924

@cgardens cgardens mentioned this issue Feb 2, 2021
Closed
@cgardens
Copy link
Contributor Author

cgardens commented Feb 8, 2021

This is code complete and waiting for work to be done on the FE.

@cgardens cgardens modified the milestones: 2021/02/05, 20201/02/12 Feb 8, 2021
@staufman staufman mentioned this issue Feb 10, 2021
Closed
@cgardens
Copy link
Contributor Author

will be merged on monday.

@cgardens cgardens modified the milestones: 20201/02/12, 2021/02/19 Feb 15, 2021
@cgardens
Copy link
Contributor Author

closed by #1934

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ChristopheDuong ChristopheDuong

@cgardens cgardens

Labels
two-third type/enhancement New feature or request
Projects
None yet
Milestone
2021-02-19
Development

No branches or pull requests
2 participants
@ChristopheDuong @cgardens