TypeParameters to use string_view

[anand-trehan]

Description

Refactored TypeParameters to use string_view while parsing the raw string.

The function changed is TypeParameters::from_raw in 'type_context.cc'.

Related Issue

#363

How was this tested?

• Debug Build cmake ~/villagesql-server -DWITH_DEBUG=1 -DWITH_SSL=system followed by make -j $(($(getconf _NPROCESSORS_ONLN) - 2))
• Run village-sql unit tests ctest -L villagesql
• Run village test suite 'mysql-test/mysql-test-run.pl --do-suite=village --nounit-tests --parallel=auto'

Checklist

• I have signed the CLA
• I have read CONTRIBUTING.md
• I have added or updated tests as appropriate
• My changes maintain compatibility with upstream MySQL 8.4

[github-actions]

All contributors have signed the CLA ✍️ ✅
_{Posted by the CLA Assistant Lite bot.}

[anand-trehan]

I have read the CLA Document and I hereby sign the CLA

[anand-trehan]

Recheck

[villagestevers]

Closes #363

I opened 518 as a follow on.

[villagestevers]

You need to run scripts/villint (PR validation is failing)

I changed the title to be shorter thinking that was the only problem, but linting is the issue.

Also, please don't use merge commits in the future. The villint change should be layered on as a separate commit. We squash and merge at the end, so we only rebase after approval.

Thanks!

[anand-trehan]

Okay I ran the linting script and created a new commit. Thanks for the feedback will keep in mind not to use merge commits from here on. Let me know if this looks good now.

Thanks!

[villagestevers]

Thanks @anand-trehan - I will squash and merge your change in once the CI finishes. Please join our discord if you have not already! (makes coordinating easier)