[ 
https://issues.apache.org/jira/browse/HIVE-123?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13197320#comment-13197320
 ] 

Phabricator commented on HIVE-123:
----------------------------------

hobbymanyp has commented on the revision "HIVE-123 [jira] refactor DDL code 
(both DDLWork and DDLTask)".

  According to the steps, I think i am using HIVE-123.  Should I change it to 
"arc diff --jira HIVE-123x" (instead of  "arc diff --jira HIVE-123") ?

  https://cwiki.apache.org/Hive/phabricatorcodereview.html
  From a git checkout (e.g. from git clone git://github.com/apache/hive.git), a 
typical usage sequence is as follows:

  ant arc-setup
  git checkout -b HIVE-123-dev-branch
  edit some files to implement HIVE-123
  git commit -a -m "initial description of how I have gone about making DDL 
code suck less"
  arc diff --jira HIVE-123
  optionally, visit diff URL provided by Phabricator to do things like add 
reviewers and CC's


REVISION DETAIL
  https://reviews.facebook.net/D1485

                
> refactor DDL code (both DDLWork and DDLTask)
> --------------------------------------------
>
>                 Key: HIVE-123
>                 URL: https://issues.apache.org/jira/browse/HIVE-123
>             Project: Hive
>          Issue Type: Bug
>          Components: Query Processor
>            Reporter: Namit Jain
>            Assignee: Ashutosh Chauhan
>              Labels: cleanup, refactoring
>         Attachments: HIVE-123.D1485.1.patch, hive-123.patch
>
>
> It might be good to break DDLTask into separate tasks. The abstract class 
> DDLWork can have various subclasses:
> showTablesWork, DescribeTableWork etc. and a separate task for each of them. 
> This will make them completely independent.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to