Not sure how significant it is in this case, but in general we would prefer to do the work in the callback (which is hardly ever called) rather than in openNext() which is called a lot.