[ https://issues.apache.org/jira/browse/HIVE-22188?focusedWorklogId=310767&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-310767 ]
ASF GitHub Bot logged work on HIVE-22188: ----------------------------------------- Author: ASF GitHub Bot Created on: 11/Sep/19 15:59 Start Date: 11/Sep/19 15:59 Worklog Time Spent: 10m Work Description: miklosgergely commented on pull request #768: HIVE-22188 Clean up the SemanticAnalyzerFactory URL: https://github.com/apache/hive/pull/768 SemanticAnalyzerFactory contains some huge maps that assign tokentypes to hive operations. Some operations share a token type for doing them on a table, or on a partition, which leads to some more overcomplication. Two things need to be done: 1. Each operation must have it's own token. 2. The HiveOperation enum constants themselves must declare which token they are related to, and not some external source should declare information about them. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org Issue Time Tracking ------------------- Worklog Id: (was: 310767) Remaining Estimate: 0h Time Spent: 10m > Clean up the SemanticAnalyzerFactory > ------------------------------------ > > Key: HIVE-22188 > URL: https://issues.apache.org/jira/browse/HIVE-22188 > Project: Hive > Issue Type: Sub-task > Components: Hive > Reporter: Miklos Gergely > Assignee: Miklos Gergely > Priority: Major > Labels: pull-request-available, refactor-semanticanalyzer > Fix For: 4.0.0 > > Attachments: HIVE-22188.01.patch > > Time Spent: 10m > Remaining Estimate: 0h > > SemanticAnalyzerFactory contains some huge maps that assign tokentypes to > hive operations. Some operations share a token type for doing them on a > table, or on a partition, which leads to some more overcomplication. > Two things need to be done: > # Each operation must have it's own token. > # The HiveOperation enum constants themselves must declare which token they > are related to, and not some external source should declare information about > them. -- This message was sent by Atlassian Jira (v8.3.2#803003)