Skip to content

Commit

Permalink
Merge pull request #22 from eschava/CustomPrefix
Browse files Browse the repository at this point in the history
fixed issue with failed InfluxDB insertion if custom prefix is empty
  • Loading branch information
asimell committed Nov 21, 2017
2 parents 9338f40 + 739f95d commit 8ff9342
Showing 1 changed file with 8 additions and 3 deletions.
Expand Up @@ -21,12 +21,17 @@ public AbstractPointGenerator(MeasurementRenderer projectNameRenderer) {
@Override
public Point.Builder buildPoint(String name, String customPrefix, Run<?, ?> build) {
final String renderedProjectName = projectNameRenderer.render(build);
return Point
Point.Builder builder = Point
.measurement(name)
.addField(PROJECT_NAME, renderedProjectName)
.addField(BUILD_NUMBER, build.getNumber())
.tag(PROJECT_NAME, renderedProjectName)
.tag(CUSTOM_PREFIX, measurementName(customPrefix));
.tag(PROJECT_NAME, renderedProjectName);

if (customPrefix != null && !customPrefix.isEmpty())
builder = builder.tag(CUSTOM_PREFIX, measurementName(customPrefix));

return builder;

}

protected String measurementName(String measurement) {
Expand Down

0 comments on commit 8ff9342

Please sign in to comment.