[ https://issues.apache.org/jira/browse/HIVE-26277?focusedWorklogId=784258&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-784258 ]
ASF GitHub Bot logged work on HIVE-26277: ----------------------------------------- Author: ASF GitHub Bot Created on: 23/Jun/22 15:04 Start Date: 23/Jun/22 15:04 Worklog Time Spent: 10m Work Description: asolimando commented on code in PR #3339: URL: https://github.com/apache/hive/pull/3339#discussion_r905140408 ########## standalone-metastore/metastore-server/src/test/java/org/apache/hadoop/hive/metastore/columnstats/aggr/ColumnStatsAggregatorTest.java: ########## @@ -0,0 +1,120 @@ +/* + * 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.hadoop.hive.metastore.columnstats.aggr; + +import org.apache.hadoop.hive.metastore.TableType; +import org.apache.hadoop.hive.metastore.annotation.MetastoreUnitTest; +import org.apache.hadoop.hive.metastore.api.ColumnStatisticsData; +import org.apache.hadoop.hive.metastore.api.ColumnStatisticsObj; +import org.apache.hadoop.hive.metastore.api.Date; +import org.apache.hadoop.hive.metastore.api.FieldSchema; +import org.apache.hadoop.hive.metastore.api.MetaException; +import org.apache.hadoop.hive.metastore.api.Table; +import org.apache.hadoop.hive.metastore.columnstats.ColStatsBuilder; +import org.apache.hadoop.hive.metastore.utils.MetaStoreServerUtils; + +import org.junit.Assert; +import org.junit.Test; +import org.junit.experimental.categories.Category; +import org.junit.runner.RunWith; +import org.junit.runners.Parameterized; + +import java.util.ArrayList; +import java.util.Collections; +import java.util.List; +import java.util.stream.Collectors; + +import static org.apache.hadoop.hive.metastore.StatisticsTestUtils.createStatsWithInfo; + +@Category(MetastoreUnitTest.class) +@RunWith(Parameterized.class) +public class ColumnStatsAggregatorTest { Review Comment: As discussed offline, we are at around 100 test cases (splitting those with multiple asserts, as needed here), it's pretty hard to find the data that led to a particular test failure, a manual id would be hard to maintain (imagine we want to insert a test in the middle), auto-generated id would help a bit but the number of tests is still high, test mnemonics string to group them suffer from the same issue. For now we will leave the tests as-is. Issue Time Tracking ------------------- Worklog Id: (was: 784258) Time Spent: 3h 20m (was: 3h 10m) > NPEs and rounding issues in ColumnStatsAggregator classes > --------------------------------------------------------- > > Key: HIVE-26277 > URL: https://issues.apache.org/jira/browse/HIVE-26277 > Project: Hive > Issue Type: Bug > Components: Standalone Metastore, Statistics, Tests > Affects Versions: 4.0.0-alpha-2 > Reporter: Alessandro Solimando > Assignee: Alessandro Solimando > Priority: Major > Labels: pull-request-available > Time Spent: 3h 20m > Remaining Estimate: 0h > > Fix NPEs and rounding errors in _ColumnStatsAggregator_ classes, add > unit-tests for all the involved classes. -- This message was sent by Atlassian Jira (v8.20.7#820007)