chore: refactor AceEditorWrapper to functional component

[EugeneTorap]

SUMMARY

Converted AceEditorWrapper class component into functional component.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

• Has associated issue:
• Required feature flags:
• Changes UI
• Includes DB Migration (follow approval process in SIP-59)
  • Migration is atomic, supports rollback & is backwards-compatible
  • Confirm DB migration upgrade and downgrade tested
  • Runtime estimates and downtime expectations provided
• Introduces new feature or API
• Removes existing feature or API

[codecov]

Codecov Report

Merging #21532 (dc69bbe) into master (2224ebe) will increase coverage by 0.04%.
The diff coverage is 70.00%.

@@            Coverage Diff             @@
##           master   #21532      +/-   ##
==========================================
+ Coverage   66.67%   66.71%   +0.04%     
==========================================
  Files        1793     1795       +2     
  Lines       68523    68648     +125     
  Branches     7281     7316      +35     
==========================================
+ Hits        45688    45799     +111     
- Misses      20971    20989      +18     
+ Partials     1864     1860       -4     

Flag	Coverage Δ
javascript	52.97% <70.00%> (+0.12%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...d/src/SqlLab/components/AceEditorWrapper/index.tsx	58.97% <68.08%> (+11.28%)	⬆️
...frontend/src/SqlLab/components/SqlEditor/index.jsx	54.34% <100.00%> (+1.60%)	⬆️
...iews/CRUD/data/dataset/AddDataset/Footer/index.tsx	33.33% <0.00%> (-66.67%)	⬇️
.../plugin-chart-pivot-table/src/plugin/buildQuery.ts	42.85% <0.00%> (-12.70%)	⬇️
...i-chart-controls/src/shared-controls/constants.tsx	22.22% <0.00%> (-11.12%)	⬇️
...nd/plugins/plugin-chart-table/src/controlPanel.tsx	53.12% <0.00%> (-9.84%)	⬇️
.../BigNumber/BigNumberWithTrendline/controlPanel.tsx	16.66% <0.00%> (-8.34%)	⬇️
...ntend/plugins/plugin-chart-table/src/buildQuery.ts	56.25% <0.00%> (-4.77%)	⬇️
...ugin-chart-pivot-table/src/plugin/controlPanel.tsx	4.16% <0.00%> (-2.98%)	⬇️
...et-ui-chart-controls/src/shared-controls/index.tsx	57.14% <0.00%> (-2.48%)	⬇️
... and 44 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[EugeneTorap]

@jinghua-qa @andrey-zayats Can you test it?

[andrey-zayats]

@jinghua-qa @andrey-zayats Can you test it?
What area can this pr affect? (where should we run regression tests?)

[EugeneTorap]

SQL Lab editor

[andrey-zayats]

SQL Lab editor

Okay, I'll take a look

[andrey-zayats]

/testenv up

[github-actions]

@andrey-zayats Ephemeral environment creation is currently limited to committers.

[jinghua-qa]

/testenv up

[github-actions]

@jinghua-qa Ephemeral environment spinning up at http://34.221.133.37:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

[andrey-zayats]

User sees an empty page when trying to open a new tab in SQL lab. The same thing happens when the user follows the link of the saved query.
The issue is already in master

Empty.page.mp4

[EugeneTorap]

@andrey-zayats If the issue is already in master then we need to create GitHub issue and fix in a separate PR. Right?

[andrey-zayats]

@andrey-zayats If the issue is already in master then we need to create GitHub issue and fix in a separate PR. Right?

Yeap, you're right

[andrey-zayats]

@EugeneTorap PR LGTM, qa verified.

[zhaoyongjie]

LGTM! Thanks for your patient and hard work.

[github-actions]

Ephemeral environment shutdown and build artifacts deleted.