Github user mattyb149 commented on a diff in the pull request:
    --- Diff: 
    @@ -0,0 +1,212 @@
    + * Licensed to the Apache Software Foundation (ASF) under one or more
    + * contributor license agreements.  See the NOTICE file distributed with
    + * this work for additional information regarding copyright ownership.
    + * The ASF licenses this file to You under the Apache License, Version 2.0
    + * (the "License"); you may not use this file except in compliance with
    + * the License.  You may obtain a copy of the License at
    + *
    + *
    + *
    + * Unless required by applicable law or agreed to in writing, software
    + * distributed under the License is distributed on an "AS IS" BASIS,
    + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
    + * See the License for the specific language governing permissions and
    + * limitations under the License.
    + */
    +package org.apache.nifi.reporting;
    +import java.nio.charset.StandardCharsets;
    +import java.util.ArrayList;
    +import java.util.Collection;
    +import java.util.Collections;
    +import java.util.HashMap;
    +import java.util.List;
    +import java.util.Map;
    +import java.util.UUID;
    +import java.util.concurrent.TimeUnit;
    +import javax.json.Json;
    +import javax.json.JsonBuilderFactory;
    +import javax.json.JsonObject;
    +import org.apache.avro.Schema;
    +import org.apache.nifi.annotation.documentation.CapabilityDescription;
    +import org.apache.nifi.annotation.documentation.Tags;
    +import org.apache.nifi.avro.AvroTypeUtil;
    +import org.apache.nifi.components.AllowableValue;
    +import org.apache.nifi.components.PropertyDescriptor;
    +import org.apache.nifi.components.ValidationContext;
    +import org.apache.nifi.components.ValidationResult;
    +import org.apache.nifi.components.Validator;
    +import org.apache.nifi.controller.status.ProcessGroupStatus;
    +import org.apache.nifi.flowfile.attributes.CoreAttributes;
    +import org.apache.nifi.processor.exception.ProcessException;
    +import org.apache.nifi.processor.util.StandardValidators;
    +import org.apache.nifi.remote.Transaction;
    +import org.apache.nifi.remote.TransferDirection;
    +import org.apache.nifi.reporting.util.metrics.MetricNames;
    +import org.apache.nifi.reporting.util.metrics.MetricsService;
    +import org.apache.nifi.reporting.util.metrics.api.MetricsBuilder;
    +import com.yammer.metrics.core.VirtualMachineMetrics;
    +@Tags({"status", "metrics", "site", "site to site"})
    +@CapabilityDescription("Publishes same metrics as the Ambari Reporting 
task using the Site To Site protocol.")
    +public class SiteToSiteMetricsReportingTask extends 
AbstractSiteToSiteReportingTask {
    +    static final AllowableValue AMBARI_FORMAT = new 
AllowableValue("ambari-format", "Ambari Format", "Metrics will be formatted"
    +            + " according to the Ambari Metrics API. See Additional 
Details in Usage documentation.");
    +    static final AllowableValue RECORD_FORMAT = new 
AllowableValue("record-format", "Record Format", "Metrics will be formatted"
    +            + " using the Record Writer property of this reporting task. 
See Additional Details in Usage documentation to"
    +            + " have the description of the default schema.");
    +    static final PropertyDescriptor APPLICATION_ID = new 
    +            .name("s2s-metrics-application-id")
    +            .displayName("Application ID")
    +            .description("The Application ID to be included in the 
    +            .required(true)
    +            .expressionLanguageSupported(true)
    +            .defaultValue("nifi")
    +            .addValidator(StandardValidators.NON_EMPTY_VALIDATOR)
    +            .build();
    +    static final PropertyDescriptor HOSTNAME = new 
    +            .name("s2s-metrics-hostname")
    +            .displayName("Hostname")
    +            .description("The Hostname of this NiFi instance to be 
included in the metrics")
    +            .required(true)
    +            .expressionLanguageSupported(true)
    +            .defaultValue("${hostname(true)}")
    +            .addValidator(StandardValidators.NON_EMPTY_VALIDATOR)
    +            .build();
    +    static final PropertyDescriptor FORMAT = new 
    +            .name("s2s-metrics-format")
    +            .displayName("Output format")
    +            .description("The output format that will be used for the 
    --- End diff --
    Perhaps add to the description that if Record Format is selected, a Record 
Writer must be provided. Also for consistency, the display name should have 
both words capitalized.


Reply via email to