[Tables] Make delete_entity also convert Partition/RowKey to string when TableEntity is passed

[mtin]

Description

This pull request makes delete_entity do the same conversion of all properties of the TableEntity into string as the other methods (via _add_entity_properties e.g. create_entity, upsert_entity) to allow passing a TableEntity with non-string Partition/RowKey

See https://github.com/Azure/azure-sdk-for-python/issues/26360

All SDK Contribution checklist:

• [x] The pull request does not introduce [breaking changes]
• [x] CHANGELOG is updated for new features, bug fixes or other significant changes.
• [x] I have read the contribution guidelines.

General Guidelines and Best Practices

• [x] Title of the pull request is clear and informative.
• [x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

• [ ] Pull request includes test coverage for the included changes.
  • I did add tests, did not get them to run though; hoping for pipelines here :)

[ghost]

Thank you for your contribution mtin! We will review the pull request and get back to you soon.

[ghost]

All CLA requirements met.

[YalinLi0312]

Hi @mtin thanks for your contribution! The PR is currently in draft, I'm not sure if you are preparing it. Could you please fix the failing CI? Or I can help the rest with your permission granted in your GitHub branch.

Thanks

[mtin]

Thank you for getting back to me @YalinLi0312! I was unsure if the proposed changes are desired at all; There were no further answers at https://github.com/Azure/azure-sdk-for-python/issues/26360. I also could not really make sense of the failing pipelines, I would be grateful if you could assist. I have granted you permission to my fork!

[YalinLi0312]

Thank you for getting back to me @YalinLi0312! I was unsure if the proposed changes are desired at all; There were no further answers at #26360. I also could not really make sense of the failing pipelines, I would be grateful if you could assist. I have granted you permission to my fork!

I have accepted your invitation. No worries, I'll help on the rest. Thanks

[annatisch]

/azp run python - tables - test

[azure-pipelines[bot]]

No pipelines are associated with this pull request.

[annatisch]

/azp run python - tables - tests

[azure-pipelines[bot]]

Azure Pipelines successfully started running 1 pipeline(s).

[mtin]

@microsoft-github-policy-service agree

[YalinLi0312]

Close it as we will fix the UUID type issue in our new serialization design, thanks for your contribution as well @mtin