Re: Hive 0.7.0 Release Candidate 0

2011-02-18 Thread Ashutosh Chauhan
Wondering if https://issues.apache.org/jira/browse/HIVE-1995 should also be considered for 0.7 ? Ashutosh On Thu, Feb 17, 2011 at 23:57, Carl Steinbach c...@cloudera.com wrote: http://people.apache.org/~cws/hive-0.7.0-candidate-0/ Please vote.

Re: Hive 0.7.0 Release Candidate 0

2011-02-18 Thread Ashutosh Chauhan
Great. Thanks, Carl. Ashutosh On Fri, Feb 18, 2011 at 14:20, Carl Steinbach c...@cloudera.com wrote: Hi Ashutosh, I backported it just now. I'll cut another RC early next week to include this. Thanks. Carl On Fri, Feb 18, 2011 at 1:37 PM, Ashutosh Chauhan hashut...@apache.orgwrote

hooks in metastore functions

2011-03-08 Thread Ashutosh Chauhan
Hi all, I have a requirement that every time some change on metastore takes place, we have some logic which needs to be run. For example, if a new table is getting created in metastore I want to send a message to a message bus. Easiest way for this to work is to add the logic in createTable().

Re: hooks in metastore functions

2011-03-09 Thread Ashutosh Chauhan
handler)? JVS On Mar 8, 2011, at 2:12 PM, Ashutosh Chauhan wrote: Hi all, I have a requirement that every time some change on metastore takes place, we have some logic which needs to be run. For example, if a new table is getting created in metastore I want to send a message to a message

Re: Review Request: Review request for HIVE-2038

2011-04-12 Thread Ashutosh Chauhan
too. - Ashutosh --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/581/#review428 --- On 2011-04-12 01:29:41, Ashutosh Chauhan wrote

Review Request: Removed finalizePartition() from the patch

2011-04-23 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/648/ --- Review request for hive and Carl Steinbach. Summary --- 1. Removed

Review Request: Get rid of System.exit

2011-04-27 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/668/ --- Review request for hive, Carl Steinbach, John Sichi, and Paul Yang. Summary

Review Request: Refactor HiveMetaStore to make it maintainable

2011-04-27 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/669/ --- Review request for hive, Carl Steinbach, John Sichi, and Paul Yang. Summary

Re: Review Request: Refactor HiveMetaStore to make it maintainable

2011-04-27 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/669/ --- (Updated 2011-04-27 18:24:42.458082) Review request for hive, Carl Steinbach,

Re: ANNOUNCE: New PMC Member Carl Steinbach

2011-04-28 Thread Ashutosh Chauhan
Congrats, Carl ! On Thu, Apr 28, 2011 at 05:39, Ashish Thusoo athu...@fb.com wrote: Congratulations Carl.. Ashish On Apr 27, 2011, at 7:09 PM, John Sichi wrote: Hi all, The Hive Project Management Committee is happy to announce that Carl Steinbach has been voted in as a new PMC member.  

Review Request: HIVE-2147 : Add api to send / receive message to metastore

2011-05-12 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/738/ --- Review request for hive and Carl Steinbach. Summary --- Updated patch to

Re: Review Request: HIVE-2160 : Few code improvements in the metastore, hwi and ql packages.

2011-05-13 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/742/#review667 --- Ship it! Thanks Chinna for the cleanup work. Looks good to me. -

Re: Review Request: HIVE-2147 : Add api to send / receive message to metastore

2011-05-25 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/738/#review713 --- On 2011-05-12 21:03:29, Ashutosh Chauhan wrote: --- This is an automatically generated e-mail

Re: Review Request: HIVE-2188: Add a function to retrieve multiple tables on trip to the hive metastore

2011-06-03 Thread Ashutosh Chauhan
-06-02 23:01:00) Review request for hive, Paul Yang and Ashutosh Chauhan. Summary --- Created a function multi_get_table that retrieves multiple tables on one trip to the hive metastore, saving round trip time. This addresses bug HIVE-2188. https://issues.apache.org/jira

Review Request: HIVE-2215

2011-06-10 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/883/ --- Review request for hive and John Sichi. Summary --- Follow-up for

Re: Review Request: HIVE-2215

2011-06-13 Thread Ashutosh Chauhan
--- On 2011-06-10 21:24:13, Ashutosh Chauhan wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/883

Re: Review Request: HIVE-2215

2011-06-14 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/883/ --- (Updated 2011-06-14 20:51:53.179968) Review request for hive and John Sichi.

Re: Review Request: HIVE-2215

2011-06-14 Thread Ashutosh Chauhan
, Ashutosh Chauhan wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/883/ --- (Updated 2011-06-14 20:51:53) Review request for hive

Review Request: Review request for HIVE-2225

2011-06-21 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/940/ --- Review request for hive, Carl Steinbach and John Sichi. Summary --- This

Re: Review Request: Review request for HIVE-2225

2011-06-22 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/940/#review888 --- On 2011-06-21 17:34:28, Ashutosh Chauhan wrote

Re: Review Request: Review request for HIVE-2225

2011-06-22 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/940/ --- (Updated 2011-06-23 02:55:08.540561) Review request for hive, Carl Steinbach and

Re: Review Request: Review request for HIVE-2225

2011-06-22 Thread Ashutosh Chauhan
--- On 2011-06-23 02:55:08, Ashutosh Chauhan wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/940

Re: Review Request: Review request for HIVE-2225

2011-06-22 Thread Ashutosh Chauhan
? Yup, it is. - Ashutosh --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/940/#review891 --- On 2011-06-23 02:55:08, Ashutosh Chauhan wrote

Re: Review Request: HIVE-1537 - Allow users to specify LOCATION in CREATE DATABASE statement

2011-06-23 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/949/#review898 ---

Re: Review Request: HIVE-1537 - Allow users to specify LOCATION in CREATE DATABASE statement

2011-06-24 Thread Ashutosh Chauhan
On 2011-06-23 16:49:59, Ashutosh Chauhan wrote: trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java, line 591 https://reviews.apache.org/r/949/diff/1/?file=21560#file21560line591 This may not be always successful. You may fail to create dirs for number

Re: Hive projects for Google Summer of code 2012 ?

2012-02-04 Thread Ashutosh Chauhan
Hey Bharath, Great to see your enthusiasm for Hive! I would be happy to mentor you for the project. For the start, you can take a look at https://cwiki.apache.org/confluence/display/Hive/Roadmap for a list of open projects in Hive. The document is bit dated, so some of those projects may not be

Re: Hive projects for Google Summer of code 2012 ?

2012-02-04 Thread Ashutosh Chauhan
Ashutosh, I'm interesting in hive, I'd like to improve the compilation process, I have been that the plan query tree generated by Hive can be optimized, and I'd like to participate in Google Summer of code 2012. What do you say? Regards. El 4 de febrero de 2012 12:29, Ashutosh Chauhan hashut

Re: Hive build is back to green

2012-02-16 Thread Ashutosh Chauhan
Good idea! Created HIVE-2811 for this. Thanks, Ashutosh On Thu, Feb 16, 2012 at 14:41, Carl Steinbach c...@cloudera.com wrote: Great news! Thanks for fixing this Ashutosh! comparisons. Fix was to do export LANG=en_US.UTF-8 in environment of build machine. Is this something that we can

Re: 'arc diff' failing with Invalid or missing field 'Test Plan': You must provide a test plan.

2012-03-01 Thread Ashutosh Chauhan
Hi Carl, Include in your git commit message following line Test Plan: Include your test plan here. It is looking for string Test Plan in your commit message and fails if cant find one. Hope it helps, Ashutosh On Thu, Mar 1, 2012 at 14:45, Carl Steinbach c...@cloudera.com wrote: Hey, Today

Re: 'arc diff' failing with Invalid or missing field 'Test Plan': You must provide a test plan.

2012-03-01 Thread Ashutosh Chauhan
I don't know if there is a way to disable it. But, then I don't know much about phabricator/arc infra. On Thu, Mar 1, 2012 at 15:15, Carl Steinbach c...@cloudera.com wrote: Thanks for the tip. Is there any way to disable this behavior? On Thu, Mar 1, 2012 at 2:56 PM, Ashutosh Chauhan hashut

Re: Automatic/parallel patch testing

2012-03-07 Thread Ashutosh Chauhan
I am very much in favor of https://issues.apache.org/jira/browse/HIVE-1175 since then it publishes back on jira.. and everyone is on same page. I think Project VP has access to apache hudson. John / Namit, Can you set this up for Hive? Ashutosh On Wed, Mar 7, 2012 at 13:35, Edward Capriolo

Re: Potential bug around hive merging of small files

2012-03-13 Thread Ashutosh Chauhan
This does look like a bug. Shrijeet, mind opening a jira and attaching your patch there. Thanks, Ashutosh On Mon, Mar 12, 2012 at 16:29, Shrijeet Paliwal shrij...@rocketfuel.comwrote: I had a type in last email. Settings are as follows hive set mapred.min.split.size.per.node=10; hive

