Updating version of msprod #969

VardhanThigle · 2024-12-19T18:54:31Z

Fixes b/85117347

It's a good idea to open an issue first for discussion.

Tests pass
Appropriate changes to README are included in PR - N/A

* Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value * Updating version of msprod (#969) * fix(deps): update module golang.org/x/net to v0.33.0 [security] (#967) Co-authored-by: Vardhan Vinay Thigle <[email protected]> * feat: APIs for Backend Changes for Default Values (#965) * backend apis * linting * comment changes * comment changes * feat: default value for mysql source (#963) * source dv * test fix * change * comment changes * test fix * change * fix github v * change * Check constraint backend (#9) Backend Support for Check Constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * removed duplicate function * Fixed UT --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: Vardhan Vinay Thigle <[email protected]> Co-authored-by: Mend Renovate <[email protected]> Co-authored-by: Astha Mohta <[email protected]>

* chore(deps): update dependency jasmine-core to ~5.5.0 (GoogleCloudPlatform#953) Co-authored-by: Vardhan Vinay Thigle <[email protected]> * removed duplicate function * Fixed UT * feat: make spanner config mandatory for schema migration (GoogleCloudPlatform#970) * changes * change * change * chore(deps): update gcr.io/cloud-spanner-emulator/emulator docker tag to v1.5.28 (GoogleCloudPlatform#974) * chore(deps): update dependency @types/node to v20.17.10 (GoogleCloudPlatform#973) Co-authored-by: Vardhan Vinay Thigle <[email protected]> * Add Support for Check Constraint - Backend code changes (GoogleCloudPlatform#945) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value * Updating version of msprod (GoogleCloudPlatform#969) * fix(deps): update module golang.org/x/net to v0.33.0 [security] (GoogleCloudPlatform#967) Co-authored-by: Vardhan Vinay Thigle <[email protected]> * feat: APIs for Backend Changes for Default Values (GoogleCloudPlatform#965) * backend apis * linting * comment changes * comment changes * feat: default value for mysql source (GoogleCloudPlatform#963) * source dv * test fix * change * comment changes * test fix * change * fix github v * change * Check constraint backend (#9) Backend Support for Check Constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * removed duplicate function * Fixed UT --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: Vardhan Vinay Thigle <[email protected]> Co-authored-by: Mend Renovate <[email protected]> Co-authored-by: Astha Mohta <[email protected]> * Check constraint backend (#9) Backend Support for Check Constraint * removed duplicate code after rebase * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value * feat: APIs for Backend Changes for Default Values (GoogleCloudPlatform#965) * backend apis * linting * comment changes * comment changes * feat: default value for mysql source (GoogleCloudPlatform#963) * source dv * test fix * change * comment changes * test fix * change * fix github v * change * Check constraint backend (#9) Backend Support for Check Constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * removed duplicate function * Fixed UT * Check constraint backend (#9) Backend Support for Check Constraint * removed duplicate code after rebase * removed duplicate constants * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * updated the regrex for constraint name * address the comment * addressed the comment 1. change checkIfCcColumn to check in Spschema * address the comment 1.Modify the checkIfCcColumn to handle pk and check constraint * address the comment 1. when interleave tab is shown 4 or 3 * fixed regular expression for check constraints name * remove the node-sql-parser package * handle check constraint without name. * resolve the duplicate issue * Check constraint backend (#9) Backend Support for Check Constraint * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * fix: added regular exprression to match the exact column * Added test case for the column rename for check constraint * PR and UT fixes * added the verification api integration and struct changes * added the verification at connect button * added the integration of expression api * code fix for expression api implementation * code fix for expression verification on initial call * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * fix the UT issue * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * Fixed unit test and code refactor * fixed unit tests * added source in conv object * front-end ut fixed * removed node sql parser import * removed node dependencies * rename handler * added the generation of constraint name * added dump flow * added dump flow * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * remove the node-sql-parser package * Check constraint backend (#9) Backend Support for Check Constraint * fixed PR comments * fix: added regular exprression to match the exact column * Added test case for the column rename for check constraint * nit: doesCheckConstraintNameExist * moved regex globally * modify the query to fetch the check constraint * added the verification api integration and struct changes * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * Fixed unit test and code refactor * removed node dependencies * added dump flow * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * remove the node-sql-parser package * Check constraint backend (#9) Backend Support for Check Constraint * added the verification api integration and struct changes * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * removed node dependencies * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * remove the node-sql-parser package * Check constraint backend (#9) Backend Support for Check Constraint * added the verification api integration and struct changes * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * removed node dependencies * added UT * address the comment 1. handle the unexpected error 2. resuse the code * updated the table exist query for version8.0.16 * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * address the comment 1.Modify the checkIfCcColumn to handle pk and check constraint * rebase commit * fixed regular expression for check constraints name * remove the node-sql-parser package * resolve the duplicate issue * renamed function name to formatCheckConstraints and added check if constraint name is empty * fix: added regular exprression to match the exact column * Added test case for the column rename for check constraint * reverted white spaces * reverted white spaces * added the verification api integration and struct changes * added the integration of expression api * code fix for expression api implementation * code fix for expression verification on initial call * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * fixed unit tests * front-end ut fixed * removed node sql parser import * removed node dependencies * rename handler * added the generation of constraint name * added dump flow * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * remove the node-sql-parser package * Check constraint backend (#9) Backend Support for Check Constraint * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * fix: added regular exprression to match the exact column * Added test case for the column rename for check constraint * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * moved regex globally * modify the query to fetch the check constraint * added the verification api integration and struct changes * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * Fixed unit test and code refactor * removed node dependencies * added dump flow * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * address the comment 1.Modify the checkIfCcColumn to handle pk and check constraint * fixed regular expression for check constraints name * remove the node-sql-parser package * resolve the duplicate issue * Check constraint backend (#9) Backend Support for Check Constraint * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * moved regex globally * modify the query to fetch the check constraint * added the verification api integration and struct changes * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * fixed unit tests * front-end ut fixed * remove unwanted code * removed node sql parser import * removed node dependencies * rename handler * added the generation of constraint name * added dump flow * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * address the comment 1.Modify the checkIfCcColumn to handle pk and check constraint * fixed regular expression for check constraints name * remove the node-sql-parser package * resolve the duplicate issue * Check constraint backend (#9) Backend Support for Check Constraint * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * moved regex globally * Fix UT * modify the query to fetch the check constraint * added the verification api integration and struct changes * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * fixed unit tests * front-end ut fixed * remove unwanted code * removed node sql parser import * removed node dependencies * rename handler * added the generation of constraint name * added dump flow * added UT * address the comment 1. handle the unexpected error 2. resuse the code * added mock for RefreshSpannerClient function * addressed the comment and feedback --------- Co-authored-by: Mend Renovate <[email protected]> Co-authored-by: Vardhan Vinay Thigle <[email protected]> Co-authored-by: taherkl <[email protected]> Co-authored-by: Taher Lakdawala <[email protected]> Co-authored-by: Astha Mohta <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: Vivek Yadav <[email protected]>

Support check constraint frontend (#22) * chore(deps): update dependency jasmine-core to ~5.5.0 (GoogleCloudPlatform#953) Co-authored-by: Vardhan Vinay Thigle <[email protected]> * removed duplicate function * Fixed UT * feat: make spanner config mandatory for schema migration (GoogleCloudPlatform#970) * changes * change * change * chore(deps): update gcr.io/cloud-spanner-emulator/emulator docker tag to v1.5.28 (GoogleCloudPlatform#974) * chore(deps): update dependency @types/node to v20.17.10 (GoogleCloudPlatform#973) Co-authored-by: Vardhan Vinay Thigle <[email protected]> * Add Support for Check Constraint - Backend code changes (GoogleCloudPlatform#945) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value * Updating version of msprod (GoogleCloudPlatform#969) * fix(deps): update module golang.org/x/net to v0.33.0 [security] (GoogleCloudPlatform#967) Co-authored-by: Vardhan Vinay Thigle <[email protected]> * feat: APIs for Backend Changes for Default Values (GoogleCloudPlatform#965) * backend apis * linting * comment changes * comment changes * feat: default value for mysql source (GoogleCloudPlatform#963) * source dv * test fix * change * comment changes * test fix * change * fix github v * change * Check constraint backend (#9) Backend Support for Check Constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * removed duplicate function * Fixed UT --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: Vardhan Vinay Thigle <[email protected]> Co-authored-by: Mend Renovate <[email protected]> Co-authored-by: Astha Mohta <[email protected]> * Check constraint backend (#9) Backend Support for Check Constraint * removed duplicate code after rebase * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value * feat: APIs for Backend Changes for Default Values (GoogleCloudPlatform#965) * backend apis * linting * comment changes * comment changes * feat: default value for mysql source (GoogleCloudPlatform#963) * source dv * test fix * change * comment changes * test fix * change * fix github v * change * Check constraint backend (#9) Backend Support for Check Constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * removed duplicate function * Fixed UT * Check constraint backend (#9) Backend Support for Check Constraint * removed duplicate code after rebase * removed duplicate constants * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * updated the regrex for constraint name * address the comment * addressed the comment 1. change checkIfCcColumn to check in Spschema * address the comment 1.Modify the checkIfCcColumn to handle pk and check constraint * address the comment 1. when interleave tab is shown 4 or 3 * fixed regular expression for check constraints name * remove the node-sql-parser package * handle check constraint without name. * resolve the duplicate issue * Check constraint backend (#9) Backend Support for Check Constraint * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * fix: added regular exprression to match the exact column * Added test case for the column rename for check constraint * PR and UT fixes * added the verification api integration and struct changes * added the verification at connect button * added the integration of expression api * code fix for expression api implementation * code fix for expression verification on initial call * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * fix the UT issue * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * Fixed unit test and code refactor * fixed unit tests * added source in conv object * front-end ut fixed * removed node sql parser import * removed node dependencies * rename handler * added the generation of constraint name * added dump flow * added dump flow * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * remove the node-sql-parser package * Check constraint backend (#9) Backend Support for Check Constraint * fixed PR comments * fix: added regular exprression to match the exact column * Added test case for the column rename for check constraint * nit: doesCheckConstraintNameExist * moved regex globally * modify the query to fetch the check constraint * added the verification api integration and struct changes * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * Fixed unit test and code refactor * removed node dependencies * added dump flow * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * remove the node-sql-parser package * Check constraint backend (#9) Backend Support for Check Constraint * added the verification api integration and struct changes * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * removed node dependencies * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * remove the node-sql-parser package * Check constraint backend (#9) Backend Support for Check Constraint * added the verification api integration and struct changes * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * removed node dependencies * added UT * address the comment 1. handle the unexpected error 2. resuse the code * updated the table exist query for version8.0.16 * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * address the comment 1.Modify the checkIfCcColumn to handle pk and check constraint * rebase commit * fixed regular expression for check constraints name * remove the node-sql-parser package * resolve the duplicate issue * renamed function name to formatCheckConstraints and added check if constraint name is empty * fix: added regular exprression to match the exact column * Added test case for the column rename for check constraint * reverted white spaces * reverted white spaces * added the verification api integration and struct changes * added the integration of expression api * code fix for expression api implementation * code fix for expression verification on initial call * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * fixed unit tests * front-end ut fixed * removed node sql parser import * removed node dependencies * rename handler * added the generation of constraint name * added dump flow * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * remove the node-sql-parser package * Check constraint backend (#9) Backend Support for Check Constraint * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * fix: added regular exprression to match the exact column * Added test case for the column rename for check constraint * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * moved regex globally * modify the query to fetch the check constraint * added the verification api integration and struct changes * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * Fixed unit test and code refactor * removed node dependencies * added dump flow * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * address the comment 1.Modify the checkIfCcColumn to handle pk and check constraint * fixed regular expression for check constraints name * remove the node-sql-parser package * resolve the duplicate issue * Check constraint backend (#9) Backend Support for Check Constraint * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * moved regex globally * modify the query to fetch the check constraint * added the verification api integration and struct changes * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * fixed unit tests * front-end ut fixed * remove unwanted code * removed node sql parser import * removed node dependencies * rename handler * added the generation of constraint name * added dump flow * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * address the comment 1.Modify the checkIfCcColumn to handle pk and check constraint * fixed regular expression for check constraints name * remove the node-sql-parser package * resolve the duplicate issue * Check constraint backend (#9) Backend Support for Check Constraint * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * moved regex globally * Fix UT * modify the query to fetch the check constraint * added the verification api integration and struct changes * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * fixed unit tests * front-end ut fixed * remove unwanted code * removed node sql parser import * removed node dependencies * rename handler * added the generation of constraint name * added dump flow * added UT * address the comment 1. handle the unexpected error 2. resuse the code * added mock for RefreshSpannerClient function * addressed the comment and feedback --------- Co-authored-by: Mend Renovate <[email protected]> Co-authored-by: Vardhan Vinay Thigle <[email protected]> Co-authored-by: taherkl <[email protected]> Co-authored-by: Taher Lakdawala <[email protected]> Co-authored-by: Astha Mohta <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: Vivek Yadav <[email protected]>

Added sample session and dump file Added integration test for mysql check constraints Support check constraint frontend (#22) * chore(deps): update dependency jasmine-core to ~5.5.0 (GoogleCloudPlatform#953) Co-authored-by: Vardhan Vinay Thigle <[email protected]> * removed duplicate function * Fixed UT * feat: make spanner config mandatory for schema migration (GoogleCloudPlatform#970) * changes * change * change * chore(deps): update gcr.io/cloud-spanner-emulator/emulator docker tag to v1.5.28 (GoogleCloudPlatform#974) * chore(deps): update dependency @types/node to v20.17.10 (GoogleCloudPlatform#973) Co-authored-by: Vardhan Vinay Thigle <[email protected]> * Add Support for Check Constraint - Backend code changes (GoogleCloudPlatform#945) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value * Updating version of msprod (GoogleCloudPlatform#969) * fix(deps): update module golang.org/x/net to v0.33.0 [security] (GoogleCloudPlatform#967) Co-authored-by: Vardhan Vinay Thigle <[email protected]> * feat: APIs for Backend Changes for Default Values (GoogleCloudPlatform#965) * backend apis * linting * comment changes * comment changes * feat: default value for mysql source (GoogleCloudPlatform#963) * source dv * test fix * change * comment changes * test fix * change * fix github v * change * Check constraint backend (#9) Backend Support for Check Constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * removed duplicate function * Fixed UT --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: Vardhan Vinay Thigle <[email protected]> Co-authored-by: Mend Renovate <[email protected]> Co-authored-by: Astha Mohta <[email protected]> * Check constraint backend (#9) Backend Support for Check Constraint * removed duplicate code after rebase * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value * feat: APIs for Backend Changes for Default Values (GoogleCloudPlatform#965) * backend apis * linting * comment changes * comment changes * feat: default value for mysql source (GoogleCloudPlatform#963) * source dv * test fix * change * comment changes * test fix * change * fix github v * change * Check constraint backend (#9) Backend Support for Check Constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * removed duplicate function * Fixed UT * Check constraint backend (#9) Backend Support for Check Constraint * removed duplicate code after rebase * removed duplicate constants * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * updated the regrex for constraint name * address the comment * addressed the comment 1. change checkIfCcColumn to check in Spschema * address the comment 1.Modify the checkIfCcColumn to handle pk and check constraint * address the comment 1. when interleave tab is shown 4 or 3 * fixed regular expression for check constraints name * remove the node-sql-parser package * handle check constraint without name. * resolve the duplicate issue * Check constraint backend (#9) Backend Support for Check Constraint * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * fix: added regular exprression to match the exact column * Added test case for the column rename for check constraint * PR and UT fixes * added the verification api integration and struct changes * added the verification at connect button * added the integration of expression api * code fix for expression api implementation * code fix for expression verification on initial call * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * fix the UT issue * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * Fixed unit test and code refactor * fixed unit tests * added source in conv object * front-end ut fixed * removed node sql parser import * removed node dependencies * rename handler * added the generation of constraint name * added dump flow * added dump flow * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * remove the node-sql-parser package * Check constraint backend (#9) Backend Support for Check Constraint * fixed PR comments * fix: added regular exprression to match the exact column * Added test case for the column rename for check constraint * nit: doesCheckConstraintNameExist * moved regex globally * modify the query to fetch the check constraint * added the verification api integration and struct changes * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * Fixed unit test and code refactor * removed node dependencies * added dump flow * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * remove the node-sql-parser package * Check constraint backend (#9) Backend Support for Check Constraint * added the verification api integration and struct changes * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * removed node dependencies * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * remove the node-sql-parser package * Check constraint backend (#9) Backend Support for Check Constraint * added the verification api integration and struct changes * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * removed node dependencies * added UT * address the comment 1. handle the unexpected error 2. resuse the code * updated the table exist query for version8.0.16 * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * address the comment 1.Modify the checkIfCcColumn to handle pk and check constraint * rebase commit * fixed regular expression for check constraints name * remove the node-sql-parser package * resolve the duplicate issue * renamed function name to formatCheckConstraints and added check if constraint name is empty * fix: added regular exprression to match the exact column * Added test case for the column rename for check constraint * reverted white spaces * reverted white spaces * added the verification api integration and struct changes * added the integration of expression api * code fix for expression api implementation * code fix for expression verification on initial call * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * fixed unit tests * front-end ut fixed * removed node sql parser import * removed node dependencies * rename handler * added the generation of constraint name * added dump flow * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * remove the node-sql-parser package * Check constraint backend (#9) Backend Support for Check Constraint * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * fix: added regular exprression to match the exact column * Added test case for the column rename for check constraint * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * moved regex globally * modify the query to fetch the check constraint * added the verification api integration and struct changes * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * Fixed unit test and code refactor * removed node dependencies * added dump flow * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * address the comment 1.Modify the checkIfCcColumn to handle pk and check constraint * fixed regular expression for check constraints name * remove the node-sql-parser package * resolve the duplicate issue * Check constraint backend (#9) Backend Support for Check Constraint * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * moved regex globally * modify the query to fetch the check constraint * added the verification api integration and struct changes * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * fixed unit tests * front-end ut fixed * remove unwanted code * removed node sql parser import * removed node dependencies * rename handler * added the generation of constraint name * added dump flow * Feat/fe check contraints (#10) * Add the UI changes for supporting check constraints --------- Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: taherkl <[email protected]> * address the comment * address the comment 1.Modify the checkIfCcColumn to handle pk and check constraint * fixed regular expression for check constraints name * remove the node-sql-parser package * resolve the duplicate issue * Check constraint backend (#9) Backend Support for Check Constraint * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * moved regex globally * Fix UT * modify the query to fetch the check constraint * added the verification api integration and struct changes * added partial unit test of verify check constraint expression * refactored expression verfication api mock * Test added for verification api * Support check constraint backend (GoogleCloudPlatform#962) * Check constraint backend (#9) Backend Support for Check Constraint * update api * fix PR comment * remove api call to while validating constraints * Fixed db collation regex to remove collation name from the results * renamed function name to formatCheckConstraints and added check if constraint name is empty * fixed PR comments * added test case for the empty check constraint name * fix: added regular exprression to match the exact column * fix: added regular expression to replace table name * Added test case for the column rename for check constraint * 1. Refactored GetConstraint function 2. Fixed inforschema unit tests * added comment at handling case for check constraints * reverted white spaces * reverted white spaces * nit: doesCheckConstraintNameExist * added comments for doesCheckConstraintNameExist * PR and UT fixes * fix UT * UT fix * Removed isCheckConstraintsTablePresent function * moved regex globally * Fix UT * fixed UT * fixed handling of the constraints * removed unused function * added unit tests for incompatable name * Combined unit tests * added test case for the renaming column having substring of other column * added the query changes which return distinct value --------- Co-authored-by: taherkl <[email protected]> Co-authored-by: Akash Thawait <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> * fixed unit tests * front-end ut fixed * remove unwanted code * removed node sql parser import * removed node dependencies * rename handler * added the generation of constraint name * added dump flow * added UT * address the comment 1. handle the unexpected error 2. resuse the code * added mock for RefreshSpannerClient function * addressed the comment and feedback --------- Co-authored-by: Mend Renovate <[email protected]> Co-authored-by: Vardhan Vinay Thigle <[email protected]> Co-authored-by: taherkl <[email protected]> Co-authored-by: Taher Lakdawala <[email protected]> Co-authored-by: Astha Mohta <[email protected]> Co-authored-by: Vivek Yadav <[email protected]> Co-authored-by: Vivek Yadav <[email protected]>

VardhanThigle requested a review from asthamohta December 19, 2024 18:54

pull-request-size bot added the size/XS label Dec 19, 2024

VardhanThigle force-pushed the msprod-version branch 3 times, most recently from 467e22b to 9d04801 Compare December 19, 2024 19:35

pull-request-size bot added size/S and removed size/XS labels Dec 19, 2024

VardhanThigle force-pushed the msprod-version branch 9 times, most recently from 90c72cb to e115394 Compare December 19, 2024 20:38

Updating version of msprod

39b4bed

VardhanThigle force-pushed the msprod-version branch from e115394 to 39b4bed Compare December 19, 2024 20:44

VardhanThigle changed the title ~~[Draft] Updating version of msprod~~ Updating version of msprod Dec 19, 2024

VardhanThigle requested a review from bharadwaj-aditya December 19, 2024 20:51

VardhanThigle marked this pull request as ready for review December 19, 2024 20:51

VardhanThigle requested a review from a team as a code owner December 19, 2024 20:51

This was referenced Dec 19, 2024

fix(deps): update module golang.org/x/net to v0.33.0 [security] #967

Merged

fix: github issue fix #968

Closed

asthamohta approved these changes Dec 20, 2024

View reviewed changes

VardhanThigle merged commit 780b0fb into GoogleCloudPlatform:master Dec 20, 2024
4 checks passed

taherkl pushed a commit to ollionorg/spanner-migration-tool-fork that referenced this pull request Dec 20, 2024

Updating version of msprod (GoogleCloudPlatform#969)

ed259de

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Updating version of msprod #969

Updating version of msprod #969

VardhanThigle commented Dec 19, 2024 •

edited

Loading

Updating version of msprod #969

Updating version of msprod #969

Conversation

VardhanThigle commented Dec 19, 2024 • edited Loading

VardhanThigle commented Dec 19, 2024 •

edited

Loading