LFW211 lab 8.1
I think I solved the 8.1 lab correctly but I need someone expert in the matter to check the work as the solution felt a bit weird!
const print = (err, contents) => { if (err) console.error(err) else console.log(contents ) } const opA = (err,cb) => { setTimeout(() => { cb(null, 'A') }, 500) } const opB = (err,cb) => { setTimeout(() => { cb(null, 'B') }, 250) } const opC = (err,cb) => { setTimeout(() => { cb(null, 'C') }, 125) } opA(null,print) opB(null,print) opC(null,print)
Best Answer
-
@abdullah.yf yep this completes the exercise, nice job
@SyuheiSaito these functions don't return anything, so shouldn't be passed to
then
, they also have signatures that expect err and cb functions, where asthen
passes the prior promise into the function. Finally, chaining promises like that means the code will execute serially, and what we want is for it to execute concurrently.0
Answers
-
Hello,
I'm not the expert but since the lab name is parallel execution I guess your solution is OK0 -
thank you
0 -
I've failed 8-1
`const print = (err, contents) => {
if (err) console.error(err)
else console.log(contents)
}const opA = (err,cb) => {
setTimeout(() => {
cb(null, 'A')
}, 500)
}const opB = (err,cb) => {
setTimeout(() => {
cb(null, 'B')
}, 250)
}const opC = (err,cb) => {
setTimeout(() => {
cb(null, 'C')
}, 125)
}let promise = new Promise(print)
promise.then(opA(null,print))
.then(opB(null,print))
.then(opC(null,print))`0 -
Imho we don't need to change the lab code like @abdullah.yf, just do that no ?
opA(print);
opB(print);
opC(print);0 -
@dtravailloux that doesn't match the function signatures. The functions take take two arguments (err, cb).
The main point of this exercise is to demonstrate the non-linear execution of code in the concurrency-based JS language
0 -
Hi @davidmarkclements, i m a bit confused.
I too see what @dtravailloux sees, as function signature, onparallel.js
and on the lab description.The functions opA, opB & opC, they take just ONE parameter and this is the
cb
, like so:const opA = (cb) => {
setTimeout(() => {
cb(null, 'A')
}, 500)
}Based on the above, the correct solution should be:
opA(print)
opB(print)
opC(print)1 -
Hi @theodoros , the corect solution for labs 8.1 is the same as yours in my opinion.
All the tasks could just start in parallel.For labs 8.2, opA, opB and opC are just simulations to callback-based async functions.
These functions will do their own async jobs. In most cases we don't what exactly they do.
But we know that if there will not be errors, they finished their jobs successfully.After their sucess, they will invoke the callback -
cb
you passed to them. And the arguments passed to the callback you provide, will be error first, then the results based on their works(those async tasks).So the signatures of opA, opB and opC take just ONE parameter is correct!
What matters is the signatures ofcb
s you provide.But if you promisify opA, opB and opC, promisify will take care of the
cb
s for you. You don't really need think about them.0 -
good answer @krave
0 -
@theodoros same solution with mine
0 -
here was my solution —
const print = (err, contents) => err ? console.error(err) : console.log(contents) const opA = (cb) => setTimeout(() => { cb(null, 'A') }, 500) const opB = (cb) => setTimeout(() => { cb(null, 'B') }, 250) const opC = (cb) => setTimeout(() => { cb(null, 'C') }, 125) const run = async () => { Promise.all([opA(print), opB(print), opC(print)]) } run().catch(console.error)
that returned, in the JavaScript console —
$ node parallel.js C B A
... so I believe I got this one right, yes?
0 -
@faddah no this just creates a Promise from an array with three elements containing the values
undefined
while also executing the callbacks and then it wraps another promise (viaasync
function) but doesn't await the promise from Promise.all either so this:- creates two promises that don't do anything
- calls the ops using just callbacks
0
Categories
- All Categories
- 112 LFX Mentorship
- 112 LFX Mentorship: Linux Kernel
- 604 Linux Foundation IT Professional Programs
- 315 Cloud Engineer IT Professional Program
- 139 Advanced Cloud Engineer IT Professional Program
- 52 DevOps Engineer IT Professional Program
- 67 Cloud Native Developer IT Professional Program
- 4 Express Training Courses
- 4 Express Courses - Discussion Forum
- 5.3K Training Courses
- 17 LFC110 Class Forum - Discontinued
- 8 LFC131 Class Forum
- 30 LFD102 Class Forum
- 178 LFD103 Class Forum
- LFD110 Class Forum
- 24 LFD121 Class Forum
- LFD133 Class Forum
- 2 LFD137 Class Forum
- 62 LFD201 Class Forum
- 2 LFD210 Class Forum
- 1 LFD210-CN Class Forum
- 1 LFD213 Class Forum - Discontinued
- 127 LFD232 Class Forum - Discontinued
- LFD233 Class Forum
- LFD237 Class Forum
- 22 LFD254 Class Forum
- 643 LFD259 Class Forum
- 107 LFD272 Class Forum
- 1 LFD272-JP クラス フォーラム
- 4 LFD273 Class Forum
- 1 LFS101 Class Forum
- LFS112 Class Forum
- LFS116 Class Forum
- LFS118 Class Forum
- LFS142 Class Forum
- LFS144 Class Forum
- 2 LFS145 Class Forum
- LFS146 Class Forum
- LFS151 Class Forum
- LFS157 Class Forum
- LFS158 Class Forum
- LFS162 Class Forum
- LFS166 Class Forum
- LFS167 Class Forum
- 28 LFS200 Class Forum
- 735 LFS201 Class Forum - Discontinued
- 1 LFS201-JP クラス フォーラム
- 13 LFS203 Class Forum
- 98 LFS207 Class Forum
- 299 LFS211 Class Forum
- 54 LFS216 Class Forum
- 47 LFS241 Class Forum
- 41 LFS242 Class Forum
- 36 LFS243 Class Forum
- 12 LFS244 Class Forum
- LFS245 Class Forum
- 41 LFS250 Class Forum
- 1 LFS250-JP クラス フォーラム
- LFS251 Class Forum
- 141 LFS253 Class Forum
- LFS254 Class Forum
- LFS255 Class Forum
- 2 LFS256 Class Forum
- LFS257 Class Forum
- 1.2K LFS258 Class Forum
- 9 LFS258-JP クラス フォーラム
- 109 LFS260 Class Forum
- 144 LFS261 Class Forum
- 39 LFS262 Class Forum
- 82 LFS263 Class Forum - Discontinued
- 15 LFS264 Class Forum - Discontinued
- 11 LFS266 Class Forum - Discontinued
- 20 LFS267 Class Forum
- 18 LFS268 Class Forum
- 26 LFS269 Class Forum
- 198 LFS272 Class Forum
- 1 LFS272-JP クラス フォーラム
- LFS274 Class Forum
- 3 LFS281 Class Forum
- 254 LFW211 Class Forum
- 173 LFW212 Class Forum
- 9 SKF100 Class Forum
- SKF200 Class Forum
- 781 Hardware
- 198 Drivers
- 68 I/O Devices
- 37 Monitors
- 95 Multimedia
- 174 Networking
- 87 Printers & Scanners
- 83 Storage
- 742 Linux Distributions
- 80 Debian
- 66 Fedora
- 15 Linux Mint
- 13 Mageia
- 23 openSUSE
- 143 Red Hat Enterprise
- 31 Slackware
- 13 SUSE Enterprise
- 347 Ubuntu
- 450 Linux System Administration
- 31 Cloud Computing
- 69 Command Line/Scripting
- Github systems admin projects
- 89 Linux Security
- 76 Network Management
- 101 System Management
- 46 Web Management
- 51 Mobile Computing
- 18 Android
- 23 Development
- 1.2K New to Linux
- 1K Getting Started with Linux
- 355 Off Topic
- 109 Introductions
- 167 Small Talk
- 18 Study Material
- 504 Programming and Development
- 283 Kernel Development
- 203 Software Development
- 844 Software
- 210 Applications
- 180 Command Line
- 3 Compiling/Installing
- 107 Games
- 308 Installation
- 51 All In Program
- 51 All In Forum
Upcoming Training
-
August 20, 2018
Kubernetes Administration (LFS458)
-
August 20, 2018
Linux System Administration (LFS301)
-
August 27, 2018
Open Source Virtualization (LFS462)
-
August 27, 2018
Linux Kernel Debugging and Security (LFD440)