Skip to content
This repository has been archived by the owner on Dec 18, 2021. It is now read-only.

Allocator class stub #2

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Allocator class stub #2

wants to merge 3 commits into from

Conversation

treywood-qz
Copy link
Collaborator

@treywood-qz treywood-qz commented Feb 10, 2020

We recently had a candidate come into the backend interview set up to work in Ruby on Rails. That's fine and dandy except the candidate leveraged a Postgres database and the Rails ORM to do all of the heavy lifting. I think having such a bare example in this repo can set the wrong expectation coming into the interview, so I've added an Allocator class stub here that could hopefully clarify the expectation that we will ignore the web framework for a portion of the interview.

I wanted to also add something like

constructor() {
  this.servers = [];
}

to emphasize work to be done in-memory, but felt like that might be a bit too suggestive - thoughts?

@treywood-qz treywood-qz requested review from paulheider-qz and removed request for kgorski-quizlet February 10, 2020 21:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants