-
Notifications
You must be signed in to change notification settings - Fork 251
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
feat(wren-ui): Detects schema change - table & column #339
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
wren-ui/src/apollo/server/repositories/schemaChangeRepository.ts
Outdated
Show resolved
Hide resolved
andreashimin
force-pushed
the
feature/schema-changes
branch
from
June 11, 2024 11:22
d0a790c
to
3a8947a
Compare
* feat: Add ibis adaptor for querying supported data sources * feat: Add testIbis resolver for querying supported data sources * feat: use queryService for preview and describe statement * feat: Refactor ibisAdaptor to use DataSourceName enum consistently * test ibis using tpch query * rm test ibis * feat: Add previewSql resolver for SQL preview * rm debug log * use projectId and manifest to generate hash Implement getMDL api * pass mdl to ibis server to preview * transform manifest data type to json object when retrieved from database * better error message from Ibis server, rewrite host when development and using docker * use base64 str to transfer mdl * update README * change env name to fit naming style. update docker-compose.x.yaml and .env.example file * fix yarn build issue * update docker-compose file: fix incorrent ibis server endpoint * port forward to ibis server * fix incorrect migration rollback function * update engine version * Chore: refactor database schema to reflect how we use them. (#341) * add connectionInfo column and do data migration * drop deprecated columns that store connection info, use connectionInfo column instead * do data migration to store tableReference in model's properties * fix incorrect migration rollback function * use tableReference in mdl instead of refSql * chore: Update variable names in ibisAdaptor and queryService * chore: Update variable names in ibisAdaptor and queryService * fix lint * rename variable in ibis adaptor * - remove duplicate logic to parse string - add comment
…dd-in used (#373) * fix(wren-ui): add displayName to all view API which share ViewInfo type in graphQL * feat(wren-ui): provide limit property of preview API for usage in excel-add-in
* chore: Update ibisAdaptor to include methods for getting tables and constraints * remove strategies and connectors from codebase, use ibis server and wrenEngine to fetch metadata of data source * add validation rule api in ibisAdapter * replace invalid char when generating referenceName * fix comments
andreashimin
force-pushed
the
feature/schema-changes
branch
from
June 12, 2024 11:00
34ae502
to
6ab9650
Compare
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description