-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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: Implement data type handling in MySQL (#16621) #17017
Conversation
With this implementation we can now achieve the following under prepared statement: - Ability to distinguish between null object and "null" string - Ability to distinguish values like {{"098765"}} and {{098765}}. The former is identified as a string and the latter is identified as an integer
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Unable to find test scripts. Please add necessary tests to the PR. |
app/server/appsmith-plugins/mysqlPlugin/src/main/java/com/external/plugins/MySqlPlugin.java
Outdated
Show resolved
Hide resolved
app/server/appsmith-plugins/mysqlPlugin/src/main/java/com/external/plugins/MySqlPlugin.java
Outdated
Show resolved
Hide resolved
...smith-interfaces/src/main/java/com/appsmith/external/plugins/SmartSubstitutionInterface.java
Show resolved
Hide resolved
...plugins/mysqlPlugin/src/main/java/com/external/plugins/datatypes/MySQLSpecificDataTypes.java
Outdated
Show resolved
Hide resolved
/ok-to-test sha=a0218fb |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3126341717. |
/ok-to-test sha=f6d21b8 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3158213453. |
UI Performance test run logs and artifacts: https://github.com/appsmithorg/appsmith/actions/runs/3158213453. Click to view performance test results
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cypress tests looks good!
app/client/cypress/integration/Smoke_TestSuite/ServerSideTests/Datatypes/MySQL_Spec.ts
Outdated
Show resolved
Hide resolved
/ok-to-test sha=39a6767 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3159873742. |
UI Performance test run logs and artifacts: https://github.com/appsmithorg/appsmith/actions/runs/3159873742. Click to view performance test results
|
/ok-to-test sha=a05f496 |
Tests running at: https://github.com/appsmithorg/appsmith/actions/runs/3179606363. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Cypress looks good!
UI Performance test run logs and artifacts: https://github.com/appsmithorg/appsmith/actions/runs/3179606363. Click to view performance test results
|
Description
With this implementation we can now achieve the following under the prepared statement:
null
object and"null"
string{{"098765"}}
and{{098765}}
. The former is identified as a string and the latter is identified as an integerFixes #16621
Fixes #16600
Fixes #15572
Fixes #10456
Fixes #16811
Fixes #17058
Fixes #11691
Type of change
How Has This Been Tested?
Checklist: