The Field API generates a table storing data for each field. The structure of each record in the table includes two 128 character indexed fields -- one for the entity and one for the bundle to which the field is attached.
In the instance of making a boolean field, 300 bytes of data (256 of which are indexed) are used to store each boolean field containing a single bit of information.
This overhead is duplicated in the revisions table for the field. This results in wasted space and extra computer processing to find or save all that indexed information.
This storage overhead makes it problematic to use the Field API in applications that will store and access large quantities of data.
A New Field API could be much more efficient by saving simple two-byte integers that map to entity type and bundle information and using a mapping function to convert entity and bundle names to their appropriate integers. The New Field API could coexist with the old-school Field API, but provide leaner performance for new development.
Additionally, the names of entities and bundles would no longer be limited to 128 characters. They could be of arbitrary length.
Recent comments
Worth mentioning as it was in zulip chat by @yorkshirepudding I just tested and the two work together without issue as long as you don't use the "Exclude comments from Content...
Search does not include comments
There is also Node Comments Search - this searches the node and all comments (default search is node and first page of comments) as part of the content search. Izzy's module above is a separate...
Search does not include comments
I have created a new module comment_search
Search does not include comments