Speed up library list loading when SQL is disabled

[sebjulliand]

Changes

When SQL is disabled, loading the library list items take a significant amount of time because it runs DSPOBJD OBJ(QSYS/*ALL) to load the librarys' description.
This PR makes it run one DSPOBJD for each library which speeds up the loading of the Library List view.

The other possible fix for this is to disregard config.enableSQL entirely and always use OBJECT_STATISTICS.

Checklist

• have tested my change

[chrjorgensen]

@sebjulliand LGTM - same result as before the change.

But I don't see any speed difference with my 6 libraries in the library list. Have we had any reports of slow library list view?

[sebjulliand]

@sebjulliand LGTM - same result as before the change.

But I don't see any speed difference with my 6 libraries in the library list. Have we had any reports of slow library list view?

I did not report it, but it's been bothering me lately as I kept seeing the view loading for too long on one of the LPAR I work on.
But this is a development LPAR, and it has tons of libraries, so that must be why!

[sebjulliand]

For the record: that development LPAR has 11010 libraries 😅 And since my profile has *ALLOBJ, I can list and access them all.
I tried on another LPAR with 845 libraries and in this case, the change is not noticeable.

But thanks for testing and validating, much appreciated!

[worksofliam]

That's a lot of DSPOBJD! Makes sense, but yes, I wondering if using OBJECT_STATISTICS would be better - esp. since we don't need to worry about the CCSID stuff now :)

[chrjorgensen]

For the record: that development LPAR has 11010 libraries

Wauw - you should do some cleanup! 😆

I tested in one of our LPARs with 1227 libraries and didn't see any difference after the change. I remember once thinking about why all libraries were included, but back then we could not send multiple commands at once, so I decided that it was better keep getting all libraries with one command / round-trip than getting one at a time and having multiple commands and round-trips. Your excellent change allowing us to run multiple commands in only one round-trip now made this change possible. Thanks!

[sebjulliand]

Thank you guys!
I guess we'll be able to keep only the OBJECT_STATISTICS part for 3.0.0.