Re: Review Request: HIVE-1078: CREATE VIEW followup: CREATE OR REPLACE

2011-07-18 Thread Charles Chen

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1058/
---

(Updated 2011-07-18 07:01:02.376155)


Review request for hive.


Changes
---

Added back check: New view must contain all old partition columns


Summary
---

https://issues.apache.org/jira/browse/HIVE-1078


This addresses bug HIVE-1078.
https://issues.apache.org/jira/browse/HIVE-1078


Diffs (updated)
-

  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_view.q.out
 1146902 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_or_replace_view.q.out
 PRE-CREATION 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view4.q.out
 PRE-CREATION 
  
http://svn.apache.org/repos/asf/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java
 1146902 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
 1146902 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/Hive.g
 1146902 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
 1146902 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/CreateViewDesc.java
 1146902 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view1.q
 PRE-CREATION 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view2.q
 PRE-CREATION 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view3.q
 PRE-CREATION 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view4.q
 PRE-CREATION 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientpositive/create_or_replace_view.q
 PRE-CREATION 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view1.q.out
 PRE-CREATION 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view2.q.out
 PRE-CREATION 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view3.q.out
 PRE-CREATION 

Diff: https://reviews.apache.org/r/1058/diff


Testing
---

Passes unit tests


Thanks,

Charles



Re: Review Request: HIVE-1078: CREATE VIEW followup: CREATE OR REPLACE

2011-07-18 Thread Charles Chen

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1058/#review1088
---



http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
https://reviews.apache.org/r/1058/#comment

Haha ok so now I remember why we need this:

(unit test clientnegative/create_or_replace_view1.q):
-- New view must contain all old partition columns.

drop view v;
create view v partitioned on (ds, hr) as select * from srcpart;
create or replace view v as select key, hr from srcpart;

Here, we don't explicitly change the partition columns, so the new view 
inherits the old columns, but those columns better still exist.


- Charles


On 2011-07-18 07:01:02, Charles Chen wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/1058/
 ---
 
 (Updated 2011-07-18 07:01:02)
 
 
 Review request for hive.
 
 
 Summary
 ---
 
 https://issues.apache.org/jira/browse/HIVE-1078
 
 
 This addresses bug HIVE-1078.
 https://issues.apache.org/jira/browse/HIVE-1078
 
 
 Diffs
 -
 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_view.q.out
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_or_replace_view.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view4.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/Hive.g
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/CreateViewDesc.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view1.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view2.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view3.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view4.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientpositive/create_or_replace_view.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view1.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view2.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view3.q.out
  PRE-CREATION 
 
 Diff: https://reviews.apache.org/r/1058/diff
 
 
 Testing
 ---
 
 Passes unit tests
 
 
 Thanks,
 
 Charles
 




Re: Review Request: HIVE-1078: CREATE VIEW followup: CREATE OR REPLACE

2011-07-18 Thread Charles Chen

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1058/#review1089
---



http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
https://reviews.apache.org/r/1058/#comment2223

Oops the comment should be here


- Charles


On 2011-07-18 07:01:02, Charles Chen wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/1058/
 ---
 
 (Updated 2011-07-18 07:01:02)
 
 
 Review request for hive.
 
 
 Summary
 ---
 
 https://issues.apache.org/jira/browse/HIVE-1078
 
 
 This addresses bug HIVE-1078.
 https://issues.apache.org/jira/browse/HIVE-1078
 
 
 Diffs
 -
 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_view.q.out
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_or_replace_view.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view4.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/Hive.g
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/CreateViewDesc.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view1.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view2.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view3.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view4.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientpositive/create_or_replace_view.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view1.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view2.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view3.q.out
  PRE-CREATION 
 
 Diff: https://reviews.apache.org/r/1058/diff
 
 
 Testing
 ---
 
 Passes unit tests
 
 
 Thanks,
 
 Charles
 




Re: Review Request: HIVE-1078: CREATE VIEW followup: CREATE OR REPLACE

2011-07-18 Thread Charles Chen

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1058/#review1090
---



http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
https://reviews.apache.org/r/1058/#comment2224

Here ***


- Charles


On 2011-07-18 07:01:02, Charles Chen wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/1058/
 ---
 
 (Updated 2011-07-18 07:01:02)
 
 
 Review request for hive.
 
 
 Summary
 ---
 
 https://issues.apache.org/jira/browse/HIVE-1078
 
 
 This addresses bug HIVE-1078.
 https://issues.apache.org/jira/browse/HIVE-1078
 
 
 Diffs
 -
 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_view.q.out
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_or_replace_view.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view4.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/Hive.g
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/CreateViewDesc.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view1.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view2.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view3.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view4.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientpositive/create_or_replace_view.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view1.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view2.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view3.q.out
  PRE-CREATION 
 
 Diff: https://reviews.apache.org/r/1058/diff
 
 
 Testing
 ---
 
 Passes unit tests
 
 
 Thanks,
 
 Charles
 




[jira] [Commented] (HIVE-1078) CREATE VIEW followup: CREATE OR REPLACE

2011-07-18 Thread jirapos...@reviews.apache.org (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-1078?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13066820#comment-13066820
 ] 

jirapos...@reviews.apache.org commented on HIVE-1078:
-


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1058/
---

(Updated 2011-07-18 07:01:02.376155)


Review request for hive.


Changes
---

Added back check: New view must contain all old partition columns


Summary
---

https://issues.apache.org/jira/browse/HIVE-1078


This addresses bug HIVE-1078.
https://issues.apache.org/jira/browse/HIVE-1078


Diffs (updated)
-

  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_view.q.out
 1146902 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_or_replace_view.q.out
 PRE-CREATION 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view4.q.out
 PRE-CREATION 
  
http://svn.apache.org/repos/asf/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java
 1146902 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
 1146902 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/Hive.g
 1146902 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
 1146902 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/CreateViewDesc.java
 1146902 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view1.q
 PRE-CREATION 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view2.q
 PRE-CREATION 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view3.q
 PRE-CREATION 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view4.q
 PRE-CREATION 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientpositive/create_or_replace_view.q
 PRE-CREATION 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view1.q.out
 PRE-CREATION 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view2.q.out
 PRE-CREATION 
  
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view3.q.out
 PRE-CREATION 

Diff: https://reviews.apache.org/r/1058/diff


Testing
---

Passes unit tests


Thanks,

Charles



 CREATE VIEW followup:  CREATE OR REPLACE
 

 Key: HIVE-1078
 URL: https://issues.apache.org/jira/browse/HIVE-1078
 Project: Hive
  Issue Type: Improvement
  Components: Query Processor
Affects Versions: 0.6.0
Reporter: John Sichi
Assignee: Charles Chen
 Attachments: HIVE-1078v3.patch, HIVE-1078v4.patch, HIVE-1078v5.patch


 Currently, replacing a view requires
 DROP VIEW v;
 CREATE VIEW v AS new-definition;
 CREATE OR REPLACE would allow these to be combined into a single operation.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HIVE-975) Hive ODBC driver for Windows

2011-07-18 Thread Guy Doulberg (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-975?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13066816#comment-13066816
 ] 

Guy Doulberg commented on HIVE-975:
---

What is the status on this topic?
Has someone found a workaround?

Can I help somehow?

 Hive ODBC driver for Windows
 

 Key: HIVE-975
 URL: https://issues.apache.org/jira/browse/HIVE-975
 Project: Hive
  Issue Type: New Feature
  Components: Clients
 Environment: Windows
Reporter: Carl Steinbach

 The current Hive ODBC driver (HIVE-187) is limited to *NIX systems. We need 
 to provide an ODBC driver that is compatible with Windows and the Windows 
 ODBC driver manager.
 It appears that it may be possible to build the current unixODBC driver on 
 Windows:
 http://mailman.unixodbc.org/pipermail/unixodbc-support/2008-June/001779.html
 The other blocker is THRIFT-591: Windows compatible Thrift C++ runtime.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HIVE-1078) CREATE VIEW followup: CREATE OR REPLACE

2011-07-18 Thread jirapos...@reviews.apache.org (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-1078?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13066818#comment-13066818
 ] 

jirapos...@reviews.apache.org commented on HIVE-1078:
-


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1058/#review1089
---



http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
https://reviews.apache.org/r/1058/#comment2223

Oops the comment should be here


- Charles


On 2011-07-18 07:01:02, Charles Chen wrote:
bq.  
bq.  ---
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/1058/
bq.  ---
bq.  
bq.  (Updated 2011-07-18 07:01:02)
bq.  
bq.  
bq.  Review request for hive.
bq.  
bq.  
bq.  Summary
bq.  ---
bq.  
bq.  https://issues.apache.org/jira/browse/HIVE-1078
bq.  
bq.  
bq.  This addresses bug HIVE-1078.
bq.  https://issues.apache.org/jira/browse/HIVE-1078
bq.  
bq.  
bq.  Diffs
bq.  -
bq.  
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_view.q.out
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_or_replace_view.q.out
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view4.q.out
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/Hive.g
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/CreateViewDesc.java
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view1.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view2.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view3.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view4.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientpositive/create_or_replace_view.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view1.q.out
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view2.q.out
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view3.q.out
 PRE-CREATION 
bq.  
bq.  Diff: https://reviews.apache.org/r/1058/diff
bq.  
bq.  
bq.  Testing
bq.  ---
bq.  
bq.  Passes unit tests
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Charles
bq.  
bq.



 CREATE VIEW followup:  CREATE OR REPLACE
 

 Key: HIVE-1078
 URL: https://issues.apache.org/jira/browse/HIVE-1078
 Project: Hive
  Issue Type: Improvement
  Components: Query Processor
Affects Versions: 0.6.0
Reporter: John Sichi
Assignee: Charles Chen
 Attachments: HIVE-1078v3.patch, HIVE-1078v4.patch, HIVE-1078v5.patch


 Currently, replacing a view requires
 DROP VIEW v;
 CREATE VIEW v AS new-definition;
 CREATE OR REPLACE would allow these to be combined into a single operation.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




Re: Review Request: HIVE-1078: CREATE VIEW followup: CREATE OR REPLACE

2011-07-18 Thread Charles Chen

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1058/#review1091
---



http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
https://reviews.apache.org/r/1058/#comment2225

Here ***


- Charles


On 2011-07-18 07:01:02, Charles Chen wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/1058/
 ---
 
 (Updated 2011-07-18 07:01:02)
 
 
 Review request for hive.
 
 
 Summary
 ---
 
 https://issues.apache.org/jira/browse/HIVE-1078
 
 
 This addresses bug HIVE-1078.
 https://issues.apache.org/jira/browse/HIVE-1078
 
 
 Diffs
 -
 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_view.q.out
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_or_replace_view.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view4.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/Hive.g
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/CreateViewDesc.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view1.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view2.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view3.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view4.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientpositive/create_or_replace_view.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view1.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view2.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view3.q.out
  PRE-CREATION 
 
 Diff: https://reviews.apache.org/r/1058/diff
 
 
 Testing
 ---
 
 Passes unit tests
 
 
 Thanks,
 
 Charles
 




[jira] [Commented] (HIVE-1078) CREATE VIEW followup: CREATE OR REPLACE

2011-07-18 Thread jirapos...@reviews.apache.org (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-1078?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13066822#comment-13066822
 ] 

jirapos...@reviews.apache.org commented on HIVE-1078:
-


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1058/#review1091
---



http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
https://reviews.apache.org/r/1058/#comment2225

Here ***


- Charles


On 2011-07-18 07:01:02, Charles Chen wrote:
bq.  
bq.  ---
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/1058/
bq.  ---
bq.  
bq.  (Updated 2011-07-18 07:01:02)
bq.  
bq.  
bq.  Review request for hive.
bq.  
bq.  
bq.  Summary
bq.  ---
bq.  
bq.  https://issues.apache.org/jira/browse/HIVE-1078
bq.  
bq.  
bq.  This addresses bug HIVE-1078.
bq.  https://issues.apache.org/jira/browse/HIVE-1078
bq.  
bq.  
bq.  Diffs
bq.  -
bq.  
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_view.q.out
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_or_replace_view.q.out
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view4.q.out
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/Hive.g
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/CreateViewDesc.java
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view1.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view2.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view3.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view4.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientpositive/create_or_replace_view.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view1.q.out
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view2.q.out
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view3.q.out
 PRE-CREATION 
bq.  
bq.  Diff: https://reviews.apache.org/r/1058/diff
bq.  
bq.  
bq.  Testing
bq.  ---
bq.  
bq.  Passes unit tests
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Charles
bq.  
bq.



 CREATE VIEW followup:  CREATE OR REPLACE
 

 Key: HIVE-1078
 URL: https://issues.apache.org/jira/browse/HIVE-1078
 Project: Hive
  Issue Type: Improvement
  Components: Query Processor
Affects Versions: 0.6.0
Reporter: John Sichi
Assignee: Charles Chen
 Attachments: HIVE-1078v3.patch, HIVE-1078v4.patch, HIVE-1078v5.patch


 Currently, replacing a view requires
 DROP VIEW v;
 CREATE VIEW v AS new-definition;
 CREATE OR REPLACE would allow these to be combined into a single operation.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HIVE-1078) CREATE VIEW followup: CREATE OR REPLACE

2011-07-18 Thread jirapos...@reviews.apache.org (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-1078?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13066819#comment-13066819
 ] 

jirapos...@reviews.apache.org commented on HIVE-1078:
-


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1058/#review1088
---



http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
https://reviews.apache.org/r/1058/#comment

Haha ok so now I remember why we need this:

(unit test clientnegative/create_or_replace_view1.q):
-- New view must contain all old partition columns.

drop view v;
create view v partitioned on (ds, hr) as select * from srcpart;
create or replace view v as select key, hr from srcpart;

Here, we don't explicitly change the partition columns, so the new view 
inherits the old columns, but those columns better still exist.


- Charles


On 2011-07-18 07:01:02, Charles Chen wrote:
bq.  
bq.  ---
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/1058/
bq.  ---
bq.  
bq.  (Updated 2011-07-18 07:01:02)
bq.  
bq.  
bq.  Review request for hive.
bq.  
bq.  
bq.  Summary
bq.  ---
bq.  
bq.  https://issues.apache.org/jira/browse/HIVE-1078
bq.  
bq.  
bq.  This addresses bug HIVE-1078.
bq.  https://issues.apache.org/jira/browse/HIVE-1078
bq.  
bq.  
bq.  Diffs
bq.  -
bq.  
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_view.q.out
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_or_replace_view.q.out
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view4.q.out
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/Hive.g
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/CreateViewDesc.java
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view1.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view2.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view3.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view4.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientpositive/create_or_replace_view.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view1.q.out
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view2.q.out
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view3.q.out
 PRE-CREATION 
