uber-common / jvm-profiler

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

Influxdb-java version needs to be updated #67

Closed m-vemuri closed 4 years ago

m-vemuri commented 4 years ago

While building the jar for running with influx, the following error is seen:

[ERROR] /Users/mukund.vemuri/IdeaProjects/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/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

It appears that the version of influxdb-java is old:

                <dependency>
                    <groupId>org.influxdb</groupId>
                    <artifactId>influxdb-java</artifactId>
                    <version>2.7</version>
                </dependency>

To resolve this, it is recommend to update the influxdb-java dependency in the pom.

hiboyang commented 4 years ago

thanks for the recommendation, please feel free to submit a Pull Request.

m-vemuri commented 4 years ago

Thanks for the reply @boy-uber. I have submitted PR https://github.com/uber-common/jvm-profiler/pull/71.

m-vemuri commented 4 years ago

PR #71 merged. Closing ticket.