> On Sept. 8, 2017, 6:24 p.m., Prasanth_J wrote: > > llap-server/src/java/org/apache/hadoop/hive/llap/io/metadata/ParquetMetadataCacheImpl.java > > Lines 52 (patched) > > <https://reviews.apache.org/r/60289/diff/4/?file=1809787#file1809787line54> > > > > This file looks renamed from Parquet to generic MetadataCache but > > contains OrcSpecific objects. If it is generic remove orc related stuff or > > rename the class if it is orc specific?
It's a generic class for buffers, and contains some format-specific sub-parts > On Sept. 8, 2017, 6:24 p.m., Prasanth_J wrote: > > llap-server/src/java/org/apache/hadoop/hive/llap/io/metadata/ParquetMetadataCacheImpl.java > > Lines 106 (patched) > > <https://reviews.apache.org/r/60289/diff/4/?file=1809787#file1809787line108> > > > > why lock and unlock notification back to back? To "use" the item as far as eviction policy is concerned. > On Sept. 8, 2017, 6:24 p.m., Prasanth_J wrote: > > llap-server/src/java/org/apache/hadoop/hive/llap/io/metadata/ParquetMetadataCacheImpl.java > > Lines 134 (patched) > > <https://reviews.apache.org/r/60289/diff/4/?file=1809787#file1809787line136> > > > > can you create follow up? will be useful for debugging. or this could > > be jmx info. something that can be looked easily instead of logs. HIVE-17524 > On Sept. 8, 2017, 6:24 p.m., Prasanth_J wrote: > > llap-server/src/java/org/apache/hadoop/hive/llap/io/metadata/ParquetMetadataCacheImpl.java > > Lines 235 (patched) > > <https://reviews.apache.org/r/60289/diff/4/?file=1809787#file1809787line237> > > > > is readFully fixed in hadoop 2.8? if so, now that hive moved to 2.8.0 > > can that be used here and other places? We may be running against 2.7; also it anyway does the same thing and requires a cast, so there's no point > On Sept. 8, 2017, 6:24 p.m., Prasanth_J wrote: > > ql/src/java/org/apache/hadoop/hive/ql/io/orc/encoded/EncodedReaderImpl.java > > Lines 148 (patched) > > <https://reviews.apache.org/r/60289/diff/4/?file=1809792#file1809792line148> > > > > May be we should start using TypeDescription everywhere. OrcProto.Type > > can be huge object when compared to TypeDescription. added HIVE-17525 - Sergey ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/60289/#review184992 ----------------------------------------------------------- On Sept. 1, 2017, 12:41 a.m., Sergey Shelukhin wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/60289/ > ----------------------------------------------------------- > > (Updated Sept. 1, 2017, 12:41 a.m.) > > > Review request for hive, Gopal V and Prasanth_J. > > > Repository: hive-git > > > Description > ------- > > see jira > > > Diffs > ----- > > common/src/java/org/apache/hadoop/hive/conf/HiveConf.java e4b09a2cdd > > llap-server/src/java/org/apache/hadoop/hive/llap/cache/EvictionDispatcher.java > c5248ceb5f > > llap-server/src/java/org/apache/hadoop/hive/llap/io/api/impl/LlapIoImpl.java > f42622b892 > > llap-server/src/java/org/apache/hadoop/hive/llap/io/decode/OrcColumnVectorProducer.java > 6edd84b8b0 > > llap-server/src/java/org/apache/hadoop/hive/llap/io/encoded/OrcEncodedDataReader.java > b5db3029d1 > > llap-server/src/java/org/apache/hadoop/hive/llap/io/metadata/OrcFileEstimateErrors.java > dc053ee7cf > > llap-server/src/java/org/apache/hadoop/hive/llap/io/metadata/OrcFileMetadata.java > b9d7a77d5b > > llap-server/src/java/org/apache/hadoop/hive/llap/io/metadata/OrcMetadataCache.java > 601b622b49 > > llap-server/src/java/org/apache/hadoop/hive/llap/io/metadata/OrcStripeMetadata.java > 4565d11988 > > llap-server/src/java/org/apache/hadoop/hive/llap/io/metadata/ParquetMetadataCacheImpl.java > b61a8ca022 > > llap-server/src/test/org/apache/hadoop/hive/llap/cache/TestIncrementalObjectSizeEstimator.java > 13c7767a3b > > llap-server/src/test/org/apache/hadoop/hive/llap/cache/TestOrcMetadataCache.java > 03a955c6f7 > ql/src/java/org/apache/hadoop/hive/ql/io/orc/OrcInputFormat.java 69a9f9f35e > ql/src/java/org/apache/hadoop/hive/ql/io/orc/encoded/EncodedReader.java > 7540e72b53 > ql/src/java/org/apache/hadoop/hive/ql/io/orc/encoded/EncodedReaderImpl.java > 690cce798e > ql/src/java/org/apache/hadoop/hive/ql/io/orc/encoded/Reader.java cdd58df370 > ql/src/java/org/apache/hadoop/hive/ql/io/orc/encoded/ReaderImpl.java > d47ba6b31a > ql/src/test/results/clientpositive/llap/orc_llap_counters.q.out 8af84dce19 > ql/src/test/results/clientpositive/llap/orc_llap_counters1.q.out 4536cbbfb9 > ql/src/test/results/clientpositive/llap/orc_ppd_basic.q.out 77b7f5a2f7 > ql/src/test/results/clientpositive/llap/orc_ppd_schema_evol_3a.q.out > b799527e30 > > storage-api/src/java/org/apache/hadoop/hive/common/io/FileMetadataCache.java > 403c3ada61 > > > Diff: https://reviews.apache.org/r/60289/diff/4/ > > > Testing > ------- > > > Thanks, > > Sergey Shelukhin > >