uber-common / jvm-profiler

JVM Profiler Sending Metrics to Kafka, Console Output or Custom Reporter
Other
1.78k stars 342 forks source link

Updating influxdb-java version in pom.xml #71

Closed m-vemuri closed 4 years ago

m-vemuri commented 4 years ago

1. Description:

  1. This change is to fix the failing build for influxdb on the uber-common/jvm-profiler.

2. Summary of Changes:

  1. Updated org.influxdb.influxdb-java version in pom.xml.
  2. Version updated with is a previously used version based on the git history.

3. Test Runs:

3.1. Before fix:
[INFO] BUILD FAILURE
:
:
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.3:compile (default-compile) on project jvm-profiler: Compilation failure: Compilation failure: 
[ERROR] /Users/mukund.vemuri/IdeaProjects/m-vemuri/uber-common/jvm-profiler/src/main/java_influxdb/com/uber/profiling/reporters/InfluxDBOutputReporter.java:[9,20] cannot find symbol
[ERROR]   symbol:   class BatchOptions
[ERROR]   location: package org.influxdb
[ERROR] /Users/mukund.vemuri/IdeaProjects/m-vemuri/uber-common/jvm-profiler/src/main/java_influxdb/com/uber/profiling/reporters/InfluxDBOutputReporter.java:[150,39] cannot find symbol
[ERROR]   symbol:   variable BatchOptions
[ERROR]   location: class com.uber.profiling.reporters.InfluxDBOutputReporter
3.2. After fix:
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  58.936 s
[INFO] Finished at: 2020-04-13T18:07:13-04:00
[INFO] ------------------------------------------------------------------------

4. Reference Conversation:

Please refer to the conversation with one of the maintainers here.

CLAassistant commented 4 years ago

CLA assistant check
All committers have signed the CLA.