I can't remember making the change, or why I did. Feel free to revert it.

What was the intention behind this commit, where to find related discussions?

In T352782#9414884, @Aklapper wrote:

This custom code change did not "Removed a bunch of brand-specific icons" as its description says.
This code changed getIconQuips() and that very function only provides tooltips and does not change any selection of icons. Thus it made a bunch of icons not have a descriptive tooltip but display the internal filename as the tooltip instead:

I think the intention is to share the "single document url" from logstash instead of the request id. The instructions are unclear, however, and could be improved.

I'd be happy to answer any questions you might have about this. I think I still remember a few things

The bulk of that behavior is implemented in webroot/rsrc/js/application/transactions/behavior-show-older-transactions.js

Congrats on the migration, very happy to see this completed.

If it helps: It's unlikely that phabricator will ever need elasticsearch in the future.