Newsletter\Model\Subscriber::loadByEmail() does not use MySQL index

[devamitbera]

• Use MYSQL index
• Add an Mysql Index key via upgrade Schema
• files add at app/code/Magento/Newsletter/Setup/UpgradeScheme.php
• Modified at app/code/Magento/Newsletter/etc/module.xml

Description

Fixed Issues (if relevant)

1. Newsletter\Model\Subscriber::loadByEmail() does not use MySQL index #12787: newsletter-subscriber-loadbyemail-mysql-index-issue

Manual testing scenarios

Require to run Setup command for upgrade Newsletter Scheme
Run below query at database to check index key exists for field subscriber_email

EXPLAIN SELECT * FROM newsletter_subscriber WHERE subscriber_email='{EmailId}'

Contribution checklist

• Pull request has a meaningful description of its purpose
• All commits are accompanied by meaningful commit messages
• All new or changed code is covered with unit/integration tests (if applicable)
• All automated tests passed successfully (all builds on Travis CI are green)

[magento-cicd2]

All committers have signed the CLA.

[dmanners]

Just have a look through the styles please.

Feedback from our tests

FILE: ...LE1/magento2/app/code/Magento/Newsletter/Setup/UpgradeSchema.php
----------------------------------------------------------------------
FOUND 4 ERRORS AFFECTING 4 LINES
----------------------------------------------------------------------
  2 | ERROR | [x] There must be one blank line after the namespace
    |       |     declaration
  7 | ERROR | [x] There must be one blank line after the last USE
    |       |     statement; 0 found;
 11 | ERROR | [x] Expected 1 space before "UpgradeSchemaInterface"; 2
    |       |     found
 37 | ERROR | [x] Expected 1 newline at end of file; 0 found
----------------------------------------------------------------------
PHPCBF CAN FIX THE 4 MARKED SNIFF VIOLATIONS AUTOMATICALLY
----------------------------------------------------------------------