-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SPARK-25280 Expose metrics for Prometheus #25
Conversation
@@ -37,6 +38,7 @@ | |||
import javax.validation.Valid; | |||
import java.util.Optional; | |||
|
|||
//@Timed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should this stay commented out?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I will remove this.
Left from my experiments with metrics
* limitations under the License. | ||
*/ | ||
@Component | ||
public class MetricsProvider { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm skepticall on the purpose and implementation of those metrics. If will have multiple instances of ASMS they all will make redundant calls and unnecessary stress the database.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK,
Will remove these.
@kubagruszka @alewir I made changes according to your comments. Could review PR again? |
Be default Prometheus metrics are disable. Will be exposed in out environments by changes in deployment file.
Locally for development purposes can be enabled by adding following environment variable to run configuration:
Example of metrics: