[jira] [Created] (HIVE-14615) Temp table leaves behind insert command

2016-08-23 Thread Chaoyu Tang (JIRA)
Chaoyu Tang created HIVE-14615:
--

 Summary: Temp table leaves behind insert command
 Key: HIVE-14615
 URL: https://issues.apache.org/jira/browse/HIVE-14615
 Project: Hive
  Issue Type: Bug
  Components: Query Processor
Reporter: Chaoyu Tang
Assignee: Chaoyu Tang


{code}
create table test (key int, value string);
insert into test values (1, 'val1');
show tables;
test
values__tmp__table__1
{code}
the temp table values__tmp__table__1 was resulted from insert into ...values
and exists until logout the session.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


Review Request 51355: Bypass CBO return path for hive.test.mode

2016-08-23 Thread Vineet Garg

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

Review request for hive and Ashutosh Chauhan.


Bugs: HIVE-14610
https://issues.apache.org/jira/browse/HIVE-14610


Repository: hive-git


Description
---

Turning hive.test.mode on will try generating table sampling which isn't yet 
supported for CBO return path


Diffs
-

  ql/src/java/org/apache/hadoop/hive/ql/parse/CalcitePlanner.java 7c50155 
  ql/src/test/queries/clientpositive/input30.q 7859bbc 
  ql/src/test/results/clientpositive/input30.q.out e728f04 

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


Testing
---

* Updated texisting test to add a new test
* Pre-commit testing


Thanks,

Vineet Garg



[jira] [Created] (HIVE-14614) Insert overwrite local directory fails with IllegalStateException

2016-08-23 Thread Vihang Karajgaonkar (JIRA)
Vihang Karajgaonkar created HIVE-14614:
--

 Summary: Insert overwrite local directory fails with 
IllegalStateException
 Key: HIVE-14614
 URL: https://issues.apache.org/jira/browse/HIVE-14614
 Project: Hive
  Issue Type: Bug
Reporter: Vihang Karajgaonkar
Assignee: Vihang Karajgaonkar
Priority: Minor


insert overwrite local directory  select * from table; fails with 
"java.lang.IllegalStateException: Cannot create staging directory" when the 
path sent to the getTempDirForPath(Path path)  is a local fs path.

This is a regression caused by the fix for HIVE-14270



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (HIVE-14613) Move schema evolution tests to MiniLlap and disable LLAP IO

2016-08-23 Thread Prasanth Jayachandran (JIRA)
Prasanth Jayachandran created HIVE-14613:


 Summary: Move schema evolution tests to MiniLlap and disable LLAP 
IO
 Key: HIVE-14613
 URL: https://issues.apache.org/jira/browse/HIVE-14613
 Project: Hive
  Issue Type: Sub-task
Affects Versions: 2.2.0
Reporter: Prasanth Jayachandran
Assignee: Prasanth Jayachandran


Move the slow schema evolution tests from TestCliDriver to 
TestMiniLlapCliDriver and disable LLAP IO for these tests so that non-llap 
reader codepath is used. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (HIVE-14612) org.apache.hive.service.cli.operation.TestOperationLoggingLayout.testSwitchLogLayout failure

2016-08-23 Thread Hari Sankar Sivarama Subramaniyan (JIRA)
Hari Sankar Sivarama Subramaniyan created HIVE-14612:


 Summary: 
org.apache.hive.service.cli.operation.TestOperationLoggingLayout.testSwitchLogLayout
 failure
 Key: HIVE-14612
 URL: https://issues.apache.org/jira/browse/HIVE-14612
 Project: Hive
  Issue Type: Sub-task
Reporter: Hari Sankar Sivarama Subramaniyan
Assignee: Hari Sankar Sivarama Subramaniyan


Failing for some time



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (HIVE-14611) CBO: Calcite Operator To Hive Operator(Calcite Return Path): Support Tablesample clause

2016-08-23 Thread Vineet Garg (JIRA)
Vineet Garg created HIVE-14611:
--

 Summary: CBO: Calcite Operator To Hive Operator(Calcite Return 
Path): Support Tablesample clause
 Key: HIVE-14611
 URL: https://issues.apache.org/jira/browse/HIVE-14611
 Project: Hive
  Issue Type: Sub-task
  Components: CBO
Reporter: Vineet Garg
Assignee: Vineet Garg


Currently tablesample clause is not supported in return path.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


[jira] [Created] (HIVE-14610) CBO: Calcite Operator To Hive Operator(Calcite Return Path): Fix wrong result in input30

2016-08-23 Thread Vineet Garg (JIRA)
Vineet Garg created HIVE-14610:
--

 Summary: CBO: Calcite Operator To Hive Operator(Calcite Return 
Path): Fix wrong result in input30
 Key: HIVE-14610
 URL: https://issues.apache.org/jira/browse/HIVE-14610
 Project: Hive
  Issue Type: Sub-task
  Components: CBO
Reporter: Vineet Garg
Assignee: Vineet Garg


Setup:
{code}
set hive.mapred.mode=nonstrict;

create table dest30(a int);
create table tst_dest30(a int);

