Knex: वादे माइग्रेशन में अपेक्षित व्यवहार नहीं करते हैं

मुझे नहीं लगता कि आपको Promise.all([first, second]); वापस करना चाहिए, क्योंकि आपने पहले ही second से first । क्या आप कृपया केवल return first प्रयास कर सकते हैं? के बाद से अपने दूसरे createTable एक के अंदर है then जिनमें से बंद श्रृंखलित है first , वहाँ में जोड़ने के लिए कोई जरूरत नहीं है Promise.all , के बाद से यह निष्पादित हो जाएगा वैसे भी first का हल हो रहा है।

वादा युक्ति कहती है कि एक बार एक वादा पूरा हो जाने के बाद, इसे फिर से कभी पूरा नहीं किया जाना चाहिए, और then सभी भविष्य के आक्रमणों को कैश किया गया परिणाम वापस करना चाहिए। अगर first और second दोनों पूरे हो जाते हैं, तो Promise.all([first, second]) ~ एक सेशन नहीं होना चाहिए।

return first का व्यवहार समान है - अर्थात, यह first , first , और second टेबल बनाने की कोशिश करता है, जो _even अधिक गलत_ है, जैसा कि वहाँ है कभी भी ऐसा कोई कोड नहीं होना चाहिए जो वास्तव में उस परिदृश्य में first.then पास किए गए फ़ंक्शन के अंदर किसी भी चीज़ का मूल्यांकन करता हो।

उम्म ठीक है जो अजीब लगता है। Promise.all का उपयोग न करने से प्रदर्शन में थोड़ी वृद्धि हो सकती है;)

लेकिन हाँ, यह गलत प्रतीत होता है। मैं इसे अभी के लिए एक बग के रूप में चिह्नित करूंगा और @tgriesser की झंकार तक इंतजार

हाहा, मुझे अपने प्रवास में एक टिक के प्रदर्शन के मूल्य के बारे में विशेष रूप से परवाह नहीं है: पी

क्षमा करें, मैंने ऊपर एक गलत बयान दिया - return first को अभी भी दूसरी तालिका बनानी चाहिए, यह सिर्फ इतना है कि फ़ंक्शन के परिणाम पर .then कॉल करने का परिणाम इसके लिए इंतजार नहीं करेगा। जो, w / e - यह अभी भी निश्चित रूप से first , first , second !

मुझे यकीन नहीं है कि मैं पूरी तरह से समझता हूं: पहली तालिका के निर्माण के लिए दूसरी तालिका क्यों इंतजार नहीं करेगी? चूँकि आप then का उपयोग करके पहले वादे की श्रृंखला को बंद करते हैं,

दूसरी तालिका का निर्माण, हां, लेकिन exports.up फ़ंक्शन का परिणाम नहीं होगा। Ie, यदि आप exports.up().then कॉल करते हैं, तो यह first.then और second.then - क्योंकि यह _is_ first.then !

आह, तुम वहाँ गलत समझा - हाँ, तुम उस बारे में सही हो!

तो यहाँ भ्रम यह है कि createTable पद्धति एक वादा वापस नहीं करती है, बल्कि यह SchemaBuilder वस्तु देती है, जो कि "तत्कालीन" है, अर्थात .then पर कॉल करना ऑब्जेक्ट एक वैध A + वादा लौटाएगा, लेकिन वस्तु स्वयं एक वादा नहीं है।

यह विशेष रूप से आपको सिंटैक्स का उपयोग करने की अनुमति देने के लिए किया गया था:

  return knex.schema.createTable('first', function(table) {
    table.increments('id');
    table.string('name');
  })
  .createTable('second', function(table) {
      table.increments('id');
      table.string('name');
  }).then(function() {
    // all done
  });

जो अनुक्रम में एक ही कनेक्शन पर अपेक्षित के रूप में पलायन चलाना चाहिए।

इसके अलावा, यहाँ Promise.all की कोई आवश्यकता नहीं है, क्योंकि यह वही पूरा करेगा:

exports.up = function(knex, Promise) {
  return knex.schema.createTable('first', function(table) {
    table.increments('id');
    table.string('name');
  }).then(function() {
    return knex.schema.createTable('second', function(table) {
      table.increments('id');
      table.string('name');
    });
  });
};

यदि आप चाहते थे, तो आप कर सकते हैं:

exports.up = function(knex, Promise) {
  var first = knex.schema.createTable('first', function(table) {
    table.increments('id');
    table.string('name');
  }).then(); // coerce thenable to a promise

  var second = first.then(function() {
    return knex.schema.createTable('second', function(table) {
      table.increments('id');
      table.string('name');
    });
  });

  return Promise.all([first, second]);
};

और चीजें कल्पना के अनुसार काम करेंगी।

ठीक है कि बताते हैं। नहीं पता था कि - सीधे सीधे @SohumB जवाब देने में सक्षम नहीं होने के भ्रम के बारे में खेद है

हाँ, हमारे पास उससे अधिक जटिल निर्भरता ग्राफ हैं; यह सिर्फ एक न्यूनतम परीक्षण मामला था! सहायता के लिए धन्यवाद।

@johanneslumpe

sohum<strong i="8">@diurnal</strong> ~ % cat test.js
var Promise = require('bluebird');

function promises() {
  var first = Promise.resolve('wee');
  var second = first.then(function() {
    console.log('delayin');
    return Promise.delay(1000);
  }).then(function() {
    console.log('done!');
  });
  return first;
}

promises().then(function() { console.log('yep, first is finished'); });

sohum<strong i="9">@diurnal</strong> ~ % node test.js
delayin
yep, first is finished
done!