You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@gobblin.apache.org by "ASF GitHub Bot (Jira)" <ji...@apache.org> on 2023/02/01 23:48:00 UTC

[jira] [Work logged] (GOBBLIN-1774) Add utility method for GMIP, MR compaction, and Incremental to skip override hive schema on topics with complex union

     [ https://issues.apache.org/jira/browse/GOBBLIN-1774?focusedWorklogId=843034&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-843034 ]

ASF GitHub Bot logged work on GOBBLIN-1774:
-------------------------------------------

                Author: ASF GitHub Bot
            Created on: 01/Feb/23 23:47
            Start Date: 01/Feb/23 23:47
    Worklog Time Spent: 10m 
      Work Description: umustafi commented on code in PR #3632:
URL: https://github.com/apache/gobblin/pull/3632#discussion_r1093855867


##########
gobblin-hive-registration/src/test/java/org/apache/gobblin/hive/metastore/HiveMetaStoreUtilsTest.java:
##########
@@ -212,4 +217,84 @@ public void testGetHiveTable() throws Exception {
     Assert.assertEquals(fieldA.getType(), "int");
 
   }
+
+  @Test
+  public void testContainsUnionType_AvroSucceeds() {
+    final String databaseName = "db";
+    final String tableName = "tbl";
+    final State serdeProps = new State();
+
+    serdeProps.setProp("avro.schema.literal", "{\"type\": \"record\", \"name\": \"TestEvent\","
+        + " \"namespace\": \"test.namespace\", \"fields\": [{\"name\":\"aUnionField\", \"type\": [\"string\", \"int\"]}]}");
+    HiveTable hiveTable = createTestHiveTable_Avro(databaseName, tableName, serdeProps);
+
+    Assert.assertTrue(HiveMetaStoreUtils.containsUnionTypeColumn(hiveTable));

Review Comment:
   in the function above it looks like its looking for a `uniontype` column name and that's not here although i do see `aUnionField` has multiple options for type.





Issue Time Tracking
-------------------

    Worklog Id:     (was: 843034)
    Time Spent: 0.5h  (was: 20m)

> Add utility method for GMIP, MR compaction, and Incremental to skip override hive schema on topics with complex union
> ---------------------------------------------------------------------------------------------------------------------
>
>                 Key: GOBBLIN-1774
>                 URL: https://issues.apache.org/jira/browse/GOBBLIN-1774
>             Project: Apache Gobblin
>          Issue Type: Improvement
>            Reporter: Matthew Ho
>            Priority: Major
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
>




--
This message was sent by Atlassian Jira
(v8.20.10#820010)