terascope / teraslice

Scalable data processing pipelines in JavaScript
https://terascope.github.io/teraslice/
Apache License 2.0
50 stars 13 forks source link

Remove PromMetrics proxy, check for promMetrics with isPromAvailable() #3627

Closed busma13 closed 2 months ago

busma13 commented 2 months ago

This is a follow-up PR to #3629 which created a utility function to test if promMetrics are available, allowing assets to be backward compatible with teraslice before promMetrics were introduced (< 1.4.0). At the same time we decided to just make isPromAvailable() checks everywhere promMetrics was used, instead of using a proxy to catch calls to undefined functions. See #3625 for more details.

This PR makes the following changes: