Reduce field size #150
Reduce field size #150
Conversation
|
Please sign the CLA so I can review these contributions. https://github.com/mitchspano/apex-trigger-actions-framework/blob/main/docs/contributing.md |
| <label>Order</label> | ||
| <precision>18</precision> |
There was a problem hiding this comment.
Why? This field is used in the framework's logic and is not related to an API name with limited length.
There was a problem hiding this comment.
You are right there is not a specific or hard limit here. However, the goal is to reduce storage size and reducing precision DOES make each record take up less space. I would argue that a precision of 18 is excessively large. No one will ever need to be able to order 100 trillion trigger actions (x1000 decimal places). I am proposing cutting storage space by 1/3 and limiting to 9999 actions without using decimals or 1 million if using the 2 decimals also. This should still leave plenty of breathing room to make your order values every 10 or even 100 to leave space in between for future actions.
| <required>true</required> | ||
| <scale>3</scale> | ||
| <scale>2</scale> |
There was a problem hiding this comment.
This will truncate data for anyone who is already using all 3 decimal places.
Let's keep it at 3.
| <required>true</required> | ||
| <scale>3</scale> | ||
| <scale>2</scale> |
There was a problem hiding this comment.
This will truncate data for anyone who is already using all 3 decimal places.
Let's keep it at 3.
Custom Metadata Types have a small (10Mb) limit for the total amount you can store in the records. Storage usage is determined by a fixed per record size that is affected by the field size.
Current field size is set to the max for all field types. However, several fields reference API names which have documented max lengths they can be in Salesforce. Any field size greater than this is "wasted" storage as it will never be needed.
Changes
Reduced fields referencing Salesforce APIs to the maximum allowed API Name length for the type
Updated fieldManagability to SubscriberControlled so that the framework could be included in a managed package.
Tests pass
Appropriate changes to README are included in PR