Wishlist: Reporter Copy Statistics View

[Chris Sharp]

While attempting to create reports for a third-party collection development service, I learned that there is currently not a way to retreive various statistics in a single report.

Branch implementing a copy statistics reports view on the way.

[Mike Rylander (mrylander)]

Chris, are you still working on this?

[Chris Sharp (chrissharp123)]

Sorry for the delay. Branch here:

http://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/csharp/lp1672346_reporter_copy_statistics_view

I added it to the example.reporter-extension.sql file since it depends on a table in extend_reporter, which is not installed by default. I'm open to alternative suggestions to how to deal with that issue.

[Dan Wells (dbw2)]

Gave this a quick look, and just noting the provided branch still needs an upgrade script.

[Chris Sharp (chrissharp123)]

Dan,

I had intentionally not created an upgrade script because I understood that not all Evergreen implementations apply this script to their systems. Should we assume otherwise? PINES uses the reporter sources available in the file and we just re-run the example.reporter-extension.sql file when we need to update things.

If the consensus is that we should assume these sources to be "core" Evergreen, I'm happy to include an upgrade script.

See bug 1050384 for related discussion.

Chris

[Dan Wells (dbw2)]

Chris, thanks for the context, especially the other bug. Now that I understand that that file is all views, I can see the logic in not creating an upgrade script, but we do need some other mechanism then to keep folks at least aware of changes. This would have similarities to the latent plans to improve in-DB function changes by reapplying those en masse at upgrade time (or whenever needed).

I agree with the consensus on the other bug that it doesn't make sense to half-support (and half-enable via the IDL) these views. We should put some heads together (maybe at the Hackfest?), and if some reports can be fully "de-PINESed", then let's enable those outright, but if others cannot, let's provide those fully separate and disabled OOTB. That's my initial take, anyway.

Finally, being solely DB views, and being already in an odd state, I think straightening this out could still come pre-RC if there is energy behind it.

[Ruth Frasur Davis (redavis)]

I have tested this code and consent to signing off on it with my name, rfrasur and my email address, <email address hidden>.

[Chris Sharp (chrissharp123)]

This was committed for 3.8, but the bug was not updated.