[ 
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)

Reply via email to