Skip to content
This repository has been archived by the owner on Jul 11, 2024. It is now read-only.

Optimising an AWS microservice - part one #250

Merged

Conversation

jmunro89
Copy link
Contributor

@jmunro89 jmunro89 commented Nov 3, 2018

This is my first attempt at a blog post so hopefully I've got it right...
Have loaded using docker and had a quick review - all seems to render correctly.

There is a part 2 written, will submit another PR for that separately.

Copy link
Contributor

@adamhepton adamhepton left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Like the post Jamie, really interesting delve into the ways of thinking and the challenges you gave yourselves. Just a few minor grammar/style amends, please.

* This is stored in a DynamoDB table fronted by an API that allows the traders to PUT opinion and our system to GET it when required

### The Pricing Model
* This is a java lambda - you pass it a description of what might happen in a football match along with the necessary opinion and it returns a price.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Capitalise "Java"


### The Pricing Model
* This is a java lambda - you pass it a description of what might happen in a football match along with the necessary opinion and it returns a price.
* E.G. you might request “Leeds Utd to score 3+ goals” and it might tell you this is 2/1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"eg.", not "E.G."

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's "e.g.", but because it's at the start of a sentence, it should be "E.g." ;-) I'd just opt for spelling it out though:

Suggested change
* E.G. you might request “Leeds Utd to score 3+ goals” and it might tell you this is 2/1
* For example, you might request “Leeds Utd to score 3+ goals” and it might tell you this is 2/1


### Pricing Service
* This is made up of an API Gateway and Java/Kotlin lambda
* The API Gateway has a ‘Lambda Authoriser’ that ensures our system can only be accessed from whitelisted ip ranges
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"IP" rather than "ip"


## Our Test Scenario

We wrote a performance test in Node JS that populates our opinion store with fake fixtures and then sends requests to our pricing service, recording the latency of each request.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We tend to use "NodeJS" as the term everywhere else

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For consistency, we should match the canonical form used by the maintainer; our most common ones are listed on the jobs page, so in this instance we should use "Node.js"


## Uh oh… things aren’t as they seem :-(
Now under our revised target, we were about to wrap up the project when someone noticed something odd. Our DynamoDB table only had opinion for 1 fixture in despite the test saying it had created 100 :scream: Uh oh!
It turns out we had a typo in our node code in the performance test which had slipped through review.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Capitalise "Node"

@cmbuckley
Copy link
Contributor

I'd generally advise to remove the smart formatting (e.g. curly quotes, en dashes, ellipses) and have Markdown apply that automatically

@jmunro89
Copy link
Contributor Author

jmunro89 commented Nov 5, 2018

I'd generally advise to remove the smart formatting (e.g. curly quotes, en dashes, ellipses) and have Markdown apply that automatically
** for example curly quotes, en dashes, ellipses **

@adamhepton adamhepton merged commit 636e3c5 into skybet:master Nov 5, 2018
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.

3 participants