yujunjun/OpenMetadata
1
0
Fork 0
mirror of https://github.com/open-metadata/OpenMetadata.git synced 2025-08-25 17:37:57 +00:00
Code Issues Packages Projects Releases Wiki Activity

Hotfix to the Term Aggregation size on Data Insights (#17822)

Browse Source
This commit is contained in:
IceS2 2024-09-12 16:50:04 +02:00 committed by Pablo Takara
parent 69fcd6a572
commit adbe59d889
No known key found for this signature in database
GPG Key ID: 63381DDFBB2BF725
3 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
openmetadata-service/src/main/java/org/openmetadata/service/jdbi3/DataInsightSystemChartRepository.java
View File

@ -21,7 +21,7 @@ public class DataInsightSystemChartRepository extends EntityRepository<DataInsig
public static final String DI_SEARCH_INDEX = "di-data-assets-*";
public static final String FORMULA_FUNC_REGEX =
"\\b(count|sum|min|max|avg)+\\((k='([^']*)')?,?\\s*(q='([^']*)')?\\)?";
"\\b(count|sum|min|max|avg|unique)+\\((k='([^']*)')?,?\\s*(q='([^']*)')?\\)?";
public static final String NUMERIC_VALIDATION_REGEX = "[\\d\\.+-\\/\\*\\(\\)\s]+";

2
openmetadata-service/src/main/java/org/openmetadata/service/search/elasticsearch/dataInsightAggregators/ElasticSearchLineChartAggregator.java
View File

@ -60,7 +60,7 @@ public class ElasticSearchLineChartAggregator
excludeArr = lineChart.getExcludeGroups().toArray(new String[0]);
}
TermsAggregationBuilder termsAggregationBuilder =
AggregationBuilders.terms("0").field(lineChart.getGroupBy()).size(20);
AggregationBuilders.terms("0").field(lineChart.getGroupBy()).size(1000);
termsAggregationBuilder.subAggregation(dateHistogramAggregationBuilder);
if (includeArr != null || excludeArr != null) {
IncludeExclude includeExclude = new IncludeExclude(includeArr, excludeArr);

2
openmetadata-service/src/main/java/org/openmetadata/service/search/opensearch/dataInsightAggregator/OpenSearchLineChartAggregator.java
View File

@ -59,7 +59,7 @@ public class OpenSearchLineChartAggregator implements OpenSearchDynamicChartAggr
excludeArr = lineChart.getExcludeGroups().toArray(new String[0]);
}
TermsAggregationBuilder termsAggregationBuilder =
AggregationBuilders.terms("0").field(lineChart.getGroupBy()).size(20);
AggregationBuilders.terms("0").field(lineChart.getGroupBy()).size(1000);
termsAggregationBuilder.subAggregation(dateHistogramAggregationBuilder);
if (includeArr != null || excludeArr != null) {
IncludeExclude includeExclude = new IncludeExclude(includeArr, excludeArr);