Skip to content

Conversation

evanSegment
Copy link
Contributor

Proposed changes

We don't document the billing logic used to determine object throughput. This leads to customer confusion, and especially pertinent now that some customer are being billed for throughput on volume pricing plans.

Merge timing

  • ASAP once approved

@netlify
Copy link

netlify bot commented Sep 3, 2021

✔️ Deploy Preview for segment-docs ready!

🔨 Explore the source changes: 150d15a

🔍 Inspect the deploy log: https://app.netlify.com/sites/segment-docs/deploys/613a3b07ffef830007803123

😎 Browse the preview: https://deploy-preview-1827--segment-docs.netlify.app

markzegarelli and others added 2 commits September 7, 2021 09:08
My copy previously indicated that "each object ingested counts as 1 throughput". That's not really accurate as throughput is defined as `(objects ingested + API calls received) / MTU allowance`.  The language is really tortured here, mostly because the definition of throughput is so complex. Not sure how to make this clearer and more readable, open to suggestions!
@evanSegment
Copy link
Contributor Author

@markzegarelli , @brookstaylorjr , I made some updates to the copy to correct my usage of the term "throughput". I'm not really happy with the readability of the copy, but the concept of throughput it really complicated and hard to reason about, so I'm not sure what else to do. Could you please review the updated copy?

@markzegarelli
Copy link
Contributor

@evanSegment The copy is clear to me. I think we can ship as-is pending feedback from @brookstaylorjr.

Copy link

@brookstaylorjr brookstaylorjr left a comment

Choose a reason for hiding this comment

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

Approved! It feels a little weird to me, that we wait so long on the page to actually define what an object is (when we reference them way up in the "Throughput" definition), but ultimately what matters is that the content is somewhere.

@markzegarelli markzegarelli merged commit a16ce65 into master Sep 13, 2021
@markzegarelli markzegarelli deleted the evanSegment-clarify-object-throughput branch September 13, 2021 16:31
@evanSegment
Copy link
Contributor Author

Thanks guys! I wonder if it would make sense to include this formula higher up the page where we first define throughput? The current copy under #what-is-throughput actually appears to define throughput limit rather than throughput itself!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants