hive-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Hive QA (Jira)" <>
Subject [jira] [Commented] (HIVE-22143) Break up DDLSemanticAnalyzer - extract Database related analyzers
Date Thu, 29 Aug 2019 11:45:00 GMT


Hive QA commented on HIVE-22143:

Here are the results of testing the latest attachment:

{color:red}ERROR:{color} -1 due to build exiting with an error

Test results:
Console output:
Test logs:

Executing org.apache.hive.ptest.execution.TestCheckPhase
Tests exited with: Exception: Patch URL
was found in seen patch url's cache and a test was probably run already on it. Aborting...

This message is automatically generated.

ATTACHMENT ID: 12978824 - PreCommit-HIVE-Build

> Break up DDLSemanticAnalyzer - extract Database related analyzers
> -----------------------------------------------------------------
>                 Key: HIVE-22143
>                 URL:
>             Project: Hive
>          Issue Type: Sub-task
>            Reporter: Miklos Gergely
>            Assignee: Miklos Gergely
>            Priority: Major
>              Labels: pull-request-available, refactor-ddl
>         Attachments: HIVE-22143.01.patch, HIVE-22143.02.patch, HIVE-22143.04.patch
>          Time Spent: 10m
>  Remaining Estimate: 0h
> DDLSemanticAnalyzer is a huge class, more than 4000 lines long. The goal is to refactor
it in order to have everything cut into more handleable classes under the packageĀ  org.apache.hadoop.hive.ql.exec.ddl:
>  * have a separate class for each analyzers
>  * have a package for each operation, containing an analyzer, a description, and an operation,
so the amount of classes under a package is more manageable
> Step #1: extract all the database related analyzers from DDLSemanticAnalyzer, and move
them under the new package. Also create the new internal framework.

This message was sent by Atlassian Jira

View raw message