Hive 0.9 release

2012-04-02 Thread Ashutosh Chauhan
Hi all, Branch for 0.8-r2 was created on Dec 7, almost four months ago. Between then and now lots of cool stuff has landed in trunk waiting to be released and get to users. I think its a good time now to get the ball rolling for 0.9 release. If this sounds good, I would propose to cut a branch

Re: Hive 0.9 release

2012-04-02 Thread Ashutosh Chauhan
time doing a lot of backports from trunk to the release branch. It would be great if we could avoid all of that this time around. Thanks. Carl On Mon, Apr 2, 2012 at 6:33 PM, Ashutosh Chauhan hashut...@apache.org wrote: Hi all, Branch for 0.8-r2 was created on Dec 7, almost four months

Re: Hive 0.9 release

2012-04-06 Thread Ashutosh Chauhan
Thanks, Ashutosh On Mon, Apr 2, 2012 at 19:02, Thomas Weise t...@yahoo-inc.com wrote: Would be great to get HIVE-2646 included. Thanks, Thomas On 4/2/12 6:59 PM, Ashutosh Chauhan hashut...@apache.org wrote: Here is a list of jiras which I plan to get in 0.9. HIVE-2084 HIVE-2822

Re: Hive 0.9 release

2012-04-09 Thread Ashutosh Chauhan
As per the plan I am going to create a branch now. Please hold off any commits till I send an email for all-clear. Thanks, Ashutosh On Fri, Apr 6, 2012 at 15:00, Owen O'Malley omal...@apache.org wrote: I think we also need to get the RAT report cleaned up. Apache projects aren't supposed to

Re: Hive 0.9 release

2012-04-09 Thread Ashutosh Chauhan
. Thanks, Ashutosh On Mon, Apr 9, 2012 at 17:01, Ashutosh Chauhan hashut...@apache.org wrote: As per the plan I am going to create a branch now. Please hold off any commits till I send an email for all-clear. Thanks, Ashutosh On Fri, Apr 6, 2012 at 15:00, Owen O'Malley omal...@apache.org

Re: Looking at the columns table

2012-04-11 Thread Ashutosh Chauhan
Hey Ed, Your thinking is correct and has been implemented in https://issues.apache.org/jira/browse/HIVE-2246 Time to upgrade to 0.8 :) Thanks, Ashutosh On Wed, Apr 11, 2012 at 07:53, Edward Capriolo edlinuxg...@gmail.comwrote: Hey all. Our metastore in mysql is fairly large over 12GB. All

Re: Problems with Arc/Phabricator

2012-04-11 Thread Ashutosh Chauhan
+1 on moving away from arc/phabricator. It works great when it works, but most of the time it doesnt work. Ashutosh On Wed, Apr 11, 2012 at 11:57, Owen O'Malley omal...@apache.org wrote: On Wed, Apr 11, 2012 at 11:48 AM, Edward Capriolo edlinuxg...@gmail.com wrote: If we are going to switch

Re: Problems with Arc/Phabricator

2012-04-11 Thread Ashutosh Chauhan
Is mac only supported OS ? Arc doesn't work for me on linux, which is unfortunate since thats where I do all my testing. Thanks, Ashutosh On Wed, Apr 11, 2012 at 17:37, John Sichi jsi...@gmail.com wrote: CC'ing David Recordon, who can probably help with a point of contact for coordinating

[VOTE] Apache Hive 0.9.0 Release Candidate 0

2012-04-13 Thread Ashutosh Chauhan
Hey all, Apache Hive 0.9.0-rc0 is out and available at http://people.apache.org/~hashutosh/hive-0.9.0-rc0/ Release notes are available at: https://issues.apache.org/jira/secure/ReleaseNote.jspa?projectId=12310843version=12317742 Please give it a try, let us know. Hive PMC members: Please test

Re: [VOTE] Apache Hive 0.9.0 Release Candidate 0

2012-04-13 Thread Ashutosh Chauhan
Couple more points: Maven artifacts are available at https://repository.apache.org/content/repositories/orgapachehive-043/ for folks to try out. Vote runs for 3 business days so will expire on Wednesday, 4/18. Thanks, Ashutosh On Fri, Apr 13, 2012 at 11:50, Ashutosh Chauhan hashut

Re: [VOTE] Apache Hive 0.9.0 Release Candidate 0

