Optimize Query and BroadQuery when not including data fields

[vestrel00]

The documentation I put up for optimizing queries by using include is currently not implemented (I thought I already did though!!!).

Problem

When using the Query or BroadQuery APIs, the data table is still queried even if no data field is included.

contacts-android/core/src/main/java/contacts/core/Query.kt

Lines 666 to 674 in 142eced

	// Collect Data for non-blank RawContact and Contact in the given offsetAndLimitedContactIds.
	// If offsetAndLimitedContactIds is null, then all Data and non-blank RawContacts and Contacts
	// are collected.
	query(
	Table.Data, include, offsetAndLimitedContactIds?.let {
	Fields.Contact.Id `in` it
	},
	processCursor = contactsMapper::processDataCursor
	)

This is a completely unnecessary query if only Fields.Contact fields are included. It leads to unnecessary parsing of potentially large number of data rows.

Only the Contacts and RawContacts table need to be queried. The Data table can be skipped in such cases.

Solution

Skip the data query if no data field is included.

• Update https://vestrel00.github.io/contacts-android/entities/include-only-desired-data/#optimizing-queries by mentioning that the optimization is available sine v0.2.4. Also update the benchmark numbers (10,000 contacts currently takes 1-2 seconds but it could be reduced by 50%+ with this optimization!)
• Add a note that not including any data fields will result in all returned Contact and RawContact entities to be blank (isBlank will return true) even if they are actually not blank in the database.
• Update gh-pages branch

[vestrel00]

Queries that only include fields from Fields.Contact are now 10x faster (varies depending on data set size)!!!!

[vestrel00]

This is included in 0.2.4!