bq.  
bq.  Diff: https://reviews.apache.org/r/1058/diff
bq.  
bq.  
bq.  Testing
bq.  ---
bq.  
bq.  Passes unit tests
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Charles
bq.  
bq.



 CREATE VIEW followup:  CREATE OR REPLACE
 

 Key: HIVE-1078
 URL: https://issues.apache.org/jira/browse/HIVE-1078
 Project: Hive
  Issue Type: Improvement
  Components: Query Processor
Affects Versions: 0.6.0
Reporter: John Sichi
Assignee: Charles Chen
 Attachments: HIVE-1078v3.patch, HIVE-1078v4.patch, HIVE-1078v5.patch


 Currently, replacing a view requires
 DROP VIEW v;
 CREATE VIEW v AS new-definition;
 CREATE OR REPLACE would allow these to be combined into a single operation.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HIVE-2285) Pretty-print output of DESCRIBE TABLE EXTENDED

2011-07-18 Thread Thiruvel Thirumoolan (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-2285?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13066880#comment-13066880
 ] 

Thiruvel Thirumoolan commented on HIVE-2285:


@carl HIVE-558 formatted the describe output through 'describe formatted ..'. 
Is the intention of this JIRA different?

 Pretty-print output of DESCRIBE TABLE EXTENDED
 --

 Key: HIVE-2285
 URL: https://issues.apache.org/jira/browse/HIVE-2285
 Project: Hive
  Issue Type: Improvement
  Components: CLI
Reporter: Carl Steinbach
Assignee: Carl Steinbach



--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




Re: Review Request: HIVE-1218: CREATE TABLE LIKE view

2011-07-18 Thread Charles Chen

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1057/#review1096
---


r9 passes unit tests

- Charles


On 2011-07-16 22:51:22, Charles Chen wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/1057/
 ---
 
 (Updated 2011-07-16 22:51:22)
 
 
 Review request for hive.
 
 
 Summary
 ---
 
 https://issues.apache.org/jira/browse/HIVE-1218
 
 
 This addresses bug HIVE-1218.
 https://issues.apache.org/jira/browse/HIVE-1218
 
 
 Diffs
 -
 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
  1146883 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
  1146883 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/CreateTableLikeDesc.java
  1146883 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientpositive/create_like.q
  1146883 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientpositive/create_like_view.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_like.q.out
  1146883 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_like_view.q.out
  PRE-CREATION 
 
 Diff: https://reviews.apache.org/r/1057/diff
 
 
 Testing
 ---
 
 Unit tests passed
 
 
 Thanks,
 
 Charles
 




Re: Review Request: HIVE-1078: CREATE VIEW followup: CREATE OR REPLACE

2011-07-18 Thread Charles Chen

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1058/#review1097
---


r7 passes unit tests

- Charles


On 2011-07-18 07:01:02, Charles Chen wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/1058/
 ---
 
 (Updated 2011-07-18 07:01:02)
 
 
 Review request for hive.
 
 
 Summary
 ---
 
 https://issues.apache.org/jira/browse/HIVE-1078
 
 
 This addresses bug HIVE-1078.
 https://issues.apache.org/jira/browse/HIVE-1078
 
 
 Diffs
 -
 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_view.q.out
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_or_replace_view.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view4.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/Hive.g
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/CreateViewDesc.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view1.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view2.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view3.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view4.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientpositive/create_or_replace_view.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view1.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view2.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view3.q.out
  PRE-CREATION 
 
 Diff: https://reviews.apache.org/r/1058/diff
 
 
 Testing
 ---
 
 Passes unit tests
 
 
 Thanks,
 
 Charles
 




[jira] [Commented] (HIVE-1218) CREATE TABLE t LIKE some_view should create a new empty base table, but instead creates a copy of view

2011-07-18 Thread jirapos...@reviews.apache.org (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-1218?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13067159#comment-13067159
 ] 

jirapos...@reviews.apache.org commented on HIVE-1218:
-


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1057/#review1096
---


r9 passes unit tests

- Charles


On 2011-07-16 22:51:22, Charles Chen wrote:
bq.  
bq.  ---
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/1057/
bq.  ---
bq.  
bq.  (Updated 2011-07-16 22:51:22)
bq.  
bq.  
bq.  Review request for hive.
bq.  
bq.  
bq.  Summary
bq.  ---
bq.  
bq.  https://issues.apache.org/jira/browse/HIVE-1218
bq.  
bq.  
bq.  This addresses bug HIVE-1218.
bq.  https://issues.apache.org/jira/browse/HIVE-1218
bq.  
bq.  
bq.  Diffs
bq.  -
bq.  
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
 1146883 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
 1146883 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/CreateTableLikeDesc.java
 1146883 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientpositive/create_like.q
 1146883 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientpositive/create_like_view.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_like.q.out
 1146883 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_like_view.q.out
 PRE-CREATION 
bq.  
bq.  Diff: https://reviews.apache.org/r/1057/diff
bq.  
bq.  
bq.  Testing
bq.  ---
bq.  
bq.  Unit tests passed
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Charles
bq.  
bq.



 CREATE TABLE t LIKE some_view should create a new empty base table, but 
 instead creates a copy of view
 --

 Key: HIVE-1218
 URL: https://issues.apache.org/jira/browse/HIVE-1218
 Project: Hive
  Issue Type: Bug
  Components: Metastore
Affects Versions: 0.6.0
Reporter: John Sichi
Assignee: Charles Chen
 Fix For: 0.8.0

 Attachments: HIVE-1218v0.patch, HIVE-1218v2.patch, HIVE-1218v3.patch, 
 HIVE-1218v4.patch, HIVE-1218v5.patch, HIVE-1218v6.patch, HIVE-1218v7.patch, 
 HIVE-1218v8.patch


 I think it should copy only the column definitions from the view metadata.  
 Currently it is copying the entire descriptor, resulting in a new view 
 instead of a new base table.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Updated] (HIVE-1078) CREATE VIEW followup: CREATE OR REPLACE

2011-07-18 Thread Charles Chen (JIRA)

 [ 
https://issues.apache.org/jira/browse/HIVE-1078?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Charles Chen updated HIVE-1078:
---

Attachment: HIVE-1078v6.patch

 CREATE VIEW followup:  CREATE OR REPLACE
 

 Key: HIVE-1078
 URL: https://issues.apache.org/jira/browse/HIVE-1078
 Project: Hive
  Issue Type: Improvement
  Components: Query Processor
Affects Versions: 0.6.0
Reporter: John Sichi
Assignee: Charles Chen
 Attachments: HIVE-1078v3.patch, HIVE-1078v4.patch, HIVE-1078v5.patch, 
 HIVE-1078v6.patch


 Currently, replacing a view requires
 DROP VIEW v;
 CREATE VIEW v AS new-definition;
 CREATE OR REPLACE would allow these to be combined into a single operation.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HIVE-1078) CREATE VIEW followup: CREATE OR REPLACE

2011-07-18 Thread jirapos...@reviews.apache.org (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-1078?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13067160#comment-13067160
 ] 

jirapos...@reviews.apache.org commented on HIVE-1078:
-


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1058/#review1097
---


r7 passes unit tests

- Charles


On 2011-07-18 07:01:02, Charles Chen wrote:
bq.  
bq.  ---
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/1058/
bq.  ---
bq.  
bq.  (Updated 2011-07-18 07:01:02)
bq.  
bq.  
bq.  Review request for hive.
bq.  
bq.  
bq.  Summary
bq.  ---
bq.  
bq.  https://issues.apache.org/jira/browse/HIVE-1078
bq.  
bq.  
bq.  This addresses bug HIVE-1078.
bq.  https://issues.apache.org/jira/browse/HIVE-1078
bq.  
bq.  
bq.  Diffs
bq.  -
bq.  
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_view.q.out
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_or_replace_view.q.out
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view4.q.out
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/Hive.g
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/CreateViewDesc.java
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view1.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view2.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view3.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view4.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientpositive/create_or_replace_view.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view1.q.out
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view2.q.out
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view3.q.out
 PRE-CREATION 
bq.  
bq.  Diff: https://reviews.apache.org/r/1058/diff
bq.  
bq.  
bq.  Testing
bq.  ---
bq.  
bq.  Passes unit tests
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Charles
bq.  
bq.



 CREATE VIEW followup:  CREATE OR REPLACE
 

 Key: HIVE-1078
 URL: https://issues.apache.org/jira/browse/HIVE-1078
 Project: Hive
  Issue Type: Improvement
  Components: Query Processor
Affects Versions: 0.6.0
Reporter: John Sichi
Assignee: Charles Chen
 Attachments: HIVE-1078v3.patch, HIVE-1078v4.patch, HIVE-1078v5.patch, 
 HIVE-1078v6.patch


 Currently, replacing a view requires
 DROP VIEW v;
 CREATE VIEW v AS new-definition;
 CREATE OR REPLACE would allow these to be combined into a single operation.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Updated] (HIVE-2184) Few improvements in org.apache.hadoop.hive.ql.metadata.Hive.close()

2011-07-18 Thread John Sichi (JIRA)

 [ 
https://issues.apache.org/jira/browse/HIVE-2184?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

John Sichi updated HIVE-2184:
-

Status: Open  (was: Patch Available)

Comments added on review board

 Few improvements in org.apache.hadoop.hive.ql.metadata.Hive.close()
 ---

 Key: HIVE-2184
 URL: https://issues.apache.org/jira/browse/HIVE-2184
 Project: Hive
  Issue Type: Bug
  Components: Metastore
Affects Versions: 0.5.0, 0.8.0
 Environment: Hadoop 0.20.1, Hive0.8.0 and SUSE Linux Enterprise 
 Server 10 SP2 (i586) - Kernel 2.6.16.60-0.21-smp (5)
Reporter: Chinna Rao Lalam
Assignee: Chinna Rao Lalam
 Attachments: HIVE-2184.1.patch, HIVE-2184.1.patch, HIVE-2184.patch


 1)Hive.close() will call HiveMetaStoreClient.close() in this method the 
 variable standAloneClient is never become true then client.shutdown() never 
 call.
 2)Hive.close() After calling metaStoreClient.close() need to make 
 metaStoreClient=null

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HIVE-2224) Ability to add_partitions, and atomically

2011-07-18 Thread Paul Yang (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-2224?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13067209#comment-13067209
 ] 

Paul Yang commented on HIVE-2224:
-

Sorry for the delay, but I've running into some test issues that are likely not 
caused by your patch.

 Ability to add_partitions, and atomically
 -

 Key: HIVE-2224
 URL: https://issues.apache.org/jira/browse/HIVE-2224
 Project: Hive
  Issue Type: Improvement
  Components: Metastore
Reporter: Sushanth Sowmyan
Assignee: Sushanth Sowmyan
 Attachments: HIVE-2224.patch


 I'd like to see an atomic version of the add_partitions() call.
 Whether this is to be done by config to affect add_partitions() behaviour 
 (not my preference) or just changing add_partitions() default behaviour (my 
 preference, but likely to affect current behaviour, so will need others' 
 input) or by making a new add_partitions_atomic() call depends on discussion.
 This looks relatively doable to implement (will need a dependent 
 add_partition_core to not do a ms.commit_partition() early, and to cache list 
 of directories created to remove on rollback, and a list of AddPartitionEvent 
 to trigger in one shot later)
 Thoughts? This also seems like something to implement for allowing HIVE-1805.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Updated] (HIVE-1218) CREATE TABLE t LIKE some_view should create a new empty base table, but instead creates a copy of view

2011-07-18 Thread John Sichi (JIRA)

 [ 
https://issues.apache.org/jira/browse/HIVE-1218?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

John Sichi updated HIVE-1218:
-

Resolution: Fixed
Status: Resolved  (was: Patch Available)

Committed.  Thanks Charles!


 CREATE TABLE t LIKE some_view should create a new empty base table, but 
 instead creates a copy of view
 --

 Key: HIVE-1218
 URL: https://issues.apache.org/jira/browse/HIVE-1218
 Project: Hive
  Issue Type: Bug
  Components: Metastore
Affects Versions: 0.6.0
Reporter: John Sichi
Assignee: Charles Chen
 Fix For: 0.8.0

 Attachments: HIVE-1218v0.patch, HIVE-1218v2.patch, HIVE-1218v3.patch, 
 HIVE-1218v4.patch, HIVE-1218v5.patch, HIVE-1218v6.patch, HIVE-1218v7.patch, 
 HIVE-1218v8.patch


 I think it should copy only the column definitions from the view metadata.  
 Currently it is copying the entire descriptor, resulting in a new view 
 instead of a new base table.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




Re: Review Request: HIVE-2272: add TIMESTAMP data type

2011-07-18 Thread Franklin Hu

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1135/
---

(Updated 2011-07-18 19:12:08.745755)


Review request for hive.


Summary (updated)
---

Adds TIMESTAMP type to serde2 with both string (LazySimple) and binary 
(LazyBinary) serialization.
Supports SQL style jdbc timestamps of the format with nanosecond precision
-MM-DD HH:MM:SS[.fff...]


This addresses bug HIVE-2272.
https://issues.apache.org/jira/browse/HIVE-2272


Diffs
-

  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/PrimitiveObjectInspectorFactory.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/PrimitiveObjectInspectorUtils.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/SettableTimestampObjectInspector.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/TimestampObjectInspector.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/WritableTimestampObjectInspector.java
 PRE-CREATION 
  trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/LazyFactory.java 
1146895 
  trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/LazyTimestamp.java 
PRE-CREATION 
  trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/LazyUtils.java 
1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/objectinspector/primitive/LazyPrimitiveObjectInspectorFactory.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/objectinspector/primitive/LazyTimestampObjectInspector.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinaryFactory.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinarySerDe.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinaryTimestamp.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinaryUtils.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/ObjectInspectorConverters.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/ObjectInspectorUtils.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/PrimitiveObjectInspector.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/JavaTimestampObjectInspector.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/PrimitiveObjectInspectorConverter.java
 1146895 
  trunk/ql/src/test/results/clientpositive/timestamp_3.q.out PRE-CREATION 
  trunk/ql/src/test/results/clientpositive/timestamp_comparison.q.out 
PRE-CREATION 
  trunk/ql/src/test/results/clientpositive/timestamp_udf.q.out PRE-CREATION 
  trunk/serde/src/java/org/apache/hadoop/hive/serde2/SerDeUtils.java 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/binarysortable/BinarySortableSerDe.java
 1146895 
  trunk/serde/src/java/org/apache/hadoop/hive/serde2/io/TimestampWritable.java 
PRE-CREATION 
  trunk/ql/src/test/results/clientpositive/timestamp_2.q.out PRE-CREATION 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFToFloat.java 1146895 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFToInteger.java 1146895 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFToLong.java 1146895 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFToShort.java 1146895 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFToString.java 1146895 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFUnixTimeStamp.java 1146895 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFWeekOfYear.java 1146895 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFYear.java 1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFAverage.java 
1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFContextNGrams.java
 1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFCorrelation.java
 1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFCovariance.java
 1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFCovarianceSample.java
 1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFHistogramNumeric.java
 1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFPercentileApprox.java
 1146895 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFStd.java 
1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFStdSample.java
 1146895 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFSum.java 
1146895 
  

Re: Review Request: HIVE-2272: add TIMESTAMP data type

2011-07-18 Thread Franklin Hu

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1135/
---

(Updated 2011-07-18 19:13:31.962359)


Review request for hive.


Summary
---

Adds TIMESTAMP type to serde2 with both string (LazySimple) and binary 
(LazyBinary) serialization.
Supports SQL style jdbc timestamps of the format with nanosecond precision
-MM-DD HH:MM:SS[.fff...]


This addresses bug HIVE-2272.
https://issues.apache.org/jira/browse/HIVE-2272


Diffs (updated)
-

  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/PrimitiveObjectInspector.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/JavaTimestampObjectInspector.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/PrimitiveObjectInspectorConverter.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/PrimitiveObjectInspectorFactory.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/PrimitiveObjectInspectorUtils.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/SettableTimestampObjectInspector.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/TimestampObjectInspector.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/WritableTimestampObjectInspector.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinarySerDe.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinaryTimestamp.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinaryUtils.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/ObjectInspectorConverters.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/ObjectInspectorUtils.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinaryFactory.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/objectinspector/primitive/LazyTimestampObjectInspector.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/objectinspector/primitive/LazyPrimitiveObjectInspectorFactory.java
 1146895 
  trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/LazyFactory.java 
1146895 
  trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/LazyTimestamp.java 
PRE-CREATION 
  trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/LazyUtils.java 
1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/binarysortable/BinarySortableSerDe.java
 1146895 
  trunk/serde/src/java/org/apache/hadoop/hive/serde2/io/TimestampWritable.java 
PRE-CREATION 
  trunk/ql/src/test/results/clientpositive/timestamp_comparison.q.out 
PRE-CREATION 
  trunk/ql/src/test/results/clientpositive/timestamp_udf.q.out PRE-CREATION 
  trunk/serde/src/java/org/apache/hadoop/hive/serde2/SerDeUtils.java 1146895 
  trunk/ql/src/test/results/clientpositive/timestamp_3.q.out PRE-CREATION 
  trunk/ql/src/test/results/clientpositive/timestamp_2.q.out PRE-CREATION 
  trunk/ql/src/test/results/clientnegative/wrong_column_type.q.out 1146895 
  trunk/ql/src/test/results/clientpositive/show_functions.q.out 1146895 
  trunk/ql/src/test/results/clientpositive/timestamp_1.q.out PRE-CREATION 
  trunk/ql/src/test/results/clientnegative/invalid_t_create2.q.out 1146895 
  trunk/ql/src/test/results/clientnegative/invalid_t_transform.q.out 1146895 
  trunk/ql/src/test/results/clientnegative/invalid_t_alter2.q.out 1146895 
  trunk/ql/src/test/results/clientnegative/invalid_t_create1.q.out 1146895 
  trunk/ql/src/test/queries/clientpositive/timestamp_2.q PRE-CREATION 
  trunk/ql/src/test/queries/clientpositive/timestamp_3.q PRE-CREATION 
  trunk/ql/src/test/queries/clientpositive/timestamp_comparison.q PRE-CREATION 
  trunk/ql/src/test/queries/clientpositive/timestamp_udf.q PRE-CREATION 
  trunk/ql/src/test/results/clientnegative/invalid_create_tbl1.q.out 1146895 
  trunk/ql/src/test/results/clientnegative/invalid_t_alter1.q.out 1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFStdSample.java
 1146895 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFSum.java 
1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFVariance.java
 1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFVarianceSample.java
 1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFnGrams.java 
1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDFTimestamp.java
 PRE-CREATION 
  trunk/ql/src/test/queries/clientnegative/invalid_t_create3.q 1146895 
  

[jira] [Commented] (HIVE-2272) add TIMESTAMP data type

2011-07-18 Thread jirapos...@reviews.apache.org (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-2272?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13067222#comment-13067222
 ] 

jirapos...@reviews.apache.org commented on HIVE-2272:
-


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1135/
---

(Updated 2011-07-18 19:12:08.745755)


Review request for hive.


Summary (updated)
---

Adds TIMESTAMP type to serde2 with both string (LazySimple) and binary 
(LazyBinary) serialization.
Supports SQL style jdbc timestamps of the format with nanosecond precision
-MM-DD HH:MM:SS[.fff...]


This addresses bug HIVE-2272.
https://issues.apache.org/jira/browse/HIVE-2272


Diffs
-

  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/PrimitiveObjectInspectorFactory.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/PrimitiveObjectInspectorUtils.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/SettableTimestampObjectInspector.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/TimestampObjectInspector.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/WritableTimestampObjectInspector.java
 PRE-CREATION 
  trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/LazyFactory.java 
1146895 
  trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/LazyTimestamp.java 
PRE-CREATION 
  trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/LazyUtils.java 
1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/objectinspector/primitive/LazyPrimitiveObjectInspectorFactory.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/objectinspector/primitive/LazyTimestampObjectInspector.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinaryFactory.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinarySerDe.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinaryTimestamp.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinaryUtils.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/ObjectInspectorConverters.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/ObjectInspectorUtils.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/PrimitiveObjectInspector.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/JavaTimestampObjectInspector.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/PrimitiveObjectInspectorConverter.java
 1146895 
  trunk/ql/src/test/results/clientpositive/timestamp_3.q.out PRE-CREATION 
  trunk/ql/src/test/results/clientpositive/timestamp_comparison.q.out 
PRE-CREATION 
  trunk/ql/src/test/results/clientpositive/timestamp_udf.q.out PRE-CREATION 
  trunk/serde/src/java/org/apache/hadoop/hive/serde2/SerDeUtils.java 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/binarysortable/BinarySortableSerDe.java
 1146895 
  trunk/serde/src/java/org/apache/hadoop/hive/serde2/io/TimestampWritable.java 
PRE-CREATION 
  trunk/ql/src/test/results/clientpositive/timestamp_2.q.out PRE-CREATION 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFToFloat.java 1146895 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFToInteger.java 1146895 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFToLong.java 1146895 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFToShort.java 1146895 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFToString.java 1146895 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFUnixTimeStamp.java 1146895 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFWeekOfYear.java 1146895 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFYear.java 1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFAverage.java 
1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFContextNGrams.java
 1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFCorrelation.java
 1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFCovariance.java
 1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFCovarianceSample.java
 1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFHistogramNumeric.java
 1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFPercentileApprox.java
 1146895 
  

[jira] [Commented] (HIVE-2272) add TIMESTAMP data type

2011-07-18 Thread jirapos...@reviews.apache.org (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-2272?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13067223#comment-13067223
 ] 

jirapos...@reviews.apache.org commented on HIVE-2272:
-


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1135/
---

(Updated 2011-07-18 19:13:31.962359)


Review request for hive.


Summary
---

Adds TIMESTAMP type to serde2 with both string (LazySimple) and binary 
(LazyBinary) serialization.
Supports SQL style jdbc timestamps of the format with nanosecond precision
-MM-DD HH:MM:SS[.fff...]


This addresses bug HIVE-2272.
https://issues.apache.org/jira/browse/HIVE-2272


Diffs (updated)
-

  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/PrimitiveObjectInspector.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/JavaTimestampObjectInspector.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/PrimitiveObjectInspectorConverter.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/PrimitiveObjectInspectorFactory.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/PrimitiveObjectInspectorUtils.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/SettableTimestampObjectInspector.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/TimestampObjectInspector.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/primitive/WritableTimestampObjectInspector.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinarySerDe.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinaryTimestamp.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinaryUtils.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/ObjectInspectorConverters.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/objectinspector/ObjectInspectorUtils.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazybinary/LazyBinaryFactory.java
 1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/objectinspector/primitive/LazyTimestampObjectInspector.java
 PRE-CREATION 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/objectinspector/primitive/LazyPrimitiveObjectInspectorFactory.java
 1146895 
  trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/LazyFactory.java 
1146895 
  trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/LazyTimestamp.java 
PRE-CREATION 
  trunk/serde/src/java/org/apache/hadoop/hive/serde2/lazy/LazyUtils.java 
1146895 
  
trunk/serde/src/java/org/apache/hadoop/hive/serde2/binarysortable/BinarySortableSerDe.java
 1146895 
  trunk/serde/src/java/org/apache/hadoop/hive/serde2/io/TimestampWritable.java 
PRE-CREATION 
  trunk/ql/src/test/results/clientpositive/timestamp_comparison.q.out 
PRE-CREATION 
  trunk/ql/src/test/results/clientpositive/timestamp_udf.q.out PRE-CREATION 
  trunk/serde/src/java/org/apache/hadoop/hive/serde2/SerDeUtils.java 1146895 
  trunk/ql/src/test/results/clientpositive/timestamp_3.q.out PRE-CREATION 
  trunk/ql/src/test/results/clientpositive/timestamp_2.q.out PRE-CREATION 
  trunk/ql/src/test/results/clientnegative/wrong_column_type.q.out 1146895 
  trunk/ql/src/test/results/clientpositive/show_functions.q.out 1146895 
  trunk/ql/src/test/results/clientpositive/timestamp_1.q.out PRE-CREATION 
  trunk/ql/src/test/results/clientnegative/invalid_t_create2.q.out 1146895 
  trunk/ql/src/test/results/clientnegative/invalid_t_transform.q.out 1146895 
  trunk/ql/src/test/results/clientnegative/invalid_t_alter2.q.out 1146895 
  trunk/ql/src/test/results/clientnegative/invalid_t_create1.q.out 1146895 
  trunk/ql/src/test/queries/clientpositive/timestamp_2.q PRE-CREATION 
  trunk/ql/src/test/queries/clientpositive/timestamp_3.q PRE-CREATION 
  trunk/ql/src/test/queries/clientpositive/timestamp_comparison.q PRE-CREATION 
  trunk/ql/src/test/queries/clientpositive/timestamp_udf.q PRE-CREATION 
  trunk/ql/src/test/results/clientnegative/invalid_create_tbl1.q.out 1146895 
  trunk/ql/src/test/results/clientnegative/invalid_t_alter1.q.out 1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFStdSample.java
 1146895 
  trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFSum.java 
1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFVariance.java
 1146895 
  
trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/generic/GenericUDAFVarianceSample.java
 1146895 
  

[jira] [Updated] (HIVE-2198) While using Hive in server mode, HiveConnection.close() is not cleaning up server side resources

2011-07-18 Thread John Sichi (JIRA)

 [ 
https://issues.apache.org/jira/browse/HIVE-2198?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

John Sichi updated HIVE-2198:
-

Status: Open  (was: Patch Available)

Changes look fine, but could you fix the indentation?

 While using Hive in server mode, HiveConnection.close() is not cleaning up 
 server side resources
 

 Key: HIVE-2198
 URL: https://issues.apache.org/jira/browse/HIVE-2198
 Project: Hive
  Issue Type: Bug
  Components: Server Infrastructure
Affects Versions: 0.5.0, 0.8.0
 Environment: Hadoop 0.20.1, Hive0.8.0 and SUSE Linux Enterprise 
 Server 10 SP2 (i586) - Kernel 2.6.16.60-0.21-smp (5)
Reporter: Chinna Rao Lalam
Assignee: Chinna Rao Lalam
 Attachments: HIVE-2198.patch


 org.apache.hadoop.hive.service.ThriftHive.Client.clean() method is called for 
 every session end in CLI mode for the cleanup but in HiveServer mode this is 
 not called.
 So this can be integrate with the HiveConnection.close()

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




RE: number of maptasks of hive

2011-07-18 Thread Steven Wong
What input format are you using in Hive? (It should be the one specified by the 
hive.input.format setting.)


-Original Message-
From: 牛兆捷 [mailto:nzjem...@gmail.com] 
Sent: Sunday, July 17, 2011 10:26 PM
To: dev@hive.apache.org
Subject: number of maptasks of hive

Does hive split input table file according the ways hadoop provides which a
file is splitted by block size? Then the number of map tasks is decided by
the split size.

But I have a table file which  size is 196MB,my hdfs block size is 64MB,but
I only see 3 map tasks in my web interface.What is the reason?


Build failed in Jenkins: Hive-trunk-h0.21 #832

2011-07-18 Thread Apache Jenkins Server
See https://builds.apache.org/job/Hive-trunk-h0.21/832/

--
[...truncated 33246 lines...]
[artifact:deploy] Deploying to 
https://repository.apache.org/content/repositories/snapshots
[artifact:deploy] [INFO] Retrieving previous build number from 
apache.snapshots.https
[artifact:deploy] Uploading: 
org/apache/hive/hive-hbase-handler/0.8.0-SNAPSHOT/hive-hbase-handler-0.8.0-20110718.193101-37.jar
 to repository apache.snapshots.https at 
https://repository.apache.org/content/repositories/snapshots
[artifact:deploy] Transferring 49K from apache.snapshots.https
[artifact:deploy] Uploaded 49K
[artifact:deploy] [INFO] Uploading project information for hive-hbase-handler 
0.8.0-20110718.193101-37
[artifact:deploy] [INFO] Retrieving previous metadata from 
apache.snapshots.https
[artifact:deploy] [INFO] Uploading repository metadata for: 'snapshot 
org.apache.hive:hive-hbase-handler:0.8.0-SNAPSHOT'
[artifact:deploy] [INFO] Retrieving previous metadata from 
apache.snapshots.https
[artifact:deploy] [INFO] Uploading repository metadata for: 'artifact 
org.apache.hive:hive-hbase-handler'

ivy-init-dirs:

ivy-download:
  [get] Getting: 
http://repo2.maven.org/maven2/org/apache/ivy/ivy/2.1.0/ivy-2.1.0.jar
  [get] To: 
/x1/jenkins/jenkins-slave/workspace/Hive-trunk-h0.21/hive/build/ivy/lib/ivy-2.1.0.jar
  [get] Not modified - so not downloaded

ivy-probe-antlib:

ivy-init-antlib:

ivy-init:

ivy-resolve-maven-ant-tasks:
[ivy:resolve] :: loading settings :: file = 
/x1/jenkins/jenkins-slave/workspace/Hive-trunk-h0.21/hive/ivy/ivysettings.xml

ivy-retrieve-maven-ant-tasks:
[ivy:cachepath] DEPRECATED: 'ivy.conf.file' is deprecated, use 
'ivy.settings.file' instead
[ivy:cachepath] :: loading settings :: file = 
/x1/jenkins/jenkins-slave/workspace/Hive-trunk-h0.21/hive/ivy/ivysettings.xml

mvn-taskdef:

maven-publish-artifact:
[artifact:install-provider] Installing provider: 
org.apache.maven.wagon:wagon-http:jar:1.0-beta-2:runtime
[artifact:deploy] Deploying to 
https://repository.apache.org/content/repositories/snapshots
[artifact:deploy] [INFO] Retrieving previous build number from 
apache.snapshots.https
[artifact:deploy] Uploading: 
org/apache/hive/hive-hwi/0.8.0-SNAPSHOT/hive-hwi-0.8.0-20110718.193102-37.jar 
to repository apache.snapshots.https at 
https://repository.apache.org/content/repositories/snapshots
[artifact:deploy] Transferring 23K from apache.snapshots.https
[artifact:deploy] Uploaded 23K
[artifact:deploy] [INFO] Retrieving previous metadata from 
apache.snapshots.https
[artifact:deploy] [INFO] Uploading repository metadata for: 'snapshot 
org.apache.hive:hive-hwi:0.8.0-SNAPSHOT'
[artifact:deploy] [INFO] Retrieving previous metadata from 
apache.snapshots.https
[artifact:deploy] [INFO] Uploading repository metadata for: 'artifact 
org.apache.hive:hive-hwi'
[artifact:deploy] [INFO] Uploading project information for hive-hwi 
0.8.0-20110718.193102-37

ivy-init-dirs:

ivy-download:
  [get] Getting: 
http://repo2.maven.org/maven2/org/apache/ivy/ivy/2.1.0/ivy-2.1.0.jar
  [get] To: 
/x1/jenkins/jenkins-slave/workspace/Hive-trunk-h0.21/hive/build/ivy/lib/ivy-2.1.0.jar
  [get] Not modified - so not downloaded

ivy-probe-antlib:

ivy-init-antlib:

ivy-init:

ivy-resolve-maven-ant-tasks:
[ivy:resolve] :: loading settings :: file = 
/x1/jenkins/jenkins-slave/workspace/Hive-trunk-h0.21/hive/ivy/ivysettings.xml

ivy-retrieve-maven-ant-tasks:
[ivy:cachepath] DEPRECATED: 'ivy.conf.file' is deprecated, use 
'ivy.settings.file' instead
[ivy:cachepath] :: loading settings :: file = 
/x1/jenkins/jenkins-slave/workspace/Hive-trunk-h0.21/hive/ivy/ivysettings.xml

mvn-taskdef:

maven-publish-artifact:
[artifact:install-provider] Installing provider: 
org.apache.maven.wagon:wagon-http:jar:1.0-beta-2:runtime
[artifact:deploy] Deploying to 
https://repository.apache.org/content/repositories/snapshots
[artifact:deploy] [INFO] Retrieving previous build number from 
apache.snapshots.https
[artifact:deploy] Uploading: 
org/apache/hive/hive-jdbc/0.8.0-SNAPSHOT/hive-jdbc-0.8.0-20110718.193104-37.jar 
to repository apache.snapshots.https at 
https://repository.apache.org/content/repositories/snapshots
[artifact:deploy] Transferring 56K from apache.snapshots.https
[artifact:deploy] Uploaded 56K
[artifact:deploy] [INFO] Uploading project information for hive-jdbc 
0.8.0-20110718.193104-37
[artifact:deploy] [INFO] Retrieving previous metadata from 
apache.snapshots.https
[artifact:deploy] [INFO] Uploading repository metadata for: 'snapshot 
org.apache.hive:hive-jdbc:0.8.0-SNAPSHOT'
[artifact:deploy] [INFO] Retrieving previous metadata from 
apache.snapshots.https
[artifact:deploy] [INFO] Uploading repository metadata for: 'artifact 
org.apache.hive:hive-jdbc'

ivy-init-dirs:

ivy-download:
  [get] Getting: 
http://repo2.maven.org/maven2/org/apache/ivy/ivy/2.1.0/ivy-2.1.0.jar
  [get] To: 

[jira] [Updated] (HIVE-306) Support INSERT [INTO] destination

2011-07-18 Thread Franklin Hu (JIRA)

 [ 
https://issues.apache.org/jira/browse/HIVE-306?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Franklin Hu updated HIVE-306:
-

Resolution: Fixed
Status: Resolved  (was: Patch Available)

 Support INSERT [INTO] destination
 ---

 Key: HIVE-306
 URL: https://issues.apache.org/jira/browse/HIVE-306
 Project: Hive
  Issue Type: New Feature
Reporter: Zheng Shao
Assignee: Franklin Hu
 Attachments: hive-306.1.patch, hive-306.2.patch, hive-306.3.patch, 
 hive-306.4.patch


 Currently hive only supports INSERT OVERWRITE destination. We should 
 support INSERT [INTO] destination.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Updated] (HIVE-2035) Use block-level merge for RCFile if merging intermediate results are needed

2011-07-18 Thread Franklin Hu (JIRA)

 [ 
https://issues.apache.org/jira/browse/HIVE-2035?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Franklin Hu updated HIVE-2035:
--

   Resolution: Fixed
Fix Version/s: 0.8.0
   Status: Resolved  (was: Patch Available)

 Use block-level merge for RCFile if merging intermediate results are needed
 ---

 Key: HIVE-2035
 URL: https://issues.apache.org/jira/browse/HIVE-2035
 Project: Hive
  Issue Type: Improvement
Reporter: Ning Zhang
Assignee: Franklin Hu
 Fix For: 0.8.0

 Attachments: hive-2035.1.patch, hive-2035.3.patch


 Currently if hive.merge.mapredfiles and/or hive.merge.mapfile is set to true 
 the intermediate data could be merged using an additional MapReduce job. This 
 could be quite expensive if the data size is large. With HIVE-1950, merging 
 can be done in the RCFile block level so that it bypasses the 
 (de-)compression, (de-)serialization phases. This could improve the merge 
 process significantly. 
 This JIRA should handle the case where the input table is not stored in 
 RCFile, but the destination table is (which requires the intermediate data 
 should be stored in the same format as the destination table). 

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Created] (HIVE-2289) NumberFormatException with respect to _offsets when running a query with index

2011-07-18 Thread siddharth ramanan (JIRA)
NumberFormatException with respect to _offsets when running a query with  index
---

 Key: HIVE-2289
 URL: https://issues.apache.org/jira/browse/HIVE-2289
 Project: Hive
  Issue Type: Bug
  Components: Indexing
Affects Versions: 0.7.0
 Environment: RedHat 5
Reporter: siddharth ramanan


I am having a table named foo with columns origin, destination and information.

Steps I followed to create index named foosample for foo,

1)create index foosample on table foo(origin) as 'compact' with deferred 
rebuild;
2)alter index foosample on foo rebuild;
3)insert overwrite directory /tmp/index_result select 
'_bucketname','_offsets' from default__foo_foosample__ where origin='WAW';
4)set hive.index.compact.file=/tmp/index_result;
5)set 
hive.input.format=org.apache.hadoop.hive.ql.index.compact.HiveCompactIndexInputFormat;
6)select * from foo where origin='WAW';