set hive.test.mode=true;
set hive.test.mode.prefix=tst_;

insert overwrite table dest30
select count(1) from src;  
{code}

Query:
{code} select * from tst_dest30; {code}

Expected Result:
{code} 10 {code}

Actual Result:
{code} 500 {code}

Turning on hive test mode is suppose to generate sampling (implicit tablesample 
clause) but return path isn't honoring hive.test.mode flag



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)


Re: Review Request 51334: HIVE-14404: Allow delimiterfordsv to use multiple-character delimiters - second version

2016-08-23 Thread Marta Kuczora

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

(Updated Aug. 23, 2016, 2:45 p.m.)


Review request for hive, Naveen Gangam, Peter Vary, Sergio Pena, and Szehon Ho.


Summary (updated)
-

HIVE-14404: Allow delimiterfordsv to use multiple-character delimiters - second 
version


Bugs: HIVE-14404
https://issues.apache.org/jira/browse/HIVE-14404


Repository: hive-git


Description
---

Changed the class which is used to generate the output for the formats 'csv2', 
'tsv2' and 'dsv' not to use SuperCSV any more and support multiple characters 
as delimiter.
The class implements the same escaping logic as it had with SuperCSV if the 
quoting is not disabled.
Extended the TestBeeLineWithArgs tests with new test steps which are using 
multiple characters as delimiter.

This review is a follow-up for the review 50896.


Diffs
-

  beeline/pom.xml dc89e81 
  beeline/src/java/org/apache/hive/beeline/BeeLine.java e0fa032 
  beeline/src/java/org/apache/hive/beeline/BeeLineOpts.java e6e24b1 
  beeline/src/java/org/apache/hive/beeline/SeparatedValuesOutputFormat.java 
66d9fd0 
  
itests/hive-unit/src/test/java/org/apache/hive/beeline/TestBeeLineWithArgs.java 
49c1120 

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


Testing
---

- Tested manually in BeeLine.
- Extended the TestBeeLineWithArgs tests with new test steps which are using 
multiple characters as delimiter.


Thanks,

Marta Kuczora



Review Request 51334: HIVE-14404: Allow delimiterfordsv to use multiple-character delimiters

2016-08-23 Thread Marta Kuczora

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

Review request for hive, Naveen Gangam, Peter Vary, Sergio Pena, and Szehon Ho.


Bugs: HIVE-14404
https://issues.apache.org/jira/browse/HIVE-14404


Repository: hive-git


Description
---

Changed the class which is used to generate the output for the formats 'csv2', 
'tsv2' and 'dsv' not to use SuperCSV any more and support multiple characters 
as delimiter.
The class implements the same escaping logic as it had with SuperCSV if the 
quoting is not disabled.
Extended the TestBeeLineWithArgs tests with new test steps which are using 
multiple characters as delimiter.

This review is a follow-up for the review 50896.


Diffs
-

  beeline/pom.xml dc89e81 
  beeline/src/java/org/apache/hive/beeline/BeeLine.java e0fa032 
  beeline/src/java/org/apache/hive/beeline/BeeLineOpts.java e6e24b1 
  beeline/src/java/org/apache/hive/beeline/SeparatedValuesOutputFormat.java 
66d9fd0 
  
itests/hive-unit/src/test/java/org/apache/hive/beeline/TestBeeLineWithArgs.java 
49c1120 

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


Testing
---

- Tested manually in BeeLine.
- Extended the TestBeeLineWithArgs tests with new test steps which are using 
multiple characters as delimiter.


Thanks,

Marta Kuczora



Re: Review Request 51287: The ConsoleReader.readLine method never echoes back the executed line. Instead, Beeline.info is used, which is redirected to stderr with other messages. On interactive mode b

2016-08-23 Thread Miklos Csanady


> On Aug. 23, 2016, 12:34 a.m., Sergio Pena wrote:
> > Could you paste the test you made for this patch? like before & after the 
> > patch output?

I have discovered that it breaks some of the tests  in 
org.apache.beeline.cli.TestHiveCli.java. 
Now I try to investigate this and modify them if it seems to be the best 
solution.


- Miklos


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


On Aug. 22, 2016, 5:40 p.m., Miklos Csanady wrote:
> 
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/51287/
> ---
> 
> (Updated Aug. 22, 2016, 5:40 p.m.)
> 
> 
> Review request for hive, Peter Vary, Sergio Pena, and Vihang Karajgaonkar.
> 
> 
> Bugs: HIVE-14525
> https://issues.apache.org/jira/browse/HIVE-14525
> 
> 
> Repository: hive-git
> 
> 
> Description
> ---
> 
> HIVE-14525: beeline still writing log data to stdout as of version 2.1.0
> 
> 
> Diffs
> -
> 
>   beeline/src/java/org/apache/hive/beeline/BeeLine.java e0fa032 
> 
> Diff: https://reviews.apache.org/r/51287/diff/
> 
> 
> Testing
> ---
> 
> Beeline with redirected input and output with script file and without one.
> 
> 
> Thanks,
> 
> Miklos Csanady
> 
>