Resolve ignored_columns vs included_columns on model builders

[exalate-issue-sync]

Should ignored_columns on all the model builders be changed to included_columns?

cc [~accountid:557058:abbecef9-9266-49cd-a13f-d0c5d27cb5da], [~accountid:557058:1529d34e-fbf0-45b1-8f33-facc04ecb672]

[exalate-issue-sync]

Cliff Click commented: Minor tweak to the API - general philosophy is to choose positive over negatives. So yes, included is better than excluded, even if the included-list is likely to be large all the time.

[exalate-issue-sync]

Raymond Peck commented: I talked with Patrick about this the other day, and my proposal was to allow the client to specify either the included or the excluded (error out if they specify both), and always convert that in the REST API layer to the current ignored_columns mechanism.

[hasithjp]

JIRA Issue Migration Info

Jira Issue: PUBDEV-123
Assignee: Raymond Peck
Reporter: Prithvi Prabhu
State: Resolved
Fix Version: N/A
Attachments: N/A
Development PRs: Available

Linked PRs from JIRA

#5352