User activity tracking should be transient by default

+1

Code pushed to change the default value for the transient column from FALSE to TRUE.

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/berick/lp1570909-usr-activity-transient

This branch (as is) only affects new installs. It does not modify activity types on existing systems. My concern here was that, depending on the size of the actor.usr_activity table, some manual data cleanup may be needed/preferred before toggling the transient value, since setting an activity type to transient could result in huge numbers of deletes in a short period of time.

Would it be helpful to have a tool (SQL script) to clean up the usr_activity table by deleting all but the most recent entry per user per activity type that people can run off-hours before modifying their transient settings?

Additional commits pushed:

1. Adds a new function for purging user activity rows by activity type, leaving the most recent row per user, consistent with transience.

2. pgtap test for the above.

3. release notes.

Adding pullrequest and targeting all active releases, since it only affects new installs, while providing tools to optionally affect existing installs.

PGTap tests pass and function works as expected. Signoff branch at http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/csharp/lp1570909-usr-activity-transient-signoff

Master has it now! I've decided against backporting this myself -- the db change pushed me over the line. If back-branch RMs feel differently, I leave it to them.

Thanks, Bill!

For my part, I've decided against backporting this to 2.10.