Node-redis: retry_strategy से लौटने में त्रुटि (त्रुटि) घटना पर उत्सर्जित नहीं होती है

को निर्मित 21 जुल॰ 2016 · 10टिप्पणियाँ · स्रोत: NodeRedis/node-redis

संस्करण : नोड 4.2.1, रेडिस 2.6.2
प्लेटफार्म : मैक ओएस 10.11.5
विवरण :

हमने एक retry_strategy लागू किया था जो एक निश्चित मात्रा में पुनर्प्रयासों के बाद एक त्रुटि का उत्सर्जन करता है। इसके बाद यह रेडिस के ऑन ('त्रुटि') हैंडलर को लॉग इन करने और एप्लिकेशन से बाहर निकलने के लिए बबल होगा। हमारी रेडिस लाइब्रेरी को 2.4.2 से 2.6.2 अपडेट करते समय हमने देखा कि यह व्यवहार अब पहले जैसा नहीं रहा।

रेप्रो केस:

const redis = require('redis');
const MAX_ATTEMPTS = 10;
var client = redis.createClient(port, host, { 
  retry_strategy: redisRetryStrategy
});
client.on('error', onError);

function OnError(err) {
  console.log(err);
  throw new Error('SHUTDOWN THE APP, REDIS IS NOT RESPONDING??');
}

function redisRetryStrategy(host, options) {
  if (options.attempt >= MAX_ATTEMPTS) {
    // Stop reconnecting after reaching the maximum number of attempts
    return new Error('Maximum redis reconnect attempts reached');
  }
  return 1000; // Schedule next reconnection attempt after 1 sec.
}

स्रोत

crunchie84

👍7

सबसे उपयोगी टिप्पणी

हां: त्रुटि श्रोता को त्रुटि बबल करें

BridgeAR 21 जुल॰ 2016

👍12

सभी 10 टिप्पणियाँ

यह वास्तव में कुछ ऐसा है जो उद्देश्य से किया गया था। इसका कारण यह है कि आपके पास पहले से ही जानकारी है कि क्लाइंट विफल हो गया है, इसलिए इसे अब और उत्सर्जित करने की आवश्यकता नहीं है। यह वास्तव में सेवर को ऊपर रखने के बारे में एक अच्छा बदलाव नहीं है: / मुझे इसके लिए खेद है।

मैं यह देखने के लिए एक छोटा सा सर्वेक्षण करना चाहता हूं कि दूसरे लोग क्या सोच रहे हैं। पिंग @NodeRedis/योगदानकर्ता @dirkbonhomme

BridgeAR 21 जुल॰ 2016

हां: त्रुटि श्रोता को त्रुटि बबल करें

BridgeAR 21 जुल॰ 2016

👍12

नहीं: वर्तमान समाधान रखें

BridgeAR 21 जुल॰ 2016

पूर्व के पक्ष में उपयोग के मामले की पेशकश करने के लिए, कनेक्ट-रेडिस एक logErrors विकल्प प्रदान करता है जो (गैर-स्पष्ट रूप से) retry_strategy साथ पारस्परिक रूप से अनन्य हो जाता है यदि त्रुटियां बुलबुला नहीं होती हैं।

new RedisStore({
  logErrors: (err) => console.error, // does nothing
  retry_strategy: (options) => {
    if (options.error.code === 'ECONNREFUSED' || options.error.code === 'ENOTFOUND') {
      return new Error('The server could not be found or refused the connection');
    }
    return 1000;
  }
})

@tj

drew-r 4 अग॰ 2016

मुझे लगता है कि वर्तमान कार्यक्षमता तब तक ठीक है जब तक यह दस्तावेज़ीकरण में है, जो यह नहीं है। मुझे लगता है कि यह एक विकल्प जोड़ने के लिए भी स्वीकार्य होगा जो अभी भी पुनः प्रयास रणनीति से लौटाई गई त्रुटियों को बबल अप करेगा।
भूतपूर्व।:

redis.createClient({
    retry_strategy: ...,
    retry_strategy_bubble_up: true or false (default to false to keep existing functionality)
});

यह पता लगाने की कोशिश में बस आखिरी दिन बिताया कि त्रुटियां लॉग क्यों नहीं की जा रही थीं। अब जब लॉगिंग retry_strategy कॉलबैक में है, तो सब कुछ वैसा ही है जैसा मैं इसे अपने आवेदन में चाहता हूं।

raijinsetsu 17 फ़र॰ 2017

त्रुटि को फेंकने के बजाय, मैं मैन्युअल रूप से त्रुटि ईवेंट उत्सर्जित कर रहा हूं जिसे त्रुटि हैंडलर द्वारा पकड़ा जा सकता है:

retry_strategy: function(options) {
            logger.debug('Retry redis connection: ', options.attempt);

            // 5 mins
            if (options.total_retry_time > 1000 * 60 * 5) {
                // End reconnecting after a specific timeout and flush all commands with a individual error
                return logger.error('Redis: maximum retry time exhausted');
            }
            if (options.attempt > 5) {
                // End reconnecting with built in error
                logger.error('Redis: maximum connection retry reached');
                return client.emit('error', 'maximum retry');
            }
            return 1000 * 2;
        },

alannesta 18 अप्रैल 2017

👍2

बुलबुला उठना ! (या कुछ और)

कस्टम एरर एमिटर जैसे @alannesta ने लिखा है कि यह गड़बड़ है, retry_strategy कंस्ट्रक्टर के मापदंडों में से एक है, इसे फंक्शन के बाहर की वस्तु तक नहीं पहुंचना चाहिए।

NodeRedis पहले से ही एक ईवेंट सिस्टम था, बस इसका उपयोग करें!

चूंकि end लिया गया है, और इसका मतलब है कि "कनेक्शन बंद है", हम एक नई घटना जोड़ सकते हैं (चलिए इसे अभी "शटडाउन" कहते हैं) यह दर्शाता है कि "हमने वह सब कुछ किया जो हम कर सकते थे, यह कनेक्शन मर चुका है , अब से, हम इसके बारे में कुछ नहीं करते हैं, बेहतर होगा कि आप कुछ सफाई कार्य करें"।

मुझे वास्तव में इस बदलाव की जरूरत है।

AaronJan 10 अग॰ 2017

👍3

क्या इस मुद्दे के साथ कुछ हुआ?

lp-belliot 17 अप्रैल 2018

क्या कोई कारण है कि यह तय नहीं है या सिर्फ प्राथमिकता है? घटना का उत्सर्जन न करने का क्या लाभ है? retry_strategy का उपयोग करते समय अधिक काम करने वाले रेडिस कनेक्शन वाली सेवा को बंद करने का कोई स्पष्ट तरीका नहीं है और यह एक बहुत ही बुनियादी उपयोग का मामला होना चाहिए। हालांकि सबसे अच्छा समाधान शायद एक अलग घटना होगी जैसा कि @AaronJan ने सुझाव दिया था ..

Tapppi 11 दिस॰ 2019

मैंने v3.0.0 जारी किया है जो अब retry_strategy त्रुटियों को error ईवेंट श्रोता तक लौटाता है।

Salakar 9 फ़र॰ 2020

क्या यह पृष्ठ उपयोगी था?

0 / 5 - 0 रेटिंग्स

Node-redis: retry_strategy से लौटने में त्रुटि (त्रुटि) घटना पर उत्सर्जित नहीं होती है

सबसे उपयोगी टिप्पणी

सभी 10 टिप्पणियाँ

संबंधित मुद्दों