<commit_before>/*
 * 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
 *
 *    http://www.apache.org/licenses/LICENSE-2.0
 *
 * 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.flink.runtime.metrics;

import org.apache.flink.metrics.Histogram;
import org.apache.flink.metrics.HistogramStatistics;

import org.apache.commons.math3.stat.descriptive.DescriptiveStatistics;

/**
 * The {@link DescriptiveStatisticsHistogram} use a DescriptiveStatistics {@link DescriptiveStatistics} as a Flink {@link Histogram}.
 */
public class DescriptiveStatisticsHistogram implements org.apache.flink.metrics.Histogram {

	private final DescriptiveStatistics descriptiveStatistics;

	private long elementsSeen = 0L;

	public DescriptiveStatisticsHistogram(int windowSize) {
		this.descriptiveStatistics = new DescriptiveStatistics(windowSize);
	}

	@Override
	public void update(long value) {
		elementsSeen += 1L;
		this.descriptiveStatistics.addValue(value);
	}

	@Override
	public long getCount() {
		return this.elementsSeen;
	}

	@Override
	public HistogramStatistics getStatistics() {
		return new DescriptiveStatisticsHistogramStatistics(this.descriptiveStatistics);
	}
}
<commit_msg>[FLINK-12981][metrics] ignore NaNs in percentile implementation<commit_after>/*
 * 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
 *
 *    http://www.apache.org/licenses/LICENSE-2.0
 *
 * 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.flink.runtime.metrics;

import org.apache.flink.metrics.Histogram;
import org.apache.flink.metrics.HistogramStatistics;

import org.apache.commons.math3.stat.descriptive.DescriptiveStatistics;
import org.apache.commons.math3.stat.descriptive.rank.Percentile;
import org.apache.commons.math3.stat.ranking.NaNStrategy;

/**
 * The {@link DescriptiveStatisticsHistogram} use a DescriptiveStatistics {@link DescriptiveStatistics} as a Flink {@link Histogram}.
 */
public class DescriptiveStatisticsHistogram implements org.apache.flink.metrics.Histogram {

	private final DescriptiveStatistics descriptiveStatistics;

	private long elementsSeen = 0L;

	public DescriptiveStatisticsHistogram(int windowSize) {
		this.descriptiveStatistics = new DescriptiveStatistics(windowSize);
		// since we are storing Long values, we won't have NaN values
		Percentile percentileImpl = new Percentile().withNaNStrategy(NaNStrategy.FIXED);
		descriptiveStatistics.setPercentileImpl(percentileImpl);
	}

	@Override
	public void update(long value) {
		elementsSeen += 1L;
		this.descriptiveStatistics.addValue(value);
	}

	@Override
	public long getCount() {
		return this.elementsSeen;
	}

	@Override
	public HistogramStatistics getStatistics() {
		return new DescriptiveStatisticsHistogramStatistics(this.descriptiveStatistics);
	}
}

{'fraction_non_alphanumeric': 0.07509881422924901, 'fraction_numerical': 0.005187747035573123, 'mean_word_length': 5.0887218045112785, 'pattern_counts': {'":': 0, '<': 3, '<?xml version=': 0, '>': 3, 'https://': 0, 'lorem ipsum': 0, 'www.': 2, 'xml': 0}, 'pii_count': 0, 'substrings_counts': 0, 'word_list_counts': {'cursed_substrings.json': 0, 'profanity_word_list.json': 0, 'sexual_word_list.json': 0, 'zh_pornsignals.json': 0}}
{'dir': 'git-commits-cleaned', 'id': '1896204', 'n_tokens_mistral': 1107, 'n_tokens_neox': 1032, 'n_words': 373}