Total MapReduce jobs = 1
Launching Job 1 out of 1
Number of reduce tasks is set to 0 since there's no reduce operator
java.lang.NumberFormatException: For input string: _offsets
at 
java.lang.NumberFormatException.forInputString(NumberFormatException.java:48)
at java.lang.Long.parseLong(Long.java:410)
at java.lang.Long.parseLong(Long.java:468)
at 
org.apache.hadoop.hive.ql.index.compact.HiveCompactIndexResult.add(HiveCompactIndexResult.java:158)
at 
org.apache.hadoop.hive.ql.index.compact.HiveCompactIndexResult.init(HiveCompactIndexResult.java:107)
at 
org.apache.hadoop.hive.ql.index.compact.HiveCompactIndexInputFormat.getSplits(HiveCompactIndexInputFormat.java:89)
at org.apache.hadoop.mapred.JobClient.writeOldSplits(JobClient.java:810)
at org.apache.hadoop.mapred.JobClient.submitJobInternal(JobClient.java:781)
at org.apache.hadoop.mapred.JobClient.submitJob(JobClient.java:730)
at org.apache.hadoop.hive.ql.exec.ExecDriver.execute(ExecDriver.java:657)
at org.apache.hadoop.hive.ql.exec.MapRedTask.execute(MapRedTask.java:123)
at org.apache.hadoop.hive.ql.exec.Task.executeTask(Task.java:130)
at 
org.apache.hadoop.hive.ql.exec.TaskRunner.runSequential(TaskRunner.java:57)
at org.apache.hadoop.hive.ql.Driver.launchTask(Driver.java:1063)
at org.apache.hadoop.hive.ql.Driver.execute(Driver.java:900)
at org.apache.hadoop.hive.ql.Driver.run(Driver.java:748)
at org.apache.hadoop.hive.cli.CliDriver.processCmd(CliDriver.java:164)
at org.apache.hadoop.hive.cli.CliDriver.processLine(CliDriver.java:241)
at org.apache.hadoop.hive.cli.CliDriver.main(CliDriver.java:456)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
at java.lang.reflect.Method.invoke(Method.java:597)
at org.apache.hadoop.util.RunJar.main(RunJar.java:156)
Job Submission failed with exception 'java.lang.NumberFormatException(For input 
string: _offsets)'
FAILED: Execution Error, return code 1 from 
org.apache.hadoop.hive.ql.exec.MapRedTask



Steps 2 and 3 ran a successful mapreduce job and also the table 
default__foo_foosample__ (index table) has data with three columns origin, 
_bucketname and _offsets.

Thanks,
Siddharth

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




Re: Review Request: Potential risk of resource leaks in Hive

2011-07-18 Thread John Sichi

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1098/#review1101
---



trunk/cli/src/java/org/apache/hadoop/hive/cli/CliDriver.java
https://reviews.apache.org/r/1098/#comment2233

Rename this to rc to avoid confusion with the processReader method name.



trunk/contrib/src/java/org/apache/hadoop/hive/contrib/util/typedbytes/TypedBytesWritableInput.java
https://reviews.apache.org/r/1098/#comment2234

Fix indentation for the try block.


- John


On 2011-07-13 14:28:47, chinna wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/1098/
 ---
 
 (Updated 2011-07-13 14:28:47)
 
 
 Review request for hive and John Sichi.
 
 
 Summary
 ---
 
 There are couple of resource leaks.
 For example,
 In CliDriver.java, Method :- processReader() the buffered reader is not 
 closed.
 
 Also there are risk(s) of resource(s) getting leaked , in such cases 
 refactored the code to move closing of resources in finally block.
 
 
 This addresses bug HIVE-1884.
 https://issues.apache.org/jira/browse/HIVE-1884
 
 
 Diffs
 -
 
   trunk/cli/src/java/org/apache/hadoop/hive/cli/CliDriver.java 1145950 
   
 trunk/contrib/src/java/org/apache/hadoop/hive/contrib/util/typedbytes/TypedBytesWritableInput.java
  1145950 
   trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java 1145950 
   trunk/ql/src/java/org/apache/hadoop/hive/ql/io/RCFileInputFormat.java 
 1145950 
 
 Diff: https://reviews.apache.org/r/1098/diff
 
 
 Testing
 ---
 
 All unit tests passed
 
 
 Thanks,
 
 chinna
 




Re: Review Request: Few improvements in org.apache.hadoop.hive.ql.metadata.Hive.close()

2011-07-18 Thread John Sichi

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1088/#review1099
---



trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
https://reviews.apache.org/r/1088/#comment2230

We should get rid of the member variable standAloneClient too since it is 
part of the dead code.

Now that we'll start calling shutdown (which we were skipping before), what 
will be the effect?



- John


On 2011-07-12 14:06:28, chinna wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/1088/
 ---
 
 (Updated 2011-07-12 14:06:28)
 
 
 Review request for hive and John Sichi.
 
 
 Summary
 ---
 
 Few improvements in org.apache.hadoop.hive.ql.metadata.Hive.close()
 
 
 This addresses bug HIVE-2184.
 https://issues.apache.org/jira/browse/HIVE-2184
 
 
 Diffs
 -
 
   
 trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
  1144190 
   trunk/ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 1144190 
 
 Diff: https://reviews.apache.org/r/1088/diff
 
 
 Testing
 ---
 
 Ran all the testcases
 
 
 Thanks,
 
 chinna
 




[jira] [Commented] (HIVE-2184) Few improvements in org.apache.hadoop.hive.ql.metadata.Hive.close()

2011-07-18 Thread jirapos...@reviews.apache.org (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-2184?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13067285#comment-13067285
 ] 

jirapos...@reviews.apache.org commented on HIVE-2184:
-


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1088/#review1099
---



trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
https://reviews.apache.org/r/1088/#comment2230

We should get rid of the member variable standAloneClient too since it is 
part of the dead code.

Now that we'll start calling shutdown (which we were skipping before), what 
will be the effect?



- John


On 2011-07-12 14:06:28, chinna wrote:
bq.  
bq.  ---
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/1088/
bq.  ---
bq.  
bq.  (Updated 2011-07-12 14:06:28)
bq.  
bq.  
bq.  Review request for hive and John Sichi.
bq.  
bq.  
bq.  Summary
bq.  ---
bq.  
bq.  Few improvements in org.apache.hadoop.hive.ql.metadata.Hive.close()
bq.  
bq.  
bq.  This addresses bug HIVE-2184.
bq.  https://issues.apache.org/jira/browse/HIVE-2184
bq.  
bq.  
bq.  Diffs
bq.  -
bq.  
bq.
trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
 1144190 
bq.trunk/ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java 1144190 
bq.  
bq.  Diff: https://reviews.apache.org/r/1088/diff
bq.  
bq.  
bq.  Testing
bq.  ---
bq.  
bq.  Ran all the testcases
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  chinna
bq.  
bq.



 Few improvements in org.apache.hadoop.hive.ql.metadata.Hive.close()
 ---

 Key: HIVE-2184
 URL: https://issues.apache.org/jira/browse/HIVE-2184
 Project: Hive
  Issue Type: Bug
  Components: Metastore
Affects Versions: 0.5.0, 0.8.0
 Environment: Hadoop 0.20.1, Hive0.8.0 and SUSE Linux Enterprise 
 Server 10 SP2 (i586) - Kernel 2.6.16.60-0.21-smp (5)
