---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5247/
---
(Updated 2012-05-25 22:01:12.690191)
Review request for hive and Carl Steinbach.
I've put together a proposal describing changes to Hive's metastore and
Thrift API to support statistics at the level of a column in a Hive
table/partition. Please bear in mind that this proposal is a work in
progress. The goal of the proposal is to obtain feedback from the
community. Hence it woul
metastore/ivy.xml 3011d2f
metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaTool.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java ccb0d7f
Diff: https://reviews.apache.org/r/6532/diff/
Testing
---
Thanks,
Shreepadma Venugopalan
e/src/java/org/apache/hadoop/hive/metastore/HiveMetaTool.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/exec/Utilities.java ccb0d7f
Diff: https://reviews.apache.org/r/6650/diff/
Testing
---
Manual testing for various metatool options.
Thanks,
Shreepadma Venugopalan
a/org/apache/hadoop/hive/metastore/tools/HiveMetaTool.java
PRE-CREATION
metastore/src/test/org/apache/hadoop/hive/metastore/TestHiveMetaTool.java
PRE-CREATION
Diff: https://reviews.apache.org/r/6650/diff/
Testing
---
Manual testing for various metatool options.
Thanks,
Shree
a
<https://reviews.apache.org/r/6650/#comment22588>
done
- Shreepadma Venugopalan
On Aug. 22, 2012, 2:48 a.m., Shreepadma Venugopalan wrote:
>
> ---
> This is an automatically generated e-mail. T
in this patch.
- Shreepadma
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/6650/#review10521
---
On Aug. 22, 2012, 2:48 a.m.
jectStore methods are exposed via Thrift, I
think of HiveMetaTool as yet another way to expose some of the ObjectStore
methods.
metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java
<https://reviews.apache.org/r/6650/#comment22749>
Please let me know what the alternative
/hive/metastore/tools/HiveMetaTool.java
PRE-CREATION
metastore/src/test/org/apache/hadoop/hive/metastore/TestHiveMetaTool.java
PRE-CREATION
Diff: https://reviews.apache.org/r/6650/diff/
Testing
---
Manual testing for various metatool options.
Thanks,
Shreepadma Venugopalan
test - TestHiveMetaTool - has been added to test the various
metatool options.
Thanks,
Shreepadma Venugopalan
to 1.7.1
This addresses bug HIVE-3365.
https://issues.apache.org/jira/browse/HIVE-3365
Diffs
-
ivy/libraries.properties f0b1918
Diff: https://reviews.apache.org/r/6764/diff/
Testing
---
Avro*.q tests pass. Running the whole suite of tests.
Thanks,
Shreepadma Venugopalan
9eebe
metastore/src/test/org/apache/hadoop/hive/metastore/TestHiveMetaTool.java
PRE-CREATION
Diff: https://reviews.apache.org/r/6650/diff/
Testing
---
A new JUnit test - TestHiveMetaTool - has been added to test the various
metatool options.
Thanks,
Shreepadma Venugopalan
test - TestHiveMetaTool - has been added to test the various
metatool options.
Thanks,
Shreepadma Venugopalan
HiveHistory.printRowCount().
This addresses bug HIVE-3265.
https://issues.apache.org/jira/browse/HIVE-3265
Diffs
-
ql/src/java/org/apache/hadoop/hive/ql/history/HiveHistory.java e2f3ea5
Diff: https://reviews.apache.org/r/6779/diff/
Testing
---
Thanks,
Shreepadma Venugopalan
.
Thanks,
Shreepadma Venugopalan
<https://reviews.apache.org/r/6650/#comment23265>
I removed the exception and replaced it with a system.err.. I also print
the help option.
metastore/src/java/org/apache/hadoop/hive/metastore/tools/HiveMetaTool.java
<https://review
/hive/metastore/tools/HiveMetaTool.java
<https://reviews.apache.org/r/6650/#comment23386>
I'll make this change.
metastore/src/java/org/apache/hadoop/hive/metastore/tools/HiveMetaTool.java
<https://reviews.apache.org/r/6650/#comment23387>
This was originally in the fin
mmand.
Thanks,
Shreepadma Venugopalan
store/TestHiveMetaTool.java
PRE-CREATION
Diff: https://reviews.apache.org/r/6650/diff/
Testing
---
A new JUnit test - TestHiveMetaTool - has been added to test the various
metatool options.
Thanks,
Shreepadma Venugopalan
e in TestHiveMetaTool.java has been
modified to use AvroSerDe instead of LazySimpleSerDe.
Thanks,
Shreepadma Venugopalan
ol.java
5790ae9
Diff: https://reviews.apache.org/r/6962/diff/
Testing
---
HiveMetaTool has been tested to verify that it handles both avro.schema.url and
schema.url correctly. Existing test case in TestHiveMetaTool.java has been
modified to use AvroSerDe instead of LazySimpleSerDe.
Thanks,
Shreepadma Venugopalan
a.url correctly. Existing test case in TestHiveMetaTool.java has been
modified to use AvroSerDe instead of LazySimpleSerDe.
Thanks,
Shreepadma Venugopalan
it:
https://reviews.apache.org/r/6962/#review11291
-------
On Sept. 13, 2012, 4:50 p.m., Shreepadma Venugopalan wrote:
>
> ---
> This is an automatically generated e
ed to use AvroSerDe instead of LazySimpleSerDe.
Thanks,
Shreepadma Venugopalan
store schema and Thrift API
changes,
* Tests to exercise compute_stats UDAF for all primitive types,
* End to end test both at table and partition level for computing stats on
multiple columns. Note that these tests use the extended syntax of the analyze
command.
Thanks,
Shreepadma Venugopalan
ts hard to follow a convention
when its not very clear what it is.
- Shreepadma
-----------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/6878/#review11502
---
On Oct. 3, 2012, 3:10 a.m., Shreepadma Venugopalan
/GenericUDAFComputeStats.java
<https://reviews.apache.org/r/6878/#comment25822>
I'll replace LHS with generic java types.
- Shreepadma Venugopalan
On Oct. 3, 2012, 3:10 a.m., Shreepadma Venugopalan wrote:
>
> ---
> This is an autom
ail. To reply, visit:
https://reviews.apache.org/r/6878/#review12131
---
On Oct. 3, 2012, 3:10 a.m., Shreepadma Venugopalan wrote:
>
> ---
> This is an automatically gener
tps://reviews.apache.org/r/6878/#comment25825>
Will remove this change to MapRedTask.java. Sorry abt this.
- Shreepadma Venugopalan
On Oct. 3, 2012, 3:10 a.m., Shreepadma Venugopalan wrote:
>
> ---
> This is an automatica
tax of the analyze
command.
Thanks,
Shreepadma Venugopalan
he analyze
command.
Thanks,
Shreepadma Venugopalan
.
Thanks,
Shreepadma Venugopalan
level for computing stats on
multiple columns. Note that these tests use the extended syntax of the analyze
command.
Thanks,
Shreepadma Venugopalan
/org/apache/hadoop/hive/ql/parse/StatsSemanticAnalyzer.java
<https://reviews.apache.org/r/6878/#comment25930>
Done
ql/src/java/org/apache/hadoop/hive/ql/parse/StatsSemanticAnalyzer.java
<https://reviews.apache.org/r/6878/#comment25931>
Done
- Shr
r all primitive types,
* End to end test both at table and partition level for computing stats on
multiple columns. Note that these tests use the extended syntax of the analyze
command.
Thanks,
Shreepadma Venugopalan
r all primitive types,
* End to end test both at table and partition level for computing stats on
multiple columns. Note that these tests use the extended syntax of the analyze
command.
Thanks,
Shreepadma Venugopalan
+1. Sounds like a good idea.
Shreepadma
On Wed, Nov 7, 2012 at 4:50 PM, Ashutosh Chauhan wrote:
> Hi,
>
> Its been a while since we released 0.10 more than six months ago. All this
> while, lot of action has happened with various cool features landing in
> trunk. Additionally, I am looking forwa
/8119/diff/
Testing
---
Tested the schema upgrade scripts.
Thanks,
Shreepadma Venugopalan
metastore/src/model/package.jdo 5f91f97
Diff: https://reviews.apache.org/r/8119/diff/
Testing
---
Tested the schema upgrade scripts.
Thanks,
Shreepadma Venugopalan
63bf69b
metastore/src/model/package.jdo 5f91f97
Diff: https://reviews.apache.org/r/8119/diff/
Testing
---
Tested the schema upgrade scripts.
Thanks,
Shreepadma Venugopalan
/clientpositive/serde_regex.q.out a8ce604
serde/src/java/org/apache/hadoop/hive/serde2/RegexSerDe.java e728244
Diff: https://reviews.apache.org/r/8931/diff/
Testing
---
New test cases have been added and they pass.
Thanks,
Shreepadma Venugopalan
/
Testing
---
New test cases have been added and they pass.
Thanks,
Shreepadma Venugopalan
--------
On Jan. 12, 2013, 12:28 a.m., Shreepadma Venugopalan wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/8931/
> -
/8931/diff/
Testing
---
New test cases have been added and they pass.
Thanks,
Shreepadma Venugopalan
---
Thanks,
Shreepadma Venugopalan
All,
Can someone point me to the apache docs for hive 0.10.0?
Thanks.
Shreepadma
Measure 1: +1
Measure 2: +1
Thanks.
Shreepadma
On Tue, Jan 29, 2013 at 1:50 PM, Arvind Prabhakar wrote:
> Measure 1: +1
> Measure 2: +1
>
> Regards,
> Arvind Prabhakar
>
> On Mon, Jan 28, 2013 at 11:47 AM, Carl Steinbach wrote:
>
> > I am calling a vote on the following two measures.
> >
>
+1. This looks very useful.
Shreepadma
On Fri, Feb 1, 2013 at 11:36 AM, Ashish Thusoo wrote:
> +1.
>
> This is certainly very useful work.
>
> Ashish
>
>
> On Fri, Feb 1, 2013 at 11:31 AM, Carl Steinbach >wrote:
>
> > +1
> >
> > On Fri, Feb 1, 2013 at 10:11 AM, Ashutosh Chauhan > >wrote:
> >
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5542/#review16304
---
Ship it!
Ship It!
- Shreepadma Venugopalan
On June 25, 2012, 5
Congrats Harish!
On Fri, Nov 15, 2013 at 1:32 PM, Thejas Nair wrote:
> Congrats Harish!
>
>
> On Fri, Nov 15, 2013 at 1:08 AM, Gunther Hagleitner
> wrote:
> > Congratulations Harish! Very cool.
> >
> >
> > On Thu, Nov 14, 2013 at 10:01 PM, Prasad Mujumdar >wrote:
> >
> >> Congratulations
Congrats guys!
On Thu, Nov 21, 2013 at 3:37 PM, Vinod Kumar Vavilapalli wrote:
> Congratulations to both! Great job and keep up the good work!
>
> Thanks,
> +Vinod
>
> On Nov 21, 2013, at 3:29 PM, Carl Steinbach wrote:
>
> > The Apache Hive PMC has voted to make Jitendra Nath Pandey and Eric
>
Congrats Kevin!
On Mon, Mar 4, 2013 at 12:43 PM, Kevin Wilfong wrote:
> Thanks eveyone!
>
> From: Ricky Saltzer [ri...@cloudera.com]
> Sent: Monday, March 04, 2013 12:13 PM
> To: dev@hive.apache.org
> Cc: u...@hive.apache.org; Kevin Wilfong
> Subject: Re: [ANNOU
/columnstats_tbllvl.q.out 3390ca0
ql/src/test/results/clientpositive/compute_stats_empty_table.q.out
PRE-CREATION
Diff: https://reviews.apache.org/r/9929/diff/
Testing
---
Adds new q file tests which test column stats against an empty table.
Thanks,
Shreepadma Venugopalan
https://issues.apache.org/jira/browse/HIVE-4119
Thanks.
Shreepadma
doop/hive/ql/udf/generic/GenericUDAFComputeStats.java
<https://reviews.apache.org/r/9929/#comment38517>
done
- Shreepadma Venugopalan
On March 14, 2013, 8:53 p.m., Shreepadma Venugopalan wrote:
>
> ---
> This is an automati
file tests which test column stats against an empty table.
Thanks,
Shreepadma Venugopalan
---
The estimates are within the expected error after this fix. Tested on TPCH of
varying sizes.
Thanks,
Shreepadma Venugopalan
/NumDistinctValueEstimator.java
69e6f46
Diff: https://reviews.apache.org/r/10841/diff/
Testing
---
The estimates are within the expected error after this fix. Tested on TPCH of
varying sizes.
Thanks,
Shreepadma Venugopalan
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11084/#review21332
---
Ship it!
LGTM. +1 (non-binding).
- Shreepadma Venugopalan
On
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11172/#review21391
---
Ship it!
Ship It!
- Shreepadma Venugopalan
On June 3, 2013, 4
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11172/#review21438
---
Ship it!
Ship It!
- Shreepadma Venugopalan
On June 4, 2013, 1
> On June 4, 2013, 11:09 p.m., Shreepadma Venugopalan wrote:
> > Ship It!
I think its OK to have Long.MAX_VALUE/Long.MIN_VALUE as the min/max value if
there are no rows in the table.
- Shreepadma
---
This is an auto
/HiveSemanticAnalyzerHookContext.java
ae371f3
ql/src/java/org/apache/hadoop/hive/ql/parse/HiveSemanticAnalyzerHookContextImpl.java
9c3377e
Diff: https://reviews.apache.org/r/11661/diff/
Testing
---
Thanks,
Shreepadma Venugopalan
adoop/hive/thrift/HadoopThriftAuthBridge20S.java
777226f
shims/src/common/java/org/apache/hadoop/hive/shims/HadoopShims.java b0f5077
Diff: https://reviews.apache.org/r/11705/diff/
Testing
---
Thanks,
Shreepadma Venugopalan
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11706/#review21586
---
Ship it!
Ship It!
- Shreepadma Venugopalan
On June 7, 2013, 2
> On June 7, 2013, 5:52 p.m., Shreepadma Venugopalan wrote:
> > Ship It!
LGTM! It will break applications that assert on TABLE_SCHEMA, but then those
asserts are incorrect in the first place.
- Shreepadma
---
This is an auto
adoop/hive/shims/Hadoop23Shims.java 1975385
shims/src/common-secure/java/org/apache/hadoop/hive/thrift/HadoopThriftAuthBridge20S.java
777226f
shims/src/common/java/org/apache/hadoop/hive/shims/HadoopShims.java b0f5077
Diff: https://reviews.apache.org/r/11705/diff/
Testing
---
Thanks,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11326/#review21599
---
LGTM!
- Shreepadma Venugopalan
On May 22, 2013, 6:41 a.m
Many congrats, Brock!
On Tue, Jul 16, 2013 at 7:33 AM, Ricky Saltzer wrote:
> Congrats!!
>
>
> On Tue, Jul 16, 2013 at 10:30 AM, Alexander Alten-Lorenz <
> wget.n...@gmail.com> wrote:
>
>> Congratulations, Brock!
>>
>> On Jul 15, 2013, at 10:29 PM, Carl Steinbach wrote:
>>
>> > The Apache Hive
Congratulations, Gunther!
On Sun, Jul 21, 2013 at 10:29 AM, Thejas Nair wrote:
> Congrats Gunther !
> Great to see more bandwidth to get the patch available counts down !
> On Jul 21, 2013 9:56 AM, "Clark Yang (杨卓荦)" wrote:
>
> > Congratulations Gunther!
> >
> > 2013/7/22 Brock Noland
> >
> >
Congrats Tejas!
On Tue, Aug 20, 2013 at 9:32 AM, Eugene Koifman wrote:
> Congrats Thejas!
>
>
> On Tue, Aug 20, 2013 at 3:31 AM, Carl Steinbach wrote:
>
>> The Apache Hive PMC has voted to make Thejas Nair a committer on the
>> Apache
>> Hive project.
>>
>> Please join me in congratulating Thej
/queries/clientnegative/compute_stats_long.q PRE-CREATION
ql/src/test/results/clientnegative/compute_stats_long.q.out PRE-CREATION
Diff: https://reviews.apache.org/r/14250/diff/
Testing
---
Adds a new negative test case.
Thanks,
Shreepadma Venugopalan
new unit test to test log retrieval.
Thanks,
Shreepadma Venugopalan
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14250/#review26304
-------
On Sept. 20, 2013, 8:02 p.m., Shreepadma Venugopalan wrote:
>
> -
rrent framework, this is the best we can do.
- Shreepadma
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/14250/#review26313
-------
[
https://issues.apache.org/jira/browse/HIVE-1719?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shreepadma Venugopalan updated HIVE-1719:
-
Attachment: HIVE-1719.3.patch.txt
This patch makes the following changes
[
https://issues.apache.org/jira/browse/HIVE-1719?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shreepadma Venugopalan updated HIVE-1719:
-
Attachment: HIVE-1719.patch
> Move RegexSerDe out of hive-contrib and over
[
https://issues.apache.org/jira/browse/HIVE-1719?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shreepadma Venugopalan updated HIVE-1719:
-
Status: Patch Available (was: Open)
> Move RegexSerDe out of hive-cont
Shreepadma Venugopalan created HIVE-3023:
Summary: Move runtime exception for incorrect number of matching
groups in Regex SerDe to compile time
Key: HIVE-3023
URL: https://issues.apache.org/jira/browse
[
https://issues.apache.org/jira/browse/HIVE-1719?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shreepadma Venugopalan updated HIVE-1719:
-
Attachment: HIVE-1719.D3249.1.patch
> Move RegexSerDe out of hive-contrib
[
https://issues.apache.org/jira/browse/HIVE-3004?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shreepadma Venugopalan updated HIVE-3004:
-
Attachment: #HIVE-3004.patch#
> RegexSerDe should support other column ty
[
https://issues.apache.org/jira/browse/HIVE-3004?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shreepadma Venugopalan updated HIVE-3004:
-
Status: Patch Available (was: Open)
> RegexSerDe should support other col
[
https://issues.apache.org/jira/browse/HIVE-3004?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shreepadma Venugopalan updated HIVE-3004:
-
Attachment: HIVE-3004-1.patch
> RegexSerDe should support other column ty
[
https://issues.apache.org/jira/browse/HIVE-3004?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shreepadma Venugopalan updated HIVE-3004:
-
Attachment: (was: #HIVE-3004.patch#)
> RegexSerDe should support ot
[
https://issues.apache.org/jira/browse/HIVE-1719?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shreepadma Venugopalan updated HIVE-1719:
-
Attachment: HIVE-1719.3.patch
> Move RegexSerDe out of hive-contrib and o
Shreepadma Venugopalan created HIVE-3038:
Summary: Query with order by clause fails during semantic analysis
unless the column in the order by is referenced in the select list
Key: HIVE-3038
URL: https
[
https://issues.apache.org/jira/browse/HIVE-3038?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shreepadma Venugopalan updated HIVE-3038:
-
Summary: Query with order by clause fails during semantic analysis when
ordering
[
https://issues.apache.org/jira/browse/HIVE-3004?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shreepadma Venugopalan updated HIVE-3004:
-
Attachment: HIVE-3004.2.patch
> RegexSerDe should support other column ty
[
https://issues.apache.org/jira/browse/HIVE-1719?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shreepadma Venugopalan updated HIVE-1719:
-
Attachment: (was: HIVE-1719.patch)
> Move RegexSerDe out of hive-cont
[
https://issues.apache.org/jira/browse/HIVE-1719?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shreepadma Venugopalan updated HIVE-1719:
-
Attachment: (was: HIVE-1719.3.patch.txt)
> Move RegexSerDe out of h
Shreepadma Venugopalan created HIVE-3241:
Summary: GenericUDAF when packaged as a JAR raises
IllegalAccessException unless the JAR is placed in lib dir
Key: HIVE-3241
URL: https://issues.apache.org/jira
[
https://issues.apache.org/jira/browse/HIVE-3241?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shreepadma Venugopalan updated HIVE-3241:
-
Description:
GenericUDAFEvaluator has a member variable "Mode" which is
[
https://issues.apache.org/jira/browse/HIVE-3241?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shreepadma Venugopalan updated HIVE-3241:
-
Attachment: compute-stats.jar
> GenericUDAF when packaged as a JAR rai
[
https://issues.apache.org/jira/browse/HIVE-1653?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13422486#comment-13422486
]
Shreepadma Venugopalan commented on HIVE-1653:
--
@Namit: Added comment
[
https://issues.apache.org/jira/browse/HIVE-3056?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shreepadma Venugopalan updated HIVE-3056:
-
Assignee: Shreepadma Venugopalan (was: Carl Steinbach)
Status: Patch
[
https://issues.apache.org/jira/browse/HIVE-3056?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shreepadma Venugopalan updated HIVE-3056:
-
Attachment: HIVE-3056.patch
> Ability to bulk update location field in
[
https://issues.apache.org/jira/browse/HIVE-3056?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shreepadma Venugopalan updated HIVE-3056:
-
Description: (was:
)
> Ability to bulk update location field in Db/Ta
[
https://issues.apache.org/jira/browse/HIVE-3056?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shreepadma Venugopalan updated HIVE-3056:
-
Attachment: HIVE-3056.2.patch.txt
> Ability to bulk update location field
[
https://issues.apache.org/jira/browse/HIVE-3056?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13439265#comment-13439265
]
Shreepadma Venugopalan commented on HIVE-3056:
--
@Namit: I've also p
[
https://issues.apache.org/jira/browse/HIVE-3056?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13439354#comment-13439354
]
Shreepadma Venugopalan commented on HIVE-3056:
--
Link to review board re
1 - 100 of 428 matches
Mail list logo