> On 2011-04-21 22:51:56, Carl Steinbach wrote: > > trunk/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreEventListener.java, > > line 41 > > <https://reviews.apache.org/r/618/diff/1/?file=15927#file15927line41> > > > > Instead of passing in raw Table/Partition/Database objects please wrap > > these objects in containers, e.g. CreateTableEvent, DropTableEvent, etc. > > > > Q: Whats the advantage of wrapper container objects? > > > > 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: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/618/ > ----------------------------------------------------------- > > (Updated 2011-04-18 18:44:05) > > > Review request for hive and Carl Steinbach. > > > Summary > ------- > > Addresses Carl's comment for previous patch. > Also added a new method finalizePartition in metastore through which > metastore client can indicate to metastore that partition can be finalized. > > > This addresses bug HIVE-2038. > https://issues.apache.org/jira/browse/HIVE-2038 > > > Diffs > ----- > > trunk/common/src/java/org/apache/hadoop/hive/conf/HiveConf.java 1094688 > trunk/conf/hive-default.xml 1094688 > trunk/metastore/if/hive_metastore.thrift 1094688 > trunk/metastore/src/gen/thrift/gen-cpp/ThriftHiveMetastore.h 1094688 > trunk/metastore/src/gen/thrift/gen-cpp/ThriftHiveMetastore.cpp 1094688 > > trunk/metastore/src/gen/thrift/gen-cpp/ThriftHiveMetastore_server.skeleton.cpp > 1094688 > > trunk/metastore/src/gen/thrift/gen-javabean/org/apache/hadoop/hive/metastore/api/ThriftHiveMetastore.java > 1094688 > > trunk/metastore/src/gen/thrift/gen-php/hive_metastore/ThriftHiveMetastore.php > 1094688 > > trunk/metastore/src/gen/thrift/gen-py/hive_metastore/ThriftHiveMetastore-remote > 1094688 > trunk/metastore/src/gen/thrift/gen-py/hive_metastore/ThriftHiveMetastore.py > 1094688 > trunk/metastore/src/gen/thrift/gen-rb/thrift_hive_metastore.rb 1094688 > > trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStore.java > 1094688 > > trunk/metastore/src/java/org/apache/hadoop/hive/metastore/HiveMetaStoreClient.java > 1094688 > > trunk/metastore/src/java/org/apache/hadoop/hive/metastore/IMetaStoreClient.java > 1094688 > > trunk/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreEventListener.java > PRE-CREATION > > trunk/metastore/src/java/org/apache/hadoop/hive/metastore/MetaStoreUtils.java > 1094688 > trunk/metastore/src/java/org/apache/hadoop/hive/metastore/NoOpListener.java > PRE-CREATION > trunk/metastore/src/java/org/apache/hadoop/hive/metastore/ObjectStore.java > 1094688 > > trunk/metastore/src/test/org/apache/hadoop/hive/metastore/DummyListener.java > PRE-CREATION > > trunk/metastore/src/test/org/apache/hadoop/hive/metastore/TestMetaStoreEventListener.java > PRE-CREATION > > Diff: https://reviews.apache.org/r/618/diff > > > Testing > ------- > > > Thanks, > > Ashutosh > >