Reporter: Chinna Rao Lalam
Assignee: Chinna Rao Lalam
 Attachments: HIVE-2184.1.patch, HIVE-2184.1.patch, HIVE-2184.patch


 1)Hive.close() will call HiveMetaStoreClient.close() in this method the 
 variable standAloneClient is never become true then client.shutdown() never 
 call.
 2)Hive.close() After calling metaStoreClient.close() need to make 
 metaStoreClient=null

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Updated] (HIVE-1884) Potential risk of resource leaks in Hive

2011-07-18 Thread John Sichi (JIRA)

 [ 
https://issues.apache.org/jira/browse/HIVE-1884?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

John Sichi updated HIVE-1884:
-

Status: Open  (was: Patch Available)

Comments added on Review Board.

 Potential risk of resource leaks in Hive
 

 Key: HIVE-1884
 URL: https://issues.apache.org/jira/browse/HIVE-1884
 Project: Hive
  Issue Type: Bug
  Components: CLI, Metastore, Query Processor, Server Infrastructure
Affects Versions: 0.6.0, 0.5.0, 0.4.1, 0.4.0, 0.3.0
 Environment: Hive 0.6.0, Hadoop 0.20.1
 SUSE Linux Enterprise Server 11 (i586)
Reporter: Mohit Sikri
Assignee: Chinna Rao Lalam
 Attachments: HIVE-1884.1.PATCH, HIVE-1884.2.patch, HIVE-1884.3.patch, 
 HIVE-1884.4.patch


 h3.There are couple of resource leaks.
 h4.For example,
 In CliDriver.java, Method :- processReader() the buffered reader is not 
 closed.
 h3.Also there are risk(s) of  resource(s) getting leaked , in such cases we 
 need to re factor the code to move closing of resources in finally block.
 h4. For Example :- 
 In Throttle.java   Method:- checkJobTracker() , the following code snippet 
 might cause resource leak.
 {code}
 InputStream in = url.openStream();
 in.read(buffer);
 in.close();
 {code}
 Ideally and as per the best coding practices it should be like below
 {code}
 InputStream in=null;
 try   {
 in = url.openStream();
 int numRead = in.read(buffer);
 }
 finally {
IOUtils.closeStream(in);
 }
 {code}
 Similar cases, were found in ExplainTask.java, DDLTask.java etc.Need to re 
 factor all such occurrences.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HIVE-1884) Potential risk of resource leaks in Hive

2011-07-18 Thread jirapos...@reviews.apache.org (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-1884?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13067286#comment-13067286
 ] 

jirapos...@reviews.apache.org commented on HIVE-1884:
-


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1098/#review1101
---



trunk/cli/src/java/org/apache/hadoop/hive/cli/CliDriver.java
https://reviews.apache.org/r/1098/#comment2233

Rename this to rc to avoid confusion with the processReader method name.



trunk/contrib/src/java/org/apache/hadoop/hive/contrib/util/typedbytes/TypedBytesWritableInput.java
https://reviews.apache.org/r/1098/#comment2234

Fix indentation for the try block.


- John


On 2011-07-13 14:28:47, chinna wrote:
bq.  
bq.  ---
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/1098/
bq.  ---
bq.  
bq.  (Updated 2011-07-13 14:28:47)
bq.  
bq.  
bq.  Review request for hive and John Sichi.
bq.  
bq.  
bq.  Summary
bq.  ---
bq.  
bq.  There are couple of resource leaks.
bq.  For example,
bq.  In CliDriver.java, Method :- processReader() the buffered reader is not 
closed.
bq.  
bq.  Also there are risk(s) of resource(s) getting leaked , in such cases 
refactored the code to move closing of resources in finally block.
bq.  
bq.  
bq.  This addresses bug HIVE-1884.
bq.  https://issues.apache.org/jira/browse/HIVE-1884
bq.  
bq.  
bq.  Diffs
bq.  -
bq.  
bq.trunk/cli/src/java/org/apache/hadoop/hive/cli/CliDriver.java 1145950 
bq.
trunk/contrib/src/java/org/apache/hadoop/hive/contrib/util/typedbytes/TypedBytesWritableInput.java
 1145950 
bq.trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java 1145950 
bq.trunk/ql/src/java/org/apache/hadoop/hive/ql/io/RCFileInputFormat.java 
1145950 
bq.  
bq.  Diff: https://reviews.apache.org/r/1098/diff
bq.  
bq.  
bq.  Testing
bq.  ---
bq.  
bq.  All unit tests passed
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  chinna
bq.  
bq.



 Potential risk of resource leaks in Hive
 

 Key: HIVE-1884
 URL: https://issues.apache.org/jira/browse/HIVE-1884
 Project: Hive
  Issue Type: Bug
  Components: CLI, Metastore, Query Processor, Server Infrastructure
Affects Versions: 0.3.0, 0.4.0, 0.4.1, 0.5.0, 0.6.0
 Environment: Hive 0.6.0, Hadoop 0.20.1
 SUSE Linux Enterprise Server 11 (i586)
Reporter: Mohit Sikri
Assignee: Chinna Rao Lalam
 Attachments: HIVE-1884.1.PATCH, HIVE-1884.2.patch, HIVE-1884.3.patch, 
 HIVE-1884.4.patch


 h3.There are couple of resource leaks.
 h4.For example,
 In CliDriver.java, Method :- processReader() the buffered reader is not 
 closed.
 h3.Also there are risk(s) of  resource(s) getting leaked , in such cases we 
 need to re factor the code to move closing of resources in finally block.
 h4. For Example :- 
 In Throttle.java   Method:- checkJobTracker() , the following code snippet 
 might cause resource leak.
 {code}
 InputStream in = url.openStream();
 in.read(buffer);
 in.close();
 {code}
 Ideally and as per the best coding practices it should be like below
 {code}
 InputStream in=null;
 try   {
 in = url.openStream();
 int numRead = in.read(buffer);
 }
 finally {
IOUtils.closeStream(in);
 }
 {code}
 Similar cases, were found in ExplainTask.java, DDLTask.java etc.Need to re 
 factor all such occurrences.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Assigned] (HIVE-559) Support JDBC ResultSetMetadata

2011-07-18 Thread Carl Steinbach (JIRA)

 [ 
https://issues.apache.org/jira/browse/HIVE-559?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Carl Steinbach reassigned HIVE-559:
---

Assignee: Prasad Mujumdar  (was: Min Zhou)

 Support JDBC ResultSetMetadata
 --

 Key: HIVE-559
 URL: https://issues.apache.org/jira/browse/HIVE-559
 Project: Hive
  Issue Type: Sub-task
  Components: JDBC
Affects Versions: 0.5.0
Reporter: Bill Graham
Assignee: Prasad Mujumdar

 Support ResultSetMetadata for JDBC ResultSets. The getColumn* methods would 
 be particularly useful I'd expect:
 http://java.sun.com/javase/6/docs/api/java/sql/ResultSetMetaData.html
 The challenge as I see it though, is that the JDBC client only has access to 
 the raw query string and the result data when running in standalone mode. 
 Therefore, it will need to get the column metadata one of two way: 
 1. By parsing the query to determine the tables/columns involved and then 
 making a request to the metastore to get the metadata for the columns. This 
 certainly feels like duplicate work, since the query of course gets properly 
 parsed on the server.
 2. By returning the column metadata from the server. My thrift knowledge is 
 limited, but I suspect adding this to the response would present other 
 challenges.
 Any thoughts or suggestions? Option #1 feels clunkier, yet safer.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HIVE-2289) NumberFormatException with respect to _offsets when running a query with index

2011-07-18 Thread Syed S. Albiz (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-2289?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13067317#comment-13067317
 ] 

Syed S. Albiz commented on HIVE-2289:
-

The column names in your step 3 need to be backticked, not quoted. Look at 
https://github.com/apache/hive/blob/trunk/ql/src/test/queries/clientpositive/index_auto.q
 for an example query that runs properly.

 NumberFormatException with respect to _offsets when running a query with  
 index
 ---

 Key: HIVE-2289
 URL: https://issues.apache.org/jira/browse/HIVE-2289
 Project: Hive
  Issue Type: Bug
  Components: Indexing
Affects Versions: 0.7.0
 Environment: RedHat 5
Reporter: siddharth ramanan

 I am having a table named foo with columns origin, destination and 
 information.
 Steps I followed to create index named foosample for foo,
 1)create index foosample on table foo(origin) as 'compact' with deferred 
 rebuild;
 2)alter index foosample on foo rebuild;
 3)insert overwrite directory /tmp/index_result select 
 '_bucketname','_offsets' from default__foo_foosample__ where origin='WAW';
 4)set hive.index.compact.file=/tmp/index_result;
 5)set 
 hive.input.format=org.apache.hadoop.hive.ql.index.compact.HiveCompactIndexInputFormat;
 6)select * from foo where origin='WAW';
 Total MapReduce jobs = 1
 Launching Job 1 out of 1
 Number of reduce tasks is set to 0 since there's no reduce operator
 java.lang.NumberFormatException: For input string: _offsets
 at 
 java.lang.NumberFormatException.forInputString(NumberFormatException.java:48)
 at java.lang.Long.parseLong(Long.java:410)
 at java.lang.Long.parseLong(Long.java:468)
 at 
 org.apache.hadoop.hive.ql.index.compact.HiveCompactIndexResult.add(HiveCompactIndexResult.java:158)
 at 
 org.apache.hadoop.hive.ql.index.compact.HiveCompactIndexResult.init(HiveCompactIndexResult.java:107)
 at 
 org.apache.hadoop.hive.ql.index.compact.HiveCompactIndexInputFormat.getSplits(HiveCompactIndexInputFormat.java:89)
 at org.apache.hadoop.mapred.JobClient.writeOldSplits(JobClient.java:810)
 at 
 org.apache.hadoop.mapred.JobClient.submitJobInternal(JobClient.java:781)
 at org.apache.hadoop.mapred.JobClient.submitJob(JobClient.java:730)
 at org.apache.hadoop.hive.ql.exec.ExecDriver.execute(ExecDriver.java:657)
 at org.apache.hadoop.hive.ql.exec.MapRedTask.execute(MapRedTask.java:123)
 at org.apache.hadoop.hive.ql.exec.Task.executeTask(Task.java:130)
 at 
 org.apache.hadoop.hive.ql.exec.TaskRunner.runSequential(TaskRunner.java:57)
 at org.apache.hadoop.hive.ql.Driver.launchTask(Driver.java:1063)
 at org.apache.hadoop.hive.ql.Driver.execute(Driver.java:900)
 at org.apache.hadoop.hive.ql.Driver.run(Driver.java:748)
 at org.apache.hadoop.hive.cli.CliDriver.processCmd(CliDriver.java:164)
 at org.apache.hadoop.hive.cli.CliDriver.processLine(CliDriver.java:241)
 at org.apache.hadoop.hive.cli.CliDriver.main(CliDriver.java:456)
 at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
 at 
 sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
 at 
 sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
 at java.lang.reflect.Method.invoke(Method.java:597)
 at org.apache.hadoop.util.RunJar.main(RunJar.java:156)
 Job Submission failed with exception 'java.lang.NumberFormatException(For 
 input string: _offsets)'
 FAILED: Execution Error, return code 1 from 
 org.apache.hadoop.hive.ql.exec.MapRedTask
 Steps 2 and 3 ran a successful mapreduce job and also the table 
 default__foo_foosample__ (index table) has data with three columns origin, 
 _bucketname and _offsets.
 Thanks,
 Siddharth

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Updated] (HIVE-306) Support INSERT [INTO] destination

2011-07-18 Thread Carl Steinbach (JIRA)

 [ 
https://issues.apache.org/jira/browse/HIVE-306?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Carl Steinbach updated HIVE-306:


  Component/s: SQL
   Query Processor
Fix Version/s: 0.8.0

 Support INSERT [INTO] destination
 ---

 Key: HIVE-306
 URL: https://issues.apache.org/jira/browse/HIVE-306
 Project: Hive
  Issue Type: New Feature
  Components: Query Processor, SQL
Reporter: Zheng Shao
Assignee: Franklin Hu
 Fix For: 0.8.0

 Attachments: hive-306.1.patch, hive-306.2.patch, hive-306.3.patch, 
 hive-306.4.patch


 Currently hive only supports INSERT OVERWRITE destination. We should 
 support INSERT [INTO] destination.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Created] (HIVE-2290) Improve error messages for DESCRIBE command

2011-07-18 Thread Carl Steinbach (JIRA)
Improve error messages for DESCRIBE command
---

 Key: HIVE-2290
 URL: https://issues.apache.org/jira/browse/HIVE-2290
 Project: Hive
  Issue Type: Improvement
  Components: Diagnosability, SQL
Reporter: Carl Steinbach




--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HIVE-2290) Improve error messages for DESCRIBE command

2011-07-18 Thread Carl Steinbach (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-2290?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13067327#comment-13067327
 ] 

Carl Steinbach commented on HIVE-2290:
--

Hive's DESCRIBE command uses non-standard syntax (see HIVE-1977) which results 
in a lot of errors for users. Unfortunately, the current error message does not 
provide any help:

{noformat}
hive CREATE DATABASE d1;
OK
hive CREATE TABLE d1.t(x INT);
OK
hive DESCRIBE d1.t;
FAILED: Execution Error, return code 1 from 
org.apache.hadoop.hive.ql.exec.DDLTask
hive 
{noformat}

The error message should instead say something like Table 'd1' does not exist.

 Improve error messages for DESCRIBE command
 ---

 Key: HIVE-2290
 URL: https://issues.apache.org/jira/browse/HIVE-2290
 Project: Hive
  Issue Type: Improvement
  Components: Diagnosability, SQL
Reporter: Carl Steinbach



--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HIVE-1977) DESCRIBE TABLE syntax doesn't support specifying a database qualified table name

2011-07-18 Thread Carl Steinbach (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-1977?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13067330#comment-13067330
 ] 

Carl Steinbach commented on HIVE-1977:
--

I think we can fix this by introducing a new configuration property that 
controls whether DESCRIBE uses the standard syntax or the Hive syntax.

 DESCRIBE TABLE syntax doesn't support specifying a database qualified table 
 name
 

 Key: HIVE-1977
 URL: https://issues.apache.org/jira/browse/HIVE-1977
 Project: Hive
  Issue Type: Bug
  Components: Query Processor
Reporter: Carl Steinbach

 The syntax for DESCRIBE is broken. It should be:
 {code}
 DESCRIBE [EXTENDED] [database DOT]table [column]
 {code}
 but is actually
 {code}
 DESCRIBE [EXTENDED] table[DOT col_name]
 {code}
 Ref: http://dev.mysql.com/doc/refman/5.0/en/describe.html

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




Re: Review Request: HIVE-1078: CREATE VIEW followup: CREATE OR REPLACE

2011-07-18 Thread John Sichi

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1058/#review1102
---



