Elasticsearch: बकेट और मेट्रिक एग्रीगेशन के लिए टेस्ट कवरेज में सुधार करें

आज बकेट और मेट्रिक एग्रीगेशन के लिए बहुत कम यूनिट टेस्ट कवरेज है। इस मेटा मुद्दे का उद्देश्य उसमें काफी सुधार करना है। प्रत्येक एकत्रीकरण के लिए हमें एग्रीगेटर के लिए अधिक यूनिट परीक्षण जोड़ना चाहिए (यह org.apache.lucene.search.Collector माध्यम से ल्यूसीन इंडेक्स के साथ कैसे इंटरैक्ट करता है जिसे प्रत्येक एकत्रीकरण लागू करता है), कम तर्क और एकत्रीकरण परिणामों का क्रमांकन।

हमें निम्नलिखित Aggregator कार्यान्वयन के लिए इकाई परीक्षण जोड़ना चाहिए:

• [x] ParentToChildrenAggregator @cbuescher #२३३०५
• [x] FilterAggregator @colings86 #23826
• [x] FiltersAggregator @jimczi #२२६७८
• [x] GeoHashGridAggregator @martijnvg #२३४१७
• [x] GlobalAggregator @nik9000 #22668
• [x] DateHistogramAggregator @tlrx #२२७१४
• [x] HistogramAggregator @jpountz #२२९६१
• [x] MissingAggregator @jimczi # MissingAggregator
• [x] NestedAggregator @polyfractal
• [x] ReverseNestedAggregator @cbuescher
• [x] RangeAggregator ( DateRangeAggregationBuilder और GeoDistanceAggregationBuilder साथ भी परीक्षण करें) @tlrx #24569
• [x] BinaryRangeAggregator ( IpRangeAggregationBuilder साथ भी परीक्षण करें) @jimczi #23255
• [x] डायवर्सिफाइड सैंपलर एग्रीगेटर, जिसके निम्नलिखित कार्यान्वयन हैं जिनका परीक्षण करने की आवश्यकता है: DiversifiedBytesHashSamplerAggregator , DiversifiedMapSamplerAggregator , DiversifiedNumericSamplerAggregator और DiversifiedOrdinalsSamplerAggregator । @martijnvg #23511
• [x] SamplerAggregator @nik9000 #23243
• [x] महत्वपूर्ण शब्द एकत्रीकरण: GlobalOrdinalsSignificantTermsAggregator , GlobalOrdinalsSignificantTermsAggregator.WithHash , SignificantLongTermsAggregator और SignificantStringTermsAggregator । @markharwood #24904
• [x] शर्तें एकत्रीकरण: DoubleTermsAggregator , GlobalOrdinalsStringTermsAggregator. LowCardinality , GlobalOrdinalsStringTermsAggregator. WithHash , LongTermsAggregator और StringTermsAggregator । @martijnvg #24949
• [x] BestBucketsDeferringCollector @MaineC #२३५११
• [x] BestDocsDeferringCollector @polyfractal #२३५११
• [x] AvgAggregator @cbuescher #23000
• [x] CardinalityAggregator @colings86 #23826
• [x] GeoBoundsAggregator @jimczi #23259
• [x] GeoCentroidAggregator @martijnvg #24111
• [x] MaxAggregator @nik9000 #22668
• [एक्स] MinAggregator [ एमवीजी ] #२२२७९
• [x] प्रतिशत मीट्रिक एकत्रीकरण: TDigestPercentilesAggregator और HDRPercentilesAggregator @tlrx #24245
• [x] प्रतिशत रैंक मीट्रिक एकत्रीकरण: TDigestPercentileRanksAggregator और HDRPercentileRanksAggregator । @jpountz #23240
• [x] ScriptedMetricAggregator @cbuescher #२३४०४
• [x] StatsAggregator @jimczi
• [x] ExtendedStatsAggregator @jimczi
• [x] SumAggregator @martijnvg # SumAggregator ५४
• [x] TopHitsAggregator @nik9000 #२२७५४
• [x] ValueCountAggregator @tlrx #२२७४१
• [x] MatrixStatsAggregator @martijnvg #२४८३७

हमें निम्नलिखित InternalAggregation कार्यान्वयन के लिए परीक्षण भी जोड़ने चाहिए:
(कम करने और परिणाम क्रमांकन तर्क का परीक्षण करने के लिए)

• [x] InternalChildren @cbuescher #23261
• [x] InternalFilter @colings86 #२३३८८
• [x] InternalFilters @jimczi #२२६७८
• [x] InternalGeoHashGrid @martijnvg #२३४१७
• [x] InternalGlobal @nik9000 #23388
• [x] InternalHistogram @jpountz #२२९६१
• [x] InternalDateHistogram @tlrx #२३४०२
• [x] InternalMissing @MaineC #२३३८८
• [x] InternalNested @polyfractal #२३३८८
• [x] InternalReverseNested @cbuescher #२३३८८
• [x] InternalRange , InternalDateRange , InternalGeoDistance । @tlrx #24569
• [x] InternalBinaryRange @jimczi #23259
• [x] InternalSampler @martijnvg edada2581e75400da9fac82bdfbc7ec1f02ef0d8
• [x] महत्वपूर्ण शब्द एकत्रीकरण: SignificantLongTerms और SignificantStringTerms । @tlrx #23428
• [x] शर्तें एकत्रीकरण: DoubleTerms , LongTerms , StringTerms और UnmappedTerms । @jpountz #23149
• [x] InternalAvg @cbuescher #23000
• [x] InternalCardinality @colings86 #23826
• [x] InternalGeoBounds @jimczi #23259
• [x] InternalGeoCentroid @martijnvg #24176
• [x] InternalMax @nik9000 #22668
• [x] InternalMin @colings86 + @nik9000 #22668
• [x] प्रतिशत मीट्रिक एकत्रीकरण: InternalTDigestPercentiles (#24090) और InternalHDRPercentiles (#24157) @tlrx
• [x] प्रतिशत रैंक मीट्रिक एकत्रीकरण: InternalTDigestPercentileRanks और InternalHDRPercentileRanks । @jpountz #23240
• [x] InternalScriptedMetric @cbuescher #23330
• [x] InternalStats @jimczi
• [x] InternalExtendedStats @jimczi
• [x] InternalSum @martijnvg # InternalSum ५४
• [x] InternalTopHits @nik9000
• [x] InternalValueCount @tlrx #२२७४१
• [x] InternalMatrixStats @martijnvg #२४५५९

हो सकता है कि मैं कुछ कक्षाएं भूल गया हूं, इसलिए अगर ऐसा है तो कृपया इस मुद्दे को अपडेट करें :)

मैंने InternalAggregation कार्यान्वयन को Aggregator कार्यान्वयन से अलग सूचीबद्ध किया क्योंकि प्रत्येक के लिए इकाई परीक्षण अलग-अलग देवों द्वारा समानांतर में लिखे जा सकते हैं। हालांकि मुझे लगता है कि InternalAggregation कार्यान्वयन और Aggregator कार्यान्वयन दोनों के लिए कम जटिल एकत्रीकरण इकाई परीक्षणों के लिए एक ही पीआर में जोड़ा जा सकता है।

मुझे लगता है कि मास्टर शाखा में कम से कम यूनिट परीक्षण जोड़े जाने चाहिए। 5.x शाखा में बैकपोर्ट करना सबसे अच्छा प्रयास है और इसे केवल तभी माना जाना चाहिए जब यह कम लटकने वाला फल हो।

मेरा सुझाव है कि गलती से दो बार काम करने से बचने के लिए हम निम्नलिखित तरीके से काम करें:

• इकाई परीक्षण लिखना शुरू करने से पहले अपना नाम उस कार्य में जोड़ें जिस पर आप काम करना चाहते हैं।
• जब आप पीआर खोलते हैं तो पीआर नंबर को टास्क में जोड़ें।
• एक बार जनसंपर्क विलय हो जाने के बाद, कार्य की जांच करें।

ध्यान दें कि एकत्रीकरण का परीक्षण किया जाता है, हालांकि कोड को संरचित करने से पहले, इकाई परीक्षण वास्तव में कठिन था।