> On Jan. 9, 2019, 11:56 p.m., Yongzhi Chen wrote: > > standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/PersistenceManagerProvider.java > > Line 228 (original), 227 (patched) > > <https://reviews.apache.org/r/69672/diff/1/?file=2117725#file2117725line228> > > > > This is a little bit different from the old impl funtionally : The old > > impl will return null if there is no custom properties, where new impl will > > still return the provider. The old impl has a kind of sanity check. But if > > the custom properties are not required here, it should be fine.
The behavior is not different. In the old impl, if you don't set a custom property `dsp` will be `null`, but `JDOHelper.getPersistenceManagerFactory(prop);` will create a connection pool manager anyways based on the value of `datanucleus.connectionPoolingType`. So that is no different now. The question is either create the default provider or our custom provider. I don't think there is any difference between the default provider and the custom provider we have in Hive. - Karthik ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69672/#review211805 ----------------------------------------------------------- On Jan. 5, 2019, 12:41 a.m., Karthik Manamcheri wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/69672/ > ----------------------------------------------------------- > > (Updated Jan. 5, 2019, 12:41 a.m.) > > > Review request for hive, Adam Holley, Morio Ramdenbourg, Naveen Gangam, and > Vihang Karajgaonkar. > > > Repository: hive-git > > > Description > ------- > > HIVE-21045: Add total API timing stats and connection pool stats to metrics > > > Diffs > ----- > > > standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/PersistenceManagerProvider.java > dfd7abff85 > > standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/datasource/BoneCPDataSourceProvider.java > 7e33c519a8 > > standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/datasource/DataSourceProvider.java > 6dc63fb3bc > > standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/datasource/DataSourceProviderFactory.java > 5a92e104be > > standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/datasource/DbCPDataSourceProvider.java > 7fe487b184 > > standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/datasource/HikariCPDataSourceProvider.java > 8f6ae57e36 > > standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/metrics/MetricsConstants.java > 3b188f83af > > standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/metrics/PerfLogger.java > a2def26fc5 > > standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/txn/TxnHandler.java > 2a6290315a > > standalone-metastore/metastore-server/src/test/java/org/apache/hadoop/hive/metastore/datasource/TestDataSourceProviderFactory.java > 6ae7f50471 > > > Diff: https://reviews.apache.org/r/69672/diff/1/ > > > Testing > ------- > > Manual testing to verify that the new metrics show up for hikaricp, bonecp, > and also the total stats. Here are samples of > 1. [HikariCP json metrics > sample](https://gist.github.com/kmanamcheri/48ff2a680e85c7e925a6f95a9384dcef) > 2. [BoneCP json metrics > sample](https://gist.github.com/kmanamcheri/b005f68263a1a1be06b25156a159d975) > > In both the reports note that there are pool gauges (for tracking the > connection pool info) and also a timer for total api calls. > > > Thanks, > > Karthik Manamcheri > >