http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
https://reviews.apache.org/r/1058/#comment2236

We'll allow this if no partitions currently exist.



http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
https://reviews.apache.org/r/1058/#comment2237

Shouldn't we allow for changes to the view comment?  Also, any 
TBLPROPERTIES specified here should be applied to the view (but we shouldn't 
delete old ones, only overwrite them if they already exist).




http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientpositive/create_or_replace_view.q
https://reviews.apache.org/r/1058/#comment2235

These need explicit PARTITIONED ON clauses to indicate that the existing 
partitioning scheme is being preserved.

Also, need test case for modifying the definition of an unpartitioned view.

Plus new cases we discussed for allowing change in partitioning when no 
partitions currently exist.

Also, negative case for combining IF NOT EXISTS with OR REPLACE.

Also, negative case for updating the view to have an invalid definition.

Also, positive case for using this to fix an invalid view.

Have you tried creating a view cycle?



- John


On 2011-07-18 07:01:02, Charles Chen wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/1058/
 ---
 
 (Updated 2011-07-18 07:01:02)
 
 
 Review request for hive.
 
 
 Summary
 ---
 
 https://issues.apache.org/jira/browse/HIVE-1078
 
 
 This addresses bug HIVE-1078.
 https://issues.apache.org/jira/browse/HIVE-1078
 
 
 Diffs
 -
 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_view.q.out
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_or_replace_view.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view4.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/Hive.g
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/CreateViewDesc.java
  1146902 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view1.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view2.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view3.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view4.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientpositive/create_or_replace_view.q
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view1.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view2.q.out
  PRE-CREATION 
   
 http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view3.q.out
  PRE-CREATION 
 
 Diff: https://reviews.apache.org/r/1058/diff
 
 
 Testing
 ---
 
 Passes unit tests
 
 
 Thanks,
 
 Charles
 




[jira] [Updated] (HIVE-1078) CREATE VIEW followup: CREATE OR REPLACE

2011-07-18 Thread John Sichi (JIRA)

 [ 
https://issues.apache.org/jira/browse/HIVE-1078?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

John Sichi updated HIVE-1078:
-

Status: Open  (was: Patch Available)

Comments added in Review Board.

 CREATE VIEW followup:  CREATE OR REPLACE
 

 Key: HIVE-1078
 URL: https://issues.apache.org/jira/browse/HIVE-1078
 Project: Hive
  Issue Type: Improvement
  Components: Query Processor
Affects Versions: 0.6.0
Reporter: John Sichi
Assignee: Charles Chen
 Attachments: HIVE-1078v3.patch, HIVE-1078v4.patch, HIVE-1078v5.patch, 
 HIVE-1078v6.patch


 Currently, replacing a view requires
 DROP VIEW v;
 CREATE VIEW v AS new-definition;
 CREATE OR REPLACE would allow these to be combined into a single operation.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HIVE-1078) CREATE VIEW followup: CREATE OR REPLACE

2011-07-18 Thread jirapos...@reviews.apache.org (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-1078?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13067334#comment-13067334
 ] 

jirapos...@reviews.apache.org commented on HIVE-1078:
-


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1058/#review1102
---



http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
https://reviews.apache.org/r/1058/#comment2236

We'll allow this if no partitions currently exist.



http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
https://reviews.apache.org/r/1058/#comment2237

Shouldn't we allow for changes to the view comment?  Also, any 
TBLPROPERTIES specified here should be applied to the view (but we shouldn't 
delete old ones, only overwrite them if they already exist).




http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientpositive/create_or_replace_view.q
https://reviews.apache.org/r/1058/#comment2235

These need explicit PARTITIONED ON clauses to indicate that the existing 
partitioning scheme is being preserved.

Also, need test case for modifying the definition of an unpartitioned view.

Plus new cases we discussed for allowing change in partitioning when no 
partitions currently exist.

Also, negative case for combining IF NOT EXISTS with OR REPLACE.

Also, negative case for updating the view to have an invalid definition.

Also, positive case for using this to fix an invalid view.

Have you tried creating a view cycle?



- John


On 2011-07-18 07:01:02, Charles Chen wrote:
bq.  
bq.  ---
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/1058/
bq.  ---
bq.  
bq.  (Updated 2011-07-18 07:01:02)
bq.  
bq.  
bq.  Review request for hive.
bq.  
bq.  
bq.  Summary
bq.  ---
bq.  
bq.  https://issues.apache.org/jira/browse/HIVE-1078
bq.  
bq.  
bq.  This addresses bug HIVE-1078.
bq.  https://issues.apache.org/jira/browse/HIVE-1078
bq.  
bq.  
bq.  Diffs
bq.  -
bq.  
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_view.q.out
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientpositive/create_or_replace_view.q.out
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view4.q.out
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/Hive.g
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/CreateViewDesc.java
 1146902 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view1.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view2.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view3.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientnegative/create_or_replace_view4.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/queries/clientpositive/create_or_replace_view.q
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view1.q.out
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view2.q.out
 PRE-CREATION 
bq.
http://svn.apache.org/repos/asf/hive/trunk/ql/src/test/results/clientnegative/create_or_replace_view3.q.out
 PRE-CREATION 
bq.  
bq.  Diff: https://reviews.apache.org/r/1058/diff
bq.  
bq.  
bq.  Testing
bq.  ---
bq.  
bq.  Passes unit tests
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Charles
bq.  
bq.



 CREATE VIEW followup:  CREATE OR REPLACE
 

 Key: HIVE-1078
 URL: https://issues.apache.org/jira/browse/HIVE-1078
 Project: Hive
  Issue Type: Improvement
  Components: Query Processor
Affects Versions: 0.6.0
Reporter: John Sichi

[jira] [Updated] (HIVE-1977) DESCRIBE TABLE syntax doesn't support specifying a database qualified table name

2011-07-18 Thread Carl Steinbach (JIRA)

 [ 
https://issues.apache.org/jira/browse/HIVE-1977?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Carl Steinbach updated HIVE-1977:
-

Component/s: SQL

 DESCRIBE TABLE syntax doesn't support specifying a database qualified table 
 name
 

 Key: HIVE-1977
 URL: https://issues.apache.org/jira/browse/HIVE-1977
 Project: Hive
  Issue Type: Bug
  Components: Query Processor, SQL
Reporter: Carl Steinbach

 The syntax for DESCRIBE is broken. It should be:
 {code}
 DESCRIBE [EXTENDED] [database DOT]table [column]
 {code}
 but is actually
 {code}
 DESCRIBE [EXTENDED] table[DOT col_name]
 {code}
 Ref: http://dev.mysql.com/doc/refman/5.0/en/describe.html

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Resolved] (HIVE-2289) NumberFormatException with respect to _offsets when running a query with index

2011-07-18 Thread John Sichi (JIRA)

 [ 
https://issues.apache.org/jira/browse/HIVE-2289?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

John Sichi resolved HIVE-2289.
--

Resolution: Invalid

Closing as invalid for reason Syed mentioned.  Also, try out the new automatic 
index usage support if you get a chance!

 NumberFormatException with respect to _offsets when running a query with  
 index
 ---

 Key: HIVE-2289
 URL: https://issues.apache.org/jira/browse/HIVE-2289
 Project: Hive
  Issue Type: Bug
  Components: Indexing
Affects Versions: 0.7.0
 Environment: RedHat 5
Reporter: siddharth ramanan

 I am having a table named foo with columns origin, destination and 
 information.
 Steps I followed to create index named foosample for foo,
 1)create index foosample on table foo(origin) as 'compact' with deferred 
 rebuild;
 2)alter index foosample on foo rebuild;
 3)insert overwrite directory /tmp/index_result select 
 '_bucketname','_offsets' from default__foo_foosample__ where origin='WAW';
 4)set hive.index.compact.file=/tmp/index_result;
 5)set 
 hive.input.format=org.apache.hadoop.hive.ql.index.compact.HiveCompactIndexInputFormat;
 6)select * from foo where origin='WAW';
 Total MapReduce jobs = 1
 Launching Job 1 out of 1
 Number of reduce tasks is set to 0 since there's no reduce operator
 java.lang.NumberFormatException: For input string: _offsets
 at 
 java.lang.NumberFormatException.forInputString(NumberFormatException.java:48)
 at java.lang.Long.parseLong(Long.java:410)
 at java.lang.Long.parseLong(Long.java:468)
 at 
 org.apache.hadoop.hive.ql.index.compact.HiveCompactIndexResult.add(HiveCompactIndexResult.java:158)
 at 
 org.apache.hadoop.hive.ql.index.compact.HiveCompactIndexResult.init(HiveCompactIndexResult.java:107)
 at 
 org.apache.hadoop.hive.ql.index.compact.HiveCompactIndexInputFormat.getSplits(HiveCompactIndexInputFormat.java:89)
 at org.apache.hadoop.mapred.JobClient.writeOldSplits(JobClient.java:810)
 at 
 org.apache.hadoop.mapred.JobClient.submitJobInternal(JobClient.java:781)
 at org.apache.hadoop.mapred.JobClient.submitJob(JobClient.java:730)
 at org.apache.hadoop.hive.ql.exec.ExecDriver.execute(ExecDriver.java:657)
 at org.apache.hadoop.hive.ql.exec.MapRedTask.execute(MapRedTask.java:123)
 at org.apache.hadoop.hive.ql.exec.Task.executeTask(Task.java:130)
 at 
 org.apache.hadoop.hive.ql.exec.TaskRunner.runSequential(TaskRunner.java:57)
 at org.apache.hadoop.hive.ql.Driver.launchTask(Driver.java:1063)
 at org.apache.hadoop.hive.ql.Driver.execute(Driver.java:900)
 at org.apache.hadoop.hive.ql.Driver.run(Driver.java:748)
 at org.apache.hadoop.hive.cli.CliDriver.processCmd(CliDriver.java:164)
 at org.apache.hadoop.hive.cli.CliDriver.processLine(CliDriver.java:241)
 at org.apache.hadoop.hive.cli.CliDriver.main(CliDriver.java:456)
 at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
 at 
 sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
 at 
 sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
 at java.lang.reflect.Method.invoke(Method.java:597)
 at org.apache.hadoop.util.RunJar.main(RunJar.java:156)
 Job Submission failed with exception 'java.lang.NumberFormatException(For 
 input string: _offsets)'
 FAILED: Execution Error, return code 1 from 
 org.apache.hadoop.hive.ql.exec.MapRedTask
 Steps 2 and 3 ran a successful mapreduce job and also the table 
 default__foo_foosample__ (index table) has data with three columns origin, 
 _bucketname and _offsets.
 Thanks,
 Siddharth

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HIVE-956) Add support of columnar binary serde

2011-07-18 Thread Franklin Hu (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13067345#comment-13067345
 ] 

Franklin Hu commented on HIVE-956:
--

@Krishna looks like it works now with some simple tests

 Add support of columnar binary serde
 

 Key: HIVE-956
 URL: https://issues.apache.org/jira/browse/HIVE-956
 Project: Hive
  Issue Type: New Feature
Reporter: He Yongqiang
Assignee: Krishna Kumar
 Attachments: HIVE-956v3.patch, HIVE.956.patch.0, HIVE.956.patch.1, 
 HIVE.956.patch.2




--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




Build failed in Jenkins: Hive-trunk-h0.21 #833

2011-07-18 Thread Apache Jenkins Server
See https://builds.apache.org/job/Hive-trunk-h0.21/833/changes

Changes:

[jvs] HIVE-1218. CREATE TABLE t LIKE some_view should create a new empty base 
table,
but instead creates a copy of view
(Charles Chen via jvs)

--
[...truncated 31278 lines...]
[junit] PREHOOK: query: select count(1) as cnt from testhivedrivertable
[junit] PREHOOK: type: QUERY
[junit] PREHOOK: Input: default@testhivedrivertable
[junit] PREHOOK: Output: 
file:/tmp/jenkins/hive_2011-07-18_16-32-47_906_5812480073776199380/-mr-1
[junit] Total MapReduce jobs = 1
[junit] Launching Job 1 out of 1
[junit] Number of reduce tasks determined at compile time: 1
[junit] In order to change the average load for a reducer (in bytes):
[junit]   set hive.exec.reducers.bytes.per.reducer=number
[junit] In order to limit the maximum number of reducers:
[junit]   set hive.exec.reducers.max=number
[junit] In order to set a constant number of reducers:
[junit]   set mapred.reduce.tasks=number
[junit] Job running in-process (local Hadoop)
[junit] Hadoop job information for null: number of mappers: 0; number of 
reducers: 0
[junit] 2011-07-18 16:32:50,900 null map = 100%,  reduce = 100%
[junit] Ended Job = job_local_0001
[junit] POSTHOOK: query: select count(1) as cnt from testhivedrivertable
[junit] POSTHOOK: type: QUERY
[junit] POSTHOOK: Input: default@testhivedrivertable
[junit] POSTHOOK: Output: 
file:/tmp/jenkins/hive_2011-07-18_16-32-47_906_5812480073776199380/-mr-1
[junit] OK
[junit] PREHOOK: query: drop table testhivedrivertable
[junit] PREHOOK: type: DROPTABLE
[junit] PREHOOK: Input: default@testhivedrivertable
[junit] PREHOOK: Output: default@testhivedrivertable
[junit] POSTHOOK: query: drop table testhivedrivertable
[junit] POSTHOOK: type: DROPTABLE
[junit] POSTHOOK: Input: default@testhivedrivertable
[junit] POSTHOOK: Output: default@testhivedrivertable
[junit] OK
[junit] Hive history 
file=/x1/jenkins/jenkins-slave/workspace/Hive-trunk-h0.21/hive/build/service/tmp/hive_job_log_jenkins_201107181632_971455472.txt
[junit] PREHOOK: query: drop table testhivedrivertable
[junit] PREHOOK: type: DROPTABLE
[junit] POSTHOOK: query: drop table testhivedrivertable
[junit] POSTHOOK: type: DROPTABLE
[junit] OK
[junit] PREHOOK: query: create table testhivedrivertable (num int)
[junit] PREHOOK: type: CREATETABLE
[junit] POSTHOOK: query: create table testhivedrivertable (num int)
[junit] POSTHOOK: type: CREATETABLE
[junit] POSTHOOK: Output: default@testhivedrivertable
[junit] OK
[junit] PREHOOK: query: load data local inpath 
'/x1/jenkins/jenkins-slave/workspace/Hive-trunk-h0.21/hive/data/files/kv1.txt' 
into table testhivedrivertable
[junit] PREHOOK: type: LOAD
[junit] PREHOOK: Output: default@testhivedrivertable
[junit] Copying data from 
file:/x1/jenkins/jenkins-slave/workspace/Hive-trunk-h0.21/hive/data/files/kv1.txt
[junit] Loading data to table default.testhivedrivertable
[junit] POSTHOOK: query: load data local inpath 
'/x1/jenkins/jenkins-slave/workspace/Hive-trunk-h0.21/hive/data/files/kv1.txt' 
into table testhivedrivertable
[junit] POSTHOOK: type: LOAD
[junit] POSTHOOK: Output: default@testhivedrivertable
[junit] OK
[junit] PREHOOK: query: select * from testhivedrivertable limit 10
[junit] PREHOOK: type: QUERY
[junit] PREHOOK: Input: default@testhivedrivertable
[junit] PREHOOK: Output: 
file:/tmp/jenkins/hive_2011-07-18_16-32-52_178_3708779228214068993/-mr-1
[junit] POSTHOOK: query: select * from testhivedrivertable limit 10
[junit] POSTHOOK: type: QUERY
[junit] POSTHOOK: Input: default@testhivedrivertable
[junit] POSTHOOK: Output: 
file:/tmp/jenkins/hive_2011-07-18_16-32-52_178_3708779228214068993/-mr-1
[junit] OK
[junit] PREHOOK: query: drop table testhivedrivertable
[junit] PREHOOK: type: DROPTABLE
[junit] PREHOOK: Input: default@testhivedrivertable
[junit] PREHOOK: Output: default@testhivedrivertable
[junit] POSTHOOK: query: drop table testhivedrivertable
[junit] POSTHOOK: type: DROPTABLE
[junit] POSTHOOK: Input: default@testhivedrivertable
[junit] POSTHOOK: Output: default@testhivedrivertable
[junit] OK
[junit] Hive history 
file=/x1/jenkins/jenkins-slave/workspace/Hive-trunk-h0.21/hive/build/service/tmp/hive_job_log_jenkins_201107181632_650774882.txt
[junit] PREHOOK: query: drop table testhivedrivertable
[junit] PREHOOK: type: DROPTABLE
[junit] POSTHOOK: query: drop table testhivedrivertable
[junit] POSTHOOK: type: DROPTABLE
[junit] OK
[junit] PREHOOK: query: create table testhivedrivertable (num int)
[junit] PREHOOK: type: CREATETABLE
[junit] POSTHOOK: query: create table testhivedrivertable (num int)
[junit] POSTHOOK: type: CREATETABLE
[junit] POSTHOOK: Output: 

[jira] [Commented] (HIVE-1218) CREATE TABLE t LIKE some_view should create a new empty base table, but instead creates a copy of view

2011-07-18 Thread Hudson (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-1218?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13067394#comment-13067394
 ] 

Hudson commented on HIVE-1218:
--

Integrated in Hive-trunk-h0.21 #833 (See 
[https://builds.apache.org/job/Hive-trunk-h0.21/833/])
HIVE-1218. CREATE TABLE t LIKE some_view should create a new empty base 
table,
but instead creates a copy of view
(Charles Chen via jvs)

jvs : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVNview=revrev=1148005
Files : 
* /hive/trunk/ql/src/test/queries/clientpositive/create_like.q
* /hive/trunk/ql/src/test/results/clientpositive/create_like.q.out
* /hive/trunk/ql/src/test/results/clientpositive/create_like_view.q.out
* /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/parse/SemanticAnalyzer.java
* /hive/trunk/ql/src/test/queries/clientpositive/create_like_view.q
* 
/hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/plan/CreateTableLikeDesc.java
* /hive/trunk/ql/src/java/org/apache/hadoop/hive/ql/exec/DDLTask.java


 CREATE TABLE t LIKE some_view should create a new empty base table, but 
 instead creates a copy of view
 --

 Key: HIVE-1218
 URL: https://issues.apache.org/jira/browse/HIVE-1218
 Project: Hive
  Issue Type: Bug
  Components: Metastore
Affects Versions: 0.6.0
Reporter: John Sichi
Assignee: Charles Chen
 Fix For: 0.8.0

 Attachments: HIVE-1218v0.patch, HIVE-1218v2.patch, HIVE-1218v3.patch, 
 HIVE-1218v4.patch, HIVE-1218v5.patch, HIVE-1218v6.patch, HIVE-1218v7.patch, 
 HIVE-1218v8.patch


 I think it should copy only the column definitions from the view metadata.  
 Currently it is copying the entire descriptor, resulting in a new view 
 instead of a new base table.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HIVE-2128) Automatic Indexing with multiple tables

2011-07-18 Thread John Sichi (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-2128?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13067398#comment-13067398
 ] 

John Sichi commented on HIVE-2128:
--

Could you make sure the latest patch is uploaded here and matching Review 
Board, and then click Submit Patch?  Also make sure all spurious changes (like 
extra imports) are gone; I'm seeing some of those in Review Board.

 Automatic Indexing with multiple tables
 ---

 Key: HIVE-2128
 URL: https://issues.apache.org/jira/browse/HIVE-2128
 Project: Hive
  Issue Type: Improvement
  Components: Indexing
Affects Versions: 0.8.0
Reporter: Russell Melick
Assignee: Syed S. Albiz
 Attachments: HIVE-2128.1.patch, HIVE-2128.1.patch, HIVE-2128.2.patch


 Make automatic indexing work with jobs which access multiple tables.  We'll 
 probably need to modify the way that the index input format works in order to 
 associate index formats/files with specific tables.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Assigned] (HIVE-2286) ClassCastException when building index with security.authorization turned on

2011-07-18 Thread John Sichi (JIRA)

 [ 
https://issues.apache.org/jira/browse/HIVE-2286?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

John Sichi reassigned HIVE-2286:


Assignee: Syed S. Albiz

 ClassCastException when building index with security.authorization turned on
 

 Key: HIVE-2286
 URL: https://issues.apache.org/jira/browse/HIVE-2286
 Project: Hive
  Issue Type: Bug
Reporter: Syed S. Albiz
Assignee: Syed S. Albiz
 Attachments: HIVE-2286.1.patch


 When trying to build an index with authorization checks turned on, hive 
 issues the following ClassCastException:
 org.apache.hadoop.hive.ql.parse.DDLSemanticAnalyzer cannot be cast to
 org.apache.hadoop.hive.ql.parse.SemanticAnalyzer
  at
 org.apache.hadoop.hive.ql.Driver.doAuthorization(Driver.java:540)
  at org.apache.hadoop.hive.ql.Driver.compile(Driver.java:431)
  at org.apache.hadoop.hive.ql.Driver.compile(Driver.java:335)
  at org.apache.hadoop.hive.ql.Driver.run(Driver.java:848)
  at
 org.apache.hadoop.hive.cli.CliDriver.processCmd(CliDriver.java:224)
  at
 org.apache.hadoop.hive.cli.CliDriver.processLine(CliDriver.java:358)
  at
 org.apache.hadoop.hive.cli.CliDriver.processLine(CliDriver.java:293)
  at
 org.apache.hadoop.hive.cli.CliDriver.processReader(CliDriver.java:385)
  at
 org.apache.hadoop.hive.cli.CliDriver.processFile(CliDriver.java:392)
  at
 org.apache.hadoop.hive.cli.CliDriver.main(CliDriver.java:567)
  at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
  at
 sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.jav
 a:39)
  at
 sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessor
 Impl.java:25)
  at java.lang.reflect.Method.invoke(Method.java:597)
  at org.apache.hadoop.util.RunJar.main(RunJar.java:156)

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




Re: Review Request: HIVE-2139: Enables HiveServer to accept -hiveconf option

2011-07-18 Thread Carl Steinbach

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/958/#review1106
---



common/build.xml
https://reviews.apache.org/r/958/#comment2245

This target overrides the definition of 'compile' in build-common.xml, but 
neglects to include this action which copies over hive-log4j.properties:

copy todir=${build.classes} failonerror=false
  fileset dir=${src.dir}/conf/
/copy

If this is added in I think the missing hive-log4j.properties issue will be 
resolved.



metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
https://reviews.apache.org/r/958/#comment2244

This should be $METASTORE_PORT instead of HIVE_PORT (see metastore.sh).


- Carl


On 2011-07-15 23:51:55, Patrick Hunt wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/958/
 ---
 
 (Updated 2011-07-15 23:51:55)
 
 
 Review request for hive and Carl Steinbach.
 
 
 Summary
 ---
 
 This patch updates HiveServer and HiveMetastore to add proper cli handling - 
 similar to that used in CliDriver (ie GnuParser).
 
 There's a common HiveCli class that's used by both main classes.
 
 I've attempted to make the cli's backward compatible with the prior command 
 line processing. Notice I've deprecated (via warnings, but the code still 
 runs) if the old style CLI usage is used.
 
 commands such as the following now work as expected:
 
 bin/hive --service hiveserver -t 200 -p 12000 --hiveconf 
 hive.root.logger=DEBUG,console
 
 as does the following which generates usage information:
 
 bin/hive --service hiveserver -h
 
 Note: HiveMetastore as not initializing log4j, I updated the code to do 
 similar to HiveServer (otw the hiveconf hive.root.logger option above didn't 
 work).
 
 
 This addresses bug HIVE-2139.
 https://issues.apache.org/jira/browse/HIVE-2139
 
 
 Diffs
 -
 
   ql/src/test/org/apache/hadoop/hive/ql/history/TestHiveHistory.java 728958d 
   service/src/java/org/apache/hadoop/hive/service/HiveServer.java ea04be9 
   ql/src/java/org/apache/hadoop/hive/ql/exec/ExecDriver.java 090ecfc 
   ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java 61393c8 
   bin/ext/hiveserver.sh b5edce4 
   bin/ext/metastore.sh db15f6e 
   cli/src/java/org/apache/hadoop/hive/cli/CliDriver.java bd9f6b1 
   common/build.xml 0969e16 
   common/ivy.xml bee3bf0 
   common/src/java/org/apache/hadoop/hive/common/LogUtils.java PRE-CREATION 
   common/src/java/org/apache/hadoop/hive/common/cli/CommonCliOptions.java 
 PRE-CREATION 
   hwi/src/java/org/apache/hadoop/hive/hwi/HWISessionItem.java 1bf8c97 
   metastore/ivy.xml 83218fc 
   metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java 
 e540c63 
 
 Diff: https://reviews.apache.org/r/958/diff
 
 
 Testing
 ---
 
 I couldn't find any tests for these changes, so I verfied the changes 
 manually.
 
 
 Thanks,
 
 Patrick
 




[jira] [Updated] (HIVE-2139) Enables HiveServer to accept -hiveconf option

2011-07-18 Thread Carl Steinbach (JIRA)

 [ 
https://issues.apache.org/jira/browse/HIVE-2139?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Carl Steinbach updated HIVE-2139:
-

Status: Open  (was: Patch Available)

 Enables HiveServer to accept -hiveconf option
 -

 Key: HIVE-2139
 URL: https://issues.apache.org/jira/browse/HIVE-2139
 Project: Hive
  Issue Type: Improvement
  Components: CLI
 Environment: Linux + CDH3u0 (Hive 0.7.0+27.1-2~lucid-cdh3)
Reporter: Kazuki Ohta
Assignee: Patrick Hunt
 Attachments: HIVE-2139.patch, HIVE-2139.patch


 Currently, I'm trying to test HiveHBaseIntegration on HiveServer. But it 
 doesn't seem to accept -hiveconf command.
 {code}
 hive --service hiveserver -hiveconf hbase.zookeeper.quorum=hdp0,hdp1,hdp2
 Starting Hive Thrift Server
 java.lang.NumberFormatException: For input string: -hiveconf
 at 
 java.lang.NumberFormatException.forInputString(NumberFormatException.java:48)
 at java.lang.Integer.parseInt(Integer.java:449)
 at java.lang.Integer.parseInt(Integer.java:499)
 at org.apache.hadoop.hive.service.HiveServer.main(HiveServer.java:382)
 at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
 at 
 sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
 at 
 sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
 at java.lang.reflect.Method.invoke(Method.java:597)
 at org.apache.hadoop.util.RunJar.main(RunJar.java:186)
 {code}
 Therefore, you need to throw the query like set 
 hbase.zookeeper.quorum=hdp0,hdp1,hdp2 everytime. It's not convenient for 
 separating the configuration between server-side and client-side.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HIVE-2139) Enables HiveServer to accept -hiveconf option

2011-07-18 Thread jirapos...@reviews.apache.org (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-2139?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13067411#comment-13067411
 ] 

jirapos...@reviews.apache.org commented on HIVE-2139:
-


---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/958/#review1106
---



common/build.xml
https://reviews.apache.org/r/958/#comment2245

This target overrides the definition of 'compile' in build-common.xml, but 
neglects to include this action which copies over hive-log4j.properties:

copy todir=${build.classes} failonerror=false
  fileset dir=${src.dir}/conf/
/copy

If this is added in I think the missing hive-log4j.properties issue will be 
resolved.



metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
https://reviews.apache.org/r/958/#comment2244

This should be $METASTORE_PORT instead of HIVE_PORT (see metastore.sh).


- Carl


On 2011-07-15 23:51:55, Patrick Hunt wrote:
bq.  
bq.  ---
bq.  This is an automatically generated e-mail. To reply, visit:
bq.  https://reviews.apache.org/r/958/
bq.  ---
bq.  
bq.  (Updated 2011-07-15 23:51:55)
bq.  
bq.  
bq.  Review request for hive and Carl Steinbach.
bq.  
bq.  
bq.  Summary
bq.  ---
bq.  
bq.  This patch updates HiveServer and HiveMetastore to add proper cli handling 
- similar to that used in CliDriver (ie GnuParser).
bq.  
bq.  There's a common HiveCli class that's used by both main classes.
bq.  
bq.  I've attempted to make the cli's backward compatible with the prior 
command line processing. Notice I've deprecated (via warnings, but the code 
still runs) if the old style CLI usage is used.
bq.  
bq.  commands such as the following now work as expected:
bq.  
bq.  bin/hive --service hiveserver -t 200 -p 12000 --hiveconf 
hive.root.logger=DEBUG,console
bq.  
bq.  as does the following which generates usage information:
bq.  
bq.  bin/hive --service hiveserver -h
bq.  
bq.  Note: HiveMetastore as not initializing log4j, I updated the code to do 
similar to HiveServer (otw the hiveconf hive.root.logger option above didn't 
work).
bq.  
bq.  
bq.  This addresses bug HIVE-2139.
bq.  https://issues.apache.org/jira/browse/HIVE-2139
bq.  
bq.  
bq.  Diffs
bq.  -
bq.  
bq.ql/src/test/org/apache/hadoop/hive/ql/history/TestHiveHistory.java 
728958d 
bq.service/src/java/org/apache/hadoop/hive/service/HiveServer.java ea04be9 
bq.ql/src/java/org/apache/hadoop/hive/ql/exec/ExecDriver.java 090ecfc 
bq.ql/src/java/org/apache/hadoop/hive/ql/session/SessionState.java 61393c8 
bq.bin/ext/hiveserver.sh b5edce4 
bq.bin/ext/metastore.sh db15f6e 
bq.cli/src/java/org/apache/hadoop/hive/cli/CliDriver.java bd9f6b1 
bq.common/build.xml 0969e16 
bq.common/ivy.xml bee3bf0 
bq.common/src/java/org/apache/hadoop/hive/common/LogUtils.java PRE-CREATION 
bq.common/src/java/org/apache/hadoop/hive/common/cli/CommonCliOptions.java 
PRE-CREATION 
bq.hwi/src/java/org/apache/hadoop/hive/hwi/HWISessionItem.java 1bf8c97 
bq.metastore/ivy.xml 83218fc 
bq.metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java 
e540c63 
bq.  
bq.  Diff: https://reviews.apache.org/r/958/diff
bq.  
bq.  
bq.  Testing
bq.  ---
bq.  
bq.  I couldn't find any tests for these changes, so I verfied the changes 
manually.
bq.  
bq.  
bq.  Thanks,
bq.  
bq.  Patrick
bq.  
bq.



 Enables HiveServer to accept -hiveconf option
 -

 Key: HIVE-2139
 URL: https://issues.apache.org/jira/browse/HIVE-2139
 Project: Hive
  Issue Type: Improvement
  Components: CLI
 Environment: Linux + CDH3u0 (Hive 0.7.0+27.1-2~lucid-cdh3)
Reporter: Kazuki Ohta
Assignee: Patrick Hunt
 Attachments: HIVE-2139.patch, HIVE-2139.patch


 Currently, I'm trying to test HiveHBaseIntegration on HiveServer. But it 
 doesn't seem to accept -hiveconf command.
 {code}
 hive --service hiveserver -hiveconf hbase.zookeeper.quorum=hdp0,hdp1,hdp2
 Starting Hive Thrift Server
 java.lang.NumberFormatException: For input string: -hiveconf
 at 
 java.lang.NumberFormatException.forInputString(NumberFormatException.java:48)
 at java.lang.Integer.parseInt(Integer.java:449)
 at java.lang.Integer.parseInt(Integer.java:499)
 at org.apache.hadoop.hive.service.HiveServer.main(HiveServer.java:382)
 at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
 at 
 sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
 at 
 

Re: Review Request: HIVE-2286: ClassCastException when building index with security.authorization turned on

2011-07-18 Thread John Sichi

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1137/#review1107
---



ql/src/java/org/apache/hadoop/hive/ql/index/bitmap/BitmapIndexHandler.java
https://reviews.apache.org/r/1137/#comment2246

Two comments on this fix:

1) I think we need a helper method, since we've started using 
Driver.compile for reentrant SQL in a few places (there's actually another call 
both here and in CompactIndexHandler, and one more in DDLSemanticAnalyzer).

2) While we're at it, we can make the helper method save/restore any other 
statement-specific state (command string?).




- John


On 2011-07-15 22:33:48, Syed Albiz wrote:
 
 ---
 This is an automatically generated e-mail. To reply, visit:
 https://reviews.apache.org/r/1137/
 ---
 
 (Updated 2011-07-15 22:33:48)
 
 
 Review request for hive, John Sichi and Ning Zhang.
 
 
 Summary
 ---
 
 Save the original HiveOperation/commandType when we generate the index 
 builder task and restore it after we're done generating the task so that the 
 authorization checks make the right decision when deciding what to do.
 
 
 This addresses bug HIVE-2286.
 https://issues.apache.org/jira/browse/HIVE-2286
 
 
 Diffs
 -
 
   ql/src/test/results/clientpositive/index_auth.q.out PRE-CREATION 
   ql/src/java/org/apache/hadoop/hive/ql/index/bitmap/BitmapIndexHandler.java 
 61bbbf5 
   
 ql/src/java/org/apache/hadoop/hive/ql/index/compact/CompactIndexHandler.java 
 7c91946 
   ql/src/test/queries/clientpositive/index_auth.q PRE-CREATION 
 
 Diff: https://reviews.apache.org/r/1137/diff
 
 
 Testing
 ---
 
 Added new testcase to TestCliDriver: index_auth.q
 
 
 Thanks,
 
 Syed
 




[jira] [Updated] (HIVE-2286) ClassCastException when building index with security.authorization turned on

2011-07-18 Thread John Sichi (JIRA)

 [ 
https://issues.apache.org/jira/browse/HIVE-2286?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

John Sichi updated HIVE-2286:
-

Status: Open  (was: Patch Available)

Comment added on Review Board.

 ClassCastException when building index with security.authorization turned on
 

 Key: HIVE-2286
 URL: https://issues.apache.org/jira/browse/HIVE-2286
 Project: Hive
  Issue Type: Bug
Reporter: Syed S. Albiz
Assignee: Syed S. Albiz
 Attachments: HIVE-2286.1.patch


 When trying to build an index with authorization checks turned on, hive 
 issues the following ClassCastException:
 org.apache.hadoop.hive.ql.parse.DDLSemanticAnalyzer cannot be cast to
 org.apache.hadoop.hive.ql.parse.SemanticAnalyzer
  at
 org.apache.hadoop.hive.ql.Driver.doAuthorization(Driver.java:540)
  at org.apache.hadoop.hive.ql.Driver.compile(Driver.java:431)
  at org.apache.hadoop.hive.ql.Driver.compile(Driver.java:335)
  at org.apache.hadoop.hive.ql.Driver.run(Driver.java:848)
  at
 org.apache.hadoop.hive.cli.CliDriver.processCmd(CliDriver.java:224)
  at
 org.apache.hadoop.hive.cli.CliDriver.processLine(CliDriver.java:358)
  at
 org.apache.hadoop.hive.cli.CliDriver.processLine(CliDriver.java:293)
  at
 org.apache.hadoop.hive.cli.CliDriver.processReader(CliDriver.java:385)
  at
 org.apache.hadoop.hive.cli.CliDriver.processFile(CliDriver.java:392)
  at
 org.apache.hadoop.hive.cli.CliDriver.main(CliDriver.java:567)
  at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
  at
 sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.jav
 a:39)
  at
 sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessor
 Impl.java:25)
  at java.lang.reflect.Method.invoke(Method.java:597)
  at org.apache.hadoop.util.RunJar.main(RunJar.java:156)

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HIVE-956) Add support of columnar binary serde

2011-07-18 Thread He Yongqiang (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-956?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13067453#comment-13067453
 ] 

He Yongqiang commented on HIVE-956:
---

The patch looks good. One minor comments about LazyBinaryColumnarSerDe is that 
could some code be moved to some utils or some base class to share with 
ColumnarSerDe and/or LazyBinarySerde?

Can you confirm that it is safe to remove 'nullSequence' from 
ColumnarStructObjectInspector and ObjectInspectorFactory?

 Add support of columnar binary serde
 

 Key: HIVE-956
 URL: https://issues.apache.org/jira/browse/HIVE-956
 Project: Hive
  Issue Type: New Feature
Reporter: He Yongqiang
Assignee: Krishna Kumar
 Attachments: HIVE-956v3.patch, HIVE.956.patch.0, HIVE.956.patch.1, 
 HIVE.956.patch.2




--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Updated] (HIVE-2278) Support archiving for multiple partitions if the table is partitioned by multiple columns

2011-07-18 Thread Marcin Kurczych (JIRA)

 [ 
https://issues.apache.org/jira/browse/HIVE-2278?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Marcin Kurczych updated HIVE-2278:
--

Attachment: hive.2278.1.patch

First version of a patch, not really ready yet

 Support archiving for multiple partitions if the table is partitioned by 
 multiple columns
 -

 Key: HIVE-2278
 URL: https://issues.apache.org/jira/browse/HIVE-2278
 Project: Hive
  Issue Type: New Feature
Reporter: Namit Jain
Assignee: Marcin Kurczych
 Attachments: hive.2278.1.patch


 If a table is partitioned by ds,hr
 it should be possible to archive all the files in ds to reduce the number of 
 files

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Updated] (HIVE-2278) Support archiving for multiple partitions if the table is partitioned by multiple columns

2011-07-18 Thread Marcin Kurczych (JIRA)

 [ 
https://issues.apache.org/jira/browse/HIVE-2278?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Marcin Kurczych updated HIVE-2278:
--

Attachment: (was: hive.2278.1.patch)

 Support archiving for multiple partitions if the table is partitioned by 
 multiple columns
 -

 Key: HIVE-2278
 URL: https://issues.apache.org/jira/browse/HIVE-2278
 Project: Hive
  Issue Type: New Feature
Reporter: Namit Jain
Assignee: Marcin Kurczych

 If a table is partitioned by ds,hr
 it should be possible to archive all the files in ds to reduce the number of 
 files

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




Re: Review Request: HIVE-2128: Automatic Indexing with multiple tables

2011-07-18 Thread Syed Albiz

---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1010/
---

(Updated 2011-07-19 03:15:17.006396)


Review request for hive and John Sichi.


Changes
---

removed unnecessary imports from patch


Summary
---

Grab the indexed tables during optimized query generation, grab the associated 
path URIs, and keep those around in the Configuration object. When the job is 
passed to ExecDriver, this data is extracted and used in HiveIndexedInputFormat 
to decide whether to use the index file or delegate to the parent 
(HiveInputFormat) class. Not sure if this is robust. 


This addresses bug HIVE-2128.
https://issues.apache.org/jira/browse/HIVE-2128


Diffs (updated)
-

  ql/src/test/results/clientpositive/index_auto_self_join.q.out PRE-CREATION 
  ql/src/test/results/clientpositive/index_auto_mult_tables_compact.q.out 
PRE-CREATION 
  ql/src/test/queries/clientpositive/index_auto_self_join.q PRE-CREATION 
  ql/src/test/results/clientpositive/index_auto_mult_tables.q.out PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/plan/MapredWork.java a03a9a6 
  ql/src/test/queries/clientpositive/index_auto_mult_tables.q PRE-CREATION 
  ql/src/test/queries/clientpositive/index_auto_mult_tables_compact.q 
PRE-CREATION 
  ql/src/java/org/apache/hadoop/hive/ql/index/HiveIndexResult.java b9b586e 
  ql/src/java/org/apache/hadoop/hive/ql/index/HiveIndexedInputFormat.java 
f1ee95d 
  ql/src/java/org/apache/hadoop/hive/ql/index/bitmap/BitmapIndexHandler.java 
61bbbf5 
  ql/src/java/org/apache/hadoop/hive/ql/index/compact/CompactIndexHandler.java 
7c91946 
  
ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/index/IndexWhereProcessor.java
 dbc489f 
  ql/src/java/org/apache/hadoop/hive/ql/index/HiveIndexQueryContext.java 
617723e 

Diff: https://reviews.apache.org/r/1010/diff


Testing
---

added new testcase index_auto_mult_tables.q


Thanks,

Syed



[jira] [Updated] (HIVE-2128) Automatic Indexing with multiple tables

2011-07-18 Thread Syed S. Albiz (JIRA)

 [ 
https://issues.apache.org/jira/browse/HIVE-2128?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Syed S. Albiz updated HIVE-2128:


Attachment: HIVE-2128.5.patch

removed unnecessary imports

 Automatic Indexing with multiple tables
 ---

 Key: HIVE-2128
 URL: https://issues.apache.org/jira/browse/HIVE-2128
 Project: Hive
  Issue Type: Improvement
  Components: Indexing
Affects Versions: 0.8.0
Reporter: Russell Melick
Assignee: Syed S. Albiz
 Attachments: HIVE-2128.1.patch, HIVE-2128.1.patch, HIVE-2128.2.patch, 
 HIVE-2128.4.patch, HIVE-2128.5.patch


 Make automatic indexing work with jobs which access multiple tables.  We'll 
 probably need to modify the way that the index input format works in order to 
 associate index formats/files with specific tables.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HIVE-2156) Improve error messages emitted during task execution

2011-07-18 Thread Ning Zhang (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-2156?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13067510#comment-13067510
 ] 

Ning Zhang commented on HIVE-2156:
--

Hi Syed, it looks good in general. Can you rebase this diff to the current 
trunk? I'll start testing it then. 

 Improve error messages emitted during task execution
 

 Key: HIVE-2156
 URL: https://issues.apache.org/jira/browse/HIVE-2156
 Project: Hive
  Issue Type: Improvement
Reporter: Syed S. Albiz
Assignee: Syed S. Albiz
 Attachments: HIVE-2156.1.patch, HIVE-2156.2.patch


 Follow-up to HIVE-1731
 A number of issues were related to reporting errors from task execution and 
 surfacing these in a more useful form.
 Currently a cryptic message with Execution Error and a return code and 
 class name of the task is emitted.
 The most useful log messages here are emitted to the local logs, which can be 
 found through jobtracker. Having either a pointer to these logs as part of 
 the error message or the actual content would improve the usefulness 
 substantially. It may also warrant looking into how the underlying error 
 reporting through Hadoop is done and if more information can be propagated up 
 from there.
 Specific issues raised in  HIVE-1731:
 FAILED: Execution Error, return code 2 from 
 org.apache.hadoop.hive.ql.exec.MapRedTask
 * issue was in regexp_extract syntax
 FAILED: Execution Error, return code 1 from 
 org.apache.hadoop.hive.ql.exec.DDLTask
 * tried: desc table_does_not_exist;

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (HIVE-2086) Data loss with external table

2011-07-18 Thread Jonathan Natkins (JIRA)

[ 
https://issues.apache.org/jira/browse/HIVE-2086?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13067522#comment-13067522
 ] 

Jonathan Natkins commented on HIVE-2086:


It looks like this was actually just fixed today, as a part of HIVE-1218.  
However, that fix didn't add any regression tests for this bug, so I'm going to 
prepare an updated patch once I finish running the test suite.

 Data loss with external table
 -

 Key: HIVE-2086
 URL: https://issues.apache.org/jira/browse/HIVE-2086
 Project: Hive
  Issue Type: Bug
  Components: Metastore
Affects Versions: 0.7.0
 Environment: Amazon  elastics mapreduce cluster
Reporter: Q Long
Assignee: Jonathan Natkins
 Attachments: HIVE-2086.1.patch, HIVE-2086.2.patch, create_like.q.out


 Data loss when using create external table like statement. 
 1) Set up an external table S, point to location L. Populate data in S.
 2) Create another external table T, using statement like this:
 create external table T like S location L
Make sure table T point to the same location as the original table S.
 3) Query table T, see the same set of data in S.
 4) drop table T.
 5) Query table S will return nothing, and location L is deleted. 

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira