-
Notifications
You must be signed in to change notification settings - Fork 2.8k
support computed columns (close #1378) #1935
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
Conversation
|
Deploy preview for hasura-docs ready! Built with commit e4e574b |
|
just wondering if this PR is still alive, waiting to use this feature. the design looks good to me. thanks a lot for putting the effort into the feature |
|
checking-in on this feature, hoping to see it merged soon. thanks! |
|
Hey @chunsli, this should be out in a couple of releases. Some of the underlying code has changed significantly, so a merge and a review are pending. cc: @rakeshkky @0x777 |
|
following up again, is this still going to be merged? @rakeshkky @dsandip @0x777 thanks for the hard work |
|
I would like to echo that this would be an amazingly useful feature |
|
closing in favour of #2917 |
|
Review app https://hge-ci-pull-1935.herokuapp.com is deleted |
Description
Affected components
Related Issues
close #1378
Design
Allow support for computed columns.
-> Add SQL functions through
/v1/queryAPI of typetrack_function.-> Server will automatically determine the table to which the SQL function to be added as a computed column based on the first input parameter.
->
create_select_permissionquery type of/v1/queryis updated to have an optional fieldcomputed_columnsto permit SQL functions to be queried. An Example given in the next section.Internal changes:-
hdb_function_aggview to have qualified Postgres types for input arguments of a SQL functionhdb_computed_columnto keep track of SQL function and table to which computed column is defined. Defined an array relationship fromhdb_tablewith namecomputed_columns. This helps console to fetch list of computed columns to all tracked tables.13withSQLandMetdatamigrationSteps to test and verify
reponse:-
{ "data": { "person": [ { "id": 1, "first_name": "taylor", "last_name": "chris", "full_name": "chris taylor", "full_name_prefix": "Mr. chris taylor" }, { "id": 2, "first_name": "clarke", "last_name": "michael", "full_name": "michael clarke", "full_name_prefix": "Mr. michael clarke" } ] } }Limitations, known bugs & workarounds
BASICtype are allowed to be tracked as a computed column.cascadefield tountrack_functionquery type which will be a breaking change.