2012-04-18 Thread Ashutosh Chauhan
Hey Lars, Thanks for taking a look. HIVE-1634 introduced new storage type for hbase tables, namely binary. Since, bug manifests itself only for binary storage type. This doesnt count as a regression since functionality for binary storage itself was added through HIVE-1634. Since, this is not a

Re: Hive 0.9 now broken on HBase 0.90 ?

2012-04-18 Thread Ashutosh Chauhan
Hi Tim, Sorry that it broke your setup. Decision to move to hbase-0.92 was made in https://issues.apache.org/jira/browse/HIVE-2748 Thanks, Ashutosh On Wed, Apr 18, 2012 at 11:42, Tim Robertson timrobertson...@gmail.comwrote: Hi all, This is my first post to hive-dev so please go easy on

Re: [VOTE] Apache Hive 0.9.0 Release Candidate 0

2012-04-19 Thread Ashutosh Chauhan
This vote stands cancelled because of various problems people have found in RC0. Thanks to all who tried RC0. I will respin RC1 shortly. Thanks, Ashutosh On Fri, Apr 13, 2012 at 11:50, Ashutosh Chauhan hashut...@apache.orgwrote: Hey all, Apache Hive 0.9.0-rc0 is out and available at http

hive 0.9.0 RC1

2012-04-19 Thread Ashutosh Chauhan
RC0 failed to pass because of variety of reasons. In the meantime, various folks have requested for inclusion of other fixes in 0.9. Following is the list. Following list is for committers to review these patches and to get them committed in 0.9 branch. These are in Patch Available status.

Re: Problems with Arc/Phabricator

2012-04-19 Thread Ashutosh Chauhan
to Git. As for the rest of the Phabricator/Gerrit/Reviewboard discussion, I think we should pick this up again at the contributor meeting on Wednesday. Thanks. Carl On Wed, Apr 11, 2012 at 12:19 PM, Ashutosh Chauhan hashut...@apache.org wrote: +1 on moving away from arc

Re: hive 0.9.0 RC1

2012-04-19 Thread Ashutosh Chauhan
://issues.apache.org/jira/browse/HIVE-2777 I see no way to access this feature from the hive QL language. Should we delay releases on features not usable? Edward On Thu, Apr 19, 2012 at 1:28 PM, Ashutosh Chauhan hashut...@apache.org wrote: RC0 failed to pass because of variety of reasons

Re: POST limit

2012-04-20 Thread Ashutosh Chauhan
at 7:35 PM, Ashutosh Chauhan hashut...@apache.org wrote: Hit a new problem with arc today: Fatal error: Uncaught exception 'Exception' with message 'Host returned HTTP/200, but invalid JSON data in response to a Conduit method call: br / bWarning/b: Unknown: POST Content-Length

[VOTE] Apache Hive 0.9.0 Release Candidate 1

2012-04-23 Thread Ashutosh Chauhan
Hey all, Apache Hive 0.9.0 Release Candidate 1 is available here: http://people.apache.org/~hashutosh/hive-0.9.0-rc1/ Maven artifacts are available here: https://repository.apache.org/content/repositories/orgapachehive-084/ Change List is available here:

FixVersion in jira

2012-04-23 Thread Ashutosh Chauhan
In case you are wondering whats up with deluge of emails from jira, here is whats going on. Committers, FixVersion is targeted for usage by committers. It should be set by a committer at the time of commit. In most cases, commit is going to be only on trunk, in which case next release version

Re: [VOTE] Apache Hive 0.9.0 Release Candidate 1

2012-04-24 Thread Ashutosh Chauhan
Unfortunately, there is a small problem in RC1 that version string in build.properties is 0.9.0-SNAPSHOT, instead of 0.9.0. So, I have to rescind this vote. I will respin the RC2 shortly. Thanks, Ashutosh On Mon, Apr 23, 2012 at 10:47, Ashutosh Chauhan hashut...@apache.orgwrote: Hey all

[VOTE] Apache Hive 0.9.0 Release Candidate 2

2012-04-24 Thread Ashutosh Chauhan
Hey all, Apache Hive 0.9.0 Release Candidate 2 is available here: http://people.apache.org/~hashutosh/hive-0.9.0-rc2/ Maven artifacts are available here: https://repository.apache.org/content/repositories/orgapachehive-094/ Change List is available here:

Re: [VOTE] Apache Hive 0.9.0 Release Candidate 2

2012-04-24 Thread Ashutosh Chauhan
Downloaded the bits. Installed on 5 node cluster. Did create table. Ran basic queries. Ran unit tests. All looks good. +1 Thanks, Ashutosh On Tue, Apr 24, 2012 at 12:29, Ashutosh Chauhan hashut...@apache.orgwrote: Hey all, Apache Hive 0.9.0 Release Candidate 2 is available here: http

Re: [VOTE] Apache Hive 0.9.0 Release Candidate 2

2012-04-27 Thread Ashutosh Chauhan
RC2 vote is closed now. I am excited to report that vote has passed. I will send out the note about availability of 0.9 bits once I finish publishing it. Thanks for all those who tested/voted the release. Ashutosh On Tue, Apr 24, 2012 at 12:29, Ashutosh Chauhan hashut...@apache.orgwrote: Hey

[ANNOUNCE] Apache Hive 0.9.0 Released

2012-04-30 Thread Ashutosh Chauhan
The Apache Hive team is proud to announce the the release of Apache Hive version 0.9.0. The Apache Hive (TM) data warehouse software facilitates querying and managing large datasets residing in distributed storage. Built on top of Apache Hadoop (TM), it provides: * Tools to enable easy data

Re: Branch 0.9 maven snapshots?

2012-04-30 Thread Ashutosh Chauhan
Hey Carl, Build is reported to be failing, though all tests are passing. Any ideas? Thanks, Ashutosh On Wed, Apr 25, 2012 at 13:40, Thomas Weise t...@yahoo-inc.com wrote: Thanks Carl! On 4/25/12 1:12 PM, Carl Steinbach c...@cloudera.com wrote: Hi Thomas, I created the job:

Re: Problems with Arc/Phabricator

2012-05-08 Thread Ashutosh Chauhan
to follow the discussion on this ticket. Thanks, Ashutosh On Thu, Apr 19, 2012 at 5:24 PM, Ashutosh Chauhan hashut...@apache.orgwrote: Problem while using arc on ubuntu $ arc patch D2871 ARC: Cannot mix P and A UNIX: No such file or directory Any ideas whats up there. Thanks, Ashutosh

Re: Problems with Arc/Phabricator

2012-05-09 Thread Ashutosh Chauhan
Doesn't work for me either. I see error message. Ashutosh On Wed, May 9, 2012 at 12:24 AM, Carl Steinbach c...@cloudera.com wrote: Actually, I take that back. After logging in I'm now back to the original error message. On Wed, May 9, 2012 at 12:22 AM, Carl Steinbach c...@cloudera.com

Re: new feature in hive: links

2012-05-22 Thread Ashutosh Chauhan
To kickstart the review, I did a quick review of the doc. Few questions popped out to me, which I asked. Sambavi was kind enough to come back with replies for them. I am continuing to look into it. Will encourage other folks to look into it as well. Thanks, Ashutosh Begin Forward Message Hi

Re: non-string partition columns

2012-05-25 Thread Ashutosh Chauhan
Some discussion for this has happened on https://issues.apache.org/jira/browse/HIVE-2702 Is the underlying problem same as the one which I described on that jira ? Thanks, Ashutosh On Thu, May 24, 2012 at 10:59 PM, Namit Jain nj...@fb.com wrote: Should we disallow non-string partition columns

Re: non-string partition columns

2012-05-29 Thread Ashutosh Chauhan
? Thanks, -namit On 5/24/12 11:21 PM, Ashutosh Chauhan hashut...@apache.org wrote: Some discussion for this has happened on https://issues.apache.org/jira/browse/HIVE-2702 Is the underlying problem same as the one which I described on that jira ? Thanks, Ashutosh

Re: Behavior of Hive 2837: insert into external tables should not be allowed

2012-06-01 Thread Ashutosh Chauhan
Hi Mark, I understand your concern w.r.t backward compatibility. But as Ed pointed out there is a config variable and by default semantic is unchanged so you can continue to insert into your external table. I have a question though. Why are you creating all your tables as external tables ? Why

Re: test errors

2012-06-11 Thread Ashutosh Chauhan
Works for me. $ svn up svn At revision 1348932. $ svn st $ ant clean package test -Dtestcase=TestZooKeeperTokenStore [junit] Tests run: 4, Failures: 0, Errors: 0, Time elapsed: 1.783 sec Any logs to look at? Ashutosh On Mon, Jun 11, 2012 at 5:33 AM, Namit Jain nj...@fb.com wrote: I am

Re: Review Request: Allow to download resources from any external File Systems to local machine.

2012-07-01 Thread Ashutosh Chauhan
/hadoop/hive/ql/session/SessionState.java https://reviews.apache.org/r/5687/#comment18552 Instead of regex, it might be better to use URI to parse the string. String scheme = new Path(value).toURI().getScheme(); return (scheme != null) !scheme.equalsIgnoreCase(file); - Ashutosh Chauhan

Re: Review Request: Resource Leak: Fix the File handle leak in EximUtil.java

2012-07-10 Thread Ashutosh Chauhan
for path.startsWith('/') makes most sense here. As Kanna noted his changes make path to /D:/hive/etc from D:/hive/etc thus making URI constructor happy. - Ashutosh Chauhan On July 5, 2012, 7:50 p.m., Kanna Karanam wrote

Re: Review Request: Remove the Unix specific absolute path of “Cat” utility in several .q files to make them run on Windows with CygWin in path.

2012-08-05 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6313/#review9874 --- Ship it! Ship It! - Ashutosh Chauhan On Aug. 2, 2012, 4:51 a.m

Re: Review Request: This function overloads the current DateDiff(expr1, expr2) by adding another parameter to specify the units.

2012-08-06 Thread Ashutosh Chauhan
this function is not used anywhere. Please remove it. trunk/ql/src/java/org/apache/hadoop/hive/ql/udf/UDFDateDiff.java https://reviews.apache.org/r/6027/#comment21119 Looks like this function is not used anywhere. Please, remove it. - Ashutosh Chauhan On July 18, 2012, 12:56 a.m., Shefali Vohra

Re: [ANNOUNCE] New Hive Committer - Navis Ryu

2012-08-10 Thread Ashutosh Chauhan
Congrats, Navis! Well deserved. Welcome, aboard! Ashutosh On Fri, Aug 10, 2012 at 11:10 AM, Carl Steinbach c...@cloudera.com wrote: Congratulations Navis! This is very well deserved. Looking forward to many more patches from you. On Fri, Aug 10, 2012 at 8:10 AM, Bejoy KS bejoy...@yahoo.com

Re: Review Request: HIVE-3409. Increase test.timeout value

2012-08-27 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6780/#review10775 --- Ship it! Ship It! - Ashutosh Chauhan On Aug. 27, 2012, 10:09 a.m

Re: Review Request: HIVE-3409. Increase test.timeout value

2012-08-27 Thread Ashutosh Chauhan
On Aug. 27, 2012, 3:07 p.m., Ashutosh Chauhan wrote: Ship It! +1 Please commit. - Ashutosh --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/6780/#review10775

Re: [VOTE] Apache Hive 0.11.0 Release Candidate 2

2013-05-15 Thread Ashutosh Chauhan
+1 Built from sources, ran few unit tests and some simple queries against 1-node cluster. Thanks, Ashutosh On Wed, May 15, 2013 at 5:30 PM, Navis류승우 navis@nexr.com wrote: +1 - built from source, passed all tests (without assertion fail or conversion to backup task) - working good

Re: Review Request: HIVE-4489: beeline always return the same error message twice

2013-06-01 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10917/#review21291 --- Ship it! +1 - Ashutosh Chauhan On May 3, 2013, 11:01 p.m

Re: Review Request: Column stats : LOW_VALUE (or HIGH_VALUE) will always be 0.0000 , if all the column values larger than 0.0 (or if all column values smaller than 0.0)

2013-06-01 Thread Ashutosh Chauhan
/hadoop/hive/ql/udf/generic/GenericUDAFComputeStats.java https://reviews.apache.org/r/11172/#comment44173 Similarly can you add a comment why we need to set max to -ve infinity and not 0, since its counter intuitive? - Ashutosh Chauhan On May 15, 2013, 7:11 a.m., Zhuoluo Yang wrote

Re: Review Request: HIVE-4513 - disable hivehistory logs by default

2013-06-03 Thread Ashutosh Chauhan
://reviews.apache.org/r/11029/#comment44291 Missing apache header ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistoryViewer.java https://reviews.apache.org/r/11029/#comment44292 HiveHistoryViewer.class - Ashutosh Chauhan On May 13, 2013, 10:12 p.m., Thejas Nair wrote

Re: Review Request: Review Request for HIVE-4554 Failed to create a table from existing file if file path has spaces

2013-06-03 Thread Ashutosh Chauhan
:// path which has spaces. If its easy, it will be good to add it in test, else manual test is fine as well. - Ashutosh Chauhan On June 3, 2013, 10:18 p.m., Xuefu Zhang wrote: --- This is an automatically generated e-mail. To reply

Re: Review Request: Review Request for HIVE-4554 Failed to create a table from existing file if file path has spaces

2013-06-04 Thread Ashutosh Chauhan
On June 3, 2013, 11:15 p.m., Ashutosh Chauhan wrote: ql/src/java/org/apache/hadoop/hive/ql/parse/LoadSemanticAnalyzer.java, line 273 https://reviews.apache.org/r/11335/diff/3/?file=300140#file300140line273 Apart from this change, all other changes are contained within

Re: Review Request: Column stats : LOW_VALUE (or HIGH_VALUE) will always be 0.0000 , if all the column values larger than 0.0 (or if all column values smaller than 0.0)

2013-06-05 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11172/#review21480 --- Ship it! +1 - Ashutosh Chauhan On June 5, 2013, 2:06 p.m

Re: Review Request: HIVE-4712: Fix TestCliDriver.truncate_* on 0.23

2013-06-11 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11801/#review21708 --- Ship it! +1 - Ashutosh Chauhan On June 11, 2013, 7:18 a.m

Re: Branch for HIVE-4660

2013-06-13 Thread Ashutosh Chauhan
Makes sense. I will create the branch soon. Thanks, Ashutosh On Tue, Jun 11, 2013 at 7:44 PM, Gunther Hagleitner ghagleit...@hortonworks.com wrote: Hi, I am starting to work on integrating Tez into Hive (see HIVE-4660, design doc has already been uploaded - any feedback will be much

Re: Review Request: Initialize object inspectors with union of table properties and partition properties

2013-06-18 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11632/#review22081 --- Ship it! +1 - Ashutosh Chauhan On June 4, 2013, 6:01 p.m., Mark

Re: Branch for HIVE-4660

2013-06-19 Thread Ashutosh Chauhan
Created the branch from tip of trunk. Check it out https://svn.apache.org/repos/asf/hive/branches/tez/ Thanks, Ashutosh On Thu, Jun 13, 2013 at 5:43 PM, Ashutosh Chauhan hashut...@apache.orgwrote: Makes sense. I will create the branch soon. Thanks, Ashutosh On Tue, Jun 11, 2013 at 7:44

Re: Review Request 11925: Hive-3159 Update AvroSerde to determine schema of new tables

2013-06-29 Thread Ashutosh Chauhan
? serde/src/java/org/apache/hadoop/hive/serde2/avro/TypeInfoToSchema.java https://reviews.apache.org/r/11925/#comment46295 Lets take care of this TODO. Should be straight fwd. - Ashutosh Chauhan On June 18, 2013, 3:26 a.m., Mohammad Islam wrote

Re: Review Request 12050: HIVE-3756 (LOAD DATA does not honor permission inheritance)

2013-07-02 Thread Ashutosh Chauhan
) { fs.mkdirs(destfp, destfp.getParent().getPerms); } else { fs.mkdirs(destfp); } ql/src/java/org/apache/hadoop/hive/ql/metadata/Hive.java https://reviews.apache.org/r/12050/#comment46399 same as previous comment. Couple of comments on api usage. - Ashutosh Chauhan

Re: Hive Jenkins Builds

2013-07-03 Thread Ashutosh Chauhan
I think we should disable 0.9 and 0.10 builds. But we should create 0.11 build, since that is our last release. Ashutosh On Wed, Jul 3, 2013 at 11:39 AM, Brock Noland br...@cloudera.com wrote: Hive has four builds that currently run: Hive-trunk-h0.21 (trunk on Hadoop 1.X)

Re: Tez branch and tez based patches

2013-07-17 Thread Ashutosh Chauhan
On Wed, Jul 17, 2013 at 1:41 PM, Edward Capriolo edlinuxg...@gmail.comwrote: In my opinion we should limit the amount of tez related optimizations to and trunk Refactoring that cleans up code is good, but as you have pointed out there wont be a tez release until sometime this fall, and this

Re: Review Request 12767: [HIVE-4877] In ExecReducer, remove tag from the row which will be passed to the first Operator at the Reduce-side

2013-07-19 Thread Ashutosh Chauhan
://reviews.apache.org/r/12767/#comment47421 Should we also add following in comment? .. and directly call process on children in process() method. One more : ) - Ashutosh Chauhan On July 19, 2013, 5 p.m., Yin Huai wrote

Re: Review Request 12767: [HIVE-4877] In ExecReducer, remove tag from the row which will be passed to the first Operator at the Reduce-side

2013-07-19 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12767/#review23541 --- Ship it! Ship It! - Ashutosh Chauhan On July 19, 2013, 7:04 p.m

Re: Review Request 12767: [HIVE-4877] In ExecReducer, remove tag from the row which will be passed to the first Operator at the Reduce-side

2013-07-19 Thread Ashutosh Chauhan
as: // remove the tag from key coming out of reducer and store it in separate variable. - Ashutosh Chauhan On July 19, 2013, 5 p.m., Yin Huai wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r

Re: Review Request 12690: HIVE-4870: Explain Extended to show partition info for Fetch Task

2013-07-22 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12690/#review23654 --- Ship it! Ship It! - Ashutosh Chauhan On July 17, 2013, 5:14 p.m

Re: Review Request 12705: HIVE-4878: With Dynamic partitioning, some queries would scan default partition even if query is not using it.

2013-07-22 Thread Ashutosh Chauhan
and will be excluded from all further query processing. - Ashutosh Chauhan On July 17, 2013, 10:19 p.m., John Pullokkaran wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12705

Re: Review Request 12050: HIVE-3756 (LOAD DATA does not honor permission inheritance)

2013-07-25 Thread Ashutosh Chauhan
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12050/#review23858 --- Ship it! Ship It! - Ashutosh Chauhan On July 19, 2013, 6:55 p.m

Re: Adding to the hive contributor list

2013-08-14 Thread Ashutosh Chauhan
Done. Welcome Hari to the project. Thanks, Ashutosh On Wed, Aug 14, 2013 at 10:32 AM, Hari Subramaniyan hsubramani...@hortonworks.com wrote: Hi, I would like to get added to contributor list. Thanks Hari -- CONFIDENTIALITY NOTICE NOTICE: This message is intended for the use of the

Re: Adding WebHCat sub component to Hive project in ASF Jira

2013-08-16 Thread Ashutosh Chauhan
Done. Looking forward to contributions in that area! Thanks, Ashutosh On Fri, Aug 16, 2013 at 11:44 AM, Eugene Koifman ekoif...@hortonworks.comwrote: Hi, could somebody who has permissions to do so create WebHCat component under Hive? It will help track things. Thanks, Eugene --

Re: Last time request for cwiki update privileges

2013-08-20 Thread Ashutosh Chauhan
Hi Sanjay, Really sorry for that. I apologize for the delay. You are added now. Feel free to make changes to make Hive even better! Thanks, Ashutosh On Tue, Aug 20, 2013 at 2:39 PM, Sanjay Subramanian sanjay.subraman...@wizecommerce.com wrote: Hey guys I can only think of two reasons

Re: Last time request for cwiki update privileges

2013-08-21 Thread Ashutosh Chauhan
...@gmail.com Sanjay gets some love after all! :) On Tue, Aug 20, 2013 at 4:00 PM, Sanjay Subramanian sanjay.subraman...@wizecommerce.com wrote: Thanks Ashutosh From: Ashutosh Chauhan hashut...@apache.orgmailto:hashut...@apache.org Reply-To: u...@hive.apache.orgmailto:u...@hive.apache.org u

Re: Last time request for cwiki update privileges

2013-08-21 Thread Ashutosh Chauhan
Not able to find this id in cwiki. Did you create an account on cwiki.apache.org On Wed, Aug 21, 2013 at 2:59 PM, Mikhail Antonov olorinb...@gmail.comwrote: mantonov

Re: LIKE filter pushdown for tables and partitions

2013-08-26 Thread Ashutosh Chauhan
Couple of questions: 1. What about LIKE operator for Hive itself? Will that continue to work (presumably because there is an alternative path for that). 2. This will nonetheless break other direct consumers of metastore client api (like HCatalog). I see your point that we have a buggy

Re: Review Request 13862: [HIVE-5149] ReduceSinkDeDuplication can pick the wrong partitioning columns

2013-09-01 Thread Ashutosh Chauhan
/ReduceSinkDeDuplication.java https://reviews.apache.org/r/13862/#comment50365 else { throw new SemanticException(Not able to correctly identify partitioning columns. Hint: Try hive.optimize.reducededuplication=false; );} Thanks for adding comments! - Ashutosh Chauhan On Aug. 30, 2013, 3:29 p.m

Re: Review Request 13862: [HIVE-5149] ReduceSinkDeDuplication can pick the wrong partitioning columns

2013-09-01 Thread Ashutosh Chauhan
/ReduceSinkDeDuplication.java https://reviews.apache.org/r/13862/#comment50366 In here. if(result[0] = 0) throw new SemanticException(Sort columns and order don't match. Try hive.optimize.reducesinkdeduplication=false;); Another sanity check. - Ashutosh Chauhan On Aug. 30, 2013, 3:29 p.m., Yin

  1   2   3   4   5   6   7   8   9   10   >