[
https://issues.apache.org/jira/browse/HIVE-2038?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13023036#comment-13023036
]
jirapos...@reviews.apache.org commented on HIVE-2038:
-----------------------------------------------------
bq. On 2011-04-21 22:51:56, Carl Steinbach wrote:
bq. >
trunk/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreEventListener.java,
line 41
bq. > <https://reviews.apache.org/r/618/diff/1/?file=15927#file15927line41>
bq. >
bq. > Instead of passing in raw Table/Partition/Database objects please
wrap these objects in containers, e.g. CreateTableEvent, DropTableEvent, etc.
bq. >
bq. > Q: Whats the advantage of wrapper container objects?
bq. >
bq. > The advantage of container objects is that it allows us to evolve
the API without breaking compatibility with older clients. With the current
interface if I want to add a parameter to one of these methods I will break
compatibility, but if we use container objects I can add a new field to the
container without affecting older clients.
It seems like this same concern also dictates that MetaStoreListener should be
an abstract class instead of an interface. Can you please make this change?
- Carl
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/618/#review522
-----------------------------------------------------------
On 2011-04-18 18:44:05, Ashutosh Chauhan wrote:
bq.
bq. -----------------------------------------------------------
bq. This is an automatically generated e-mail. To reply, visit:
bq. https://reviews.apache.org/r/618/
bq. -----------------------------------------------------------
bq.
bq. (Updated 2011-04-18 18:44:05)
bq.
bq.
bq. Review request for hive and Carl Steinbach.
bq.
bq.
bq. Summary
bq. -------
bq.
bq. Addresses Carl's comment for previous patch.
bq. Also added a new method finalizePartition in metastore through which
metastore client can indicate to metastore that partition can be finalized.
bq.
bq.
bq. This addresses bug HIVE-2038.
bq. https://issues.apache.org/jira/browse/HIVE-2038
bq.
bq.
bq. Diffs
bq. -----
bq.
bq. trunk/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 1094688
bq. trunk/conf/hive-default.xml 1094688
bq. trunk/metastore/if/hive_metastore.thrift 1094688
bq. trunk/metastore/src/gen/thrift/gen-cpp/ThriftHiveMetastore.h 1094688
bq. trunk/metastore/src/gen/thrift/gen-cpp/ThriftHiveMetastore.cpp 1094688
bq.
trunk/metastore/src/gen/thrift/gen-cpp/ThriftHiveMetastore_server.skeleton.cpp
1094688
bq.
trunk/metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/ThriftHiveMetastore.java
1094688
bq.
trunk/metastore/src/gen/thrift/gen-php/hive_metastore/ThriftHiveMetastore.php
1094688
bq.
trunk/metastore/src/gen/thrift/gen-py/hive_metastore/ThriftHiveMetastore-remote
1094688
bq.
trunk/metastore/src/gen/thrift/gen-py/hive_metastore/ThriftHiveMetastore.py
1094688
bq. trunk/metastore/src/gen/thrift/gen-rb/thrift_hive_metastore.rb 1094688
bq.
trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java
1094688
bq.
trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java
1094688
bq.
trunk/metastore/src/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java
1094688
bq.
trunk/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreEventListener.java
PRE-CREATION
bq.
trunk/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java
1094688
bq.
trunk/metastore/src/java/org/apache/hadoop/hive/metastore/NoOpListener.java
PRE-CREATION
bq.
trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java
1094688
bq.
trunk/metastore/src/test/org/apache/hadoop/hive/metastore/DummyListener.java
PRE-CREATION
bq.
trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestMetaStoreEventListener.java
PRE-CREATION
bq.
bq. Diff: https://reviews.apache.org/r/618/diff
bq.
bq.
bq. Testing
bq. -------
bq.
bq.
bq. Thanks,
bq.
bq. Ashutosh
bq.
bq.
> Metastore listener
> ------------------
>
> Key: HIVE-2038
> URL: https://issues.apache.org/jira/browse/HIVE-2038
> Project: Hive
> Issue Type: Improvement
> Components: Metastore
> Reporter: Ashutosh Chauhan
> Assignee: Ashutosh Chauhan
> Fix For: 0.8.0
>
> Attachments: hive-2038.patch, metastore_listener.patch,
> metastore_listener.patch, metastore_listener.patch
>
>
> Provide to way to observe changes happening on Metastore
--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira
