feat(database): batch create sessions

[callmehiphop]

There's a decrease in code coverage, however I believe this feature is tested at 100%.

• Tests and linter pass
• Code coverage does not decrease (if any source code was changed)
• Appropriate docs were updated (if necessary)

[codecov]

Codecov Report

Merging #692 into master will increase coverage by 0.02%.
The diff coverage is 99.43%.

@@            Coverage Diff             @@
##           master     #692      +/-   ##
==========================================
+ Coverage   96.09%   96.12%   +0.02%     
==========================================
  Files          18       18              
  Lines       15483    15607     +124     
  Branches      528      532       +4     
==========================================
+ Hits        14879    15002     +123     
  Misses        600      600              
- Partials        4        5       +1

Impacted Files	Coverage Δ
src/session-pool.ts	99.2% <100%> (ø)	⬆️
src/database.ts	99.81% <99.15%> (-0.04%)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d7075e3...9abd3f4. Read the comment docs.

[bcoe]

@callmehiphop I believe the decrease in coverage is related to a slight bug with default parameters showing up as uncovered.

I wouldn't worry about such a small drop in coverage.

[bcoe]

don't have a super strong opinion on this feature, going to leave it to @jkwlui, @crwilcox, and company to chime in.

[skuruppu]

Approval for the logic in the PR. I will leave it up to repo owners to approve on the Node.js specifics.

[crwilcox]

@callmehiphop it seems some files are missing license headers?

[callmehiphop]

@crwilcox yep, but the file in question is generated so while I could add them, they're likely to magically disappear in the future. @bcoe @alexander-fenster any way around this?

[bcoe]

I don't think we should block merging on license headers for generated code, but we've discussed this in chat with @alexander-fenster (and I'm hoping we'll start populating headers appropriately soon).

[callmehiphop]

@bcoe SGTM.

@crwilcox did you have a chance to review this before I merge?