New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ref(perf): Move out the span transactions table #69771
Conversation
It's only used here!
Remove conditional routing and unused parameters
Bundle ReportChanges will increase total bundle size by 4.67kB ⬆️
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice refactor!
field: (typeof SORTABLE_FIELDS)[number]; | ||
}; | ||
|
||
export function isAValidSort(sort: Sort): sort is ValidSort { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
neat
sorts: [sort], | ||
limit: TRANSACTIONS_TABLE_ROW_COUNT, | ||
cursor, | ||
referrer: 'api.starfish.span-transaction-metrics', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we should probably rename this referrer eventually
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, definitely! Cleaning up referrers everywhere is on my list
This one's gross! In short, we had a component called the
SpanTransactionsTable
. It was a transactions table with some conditional logic, to render different columns and fetch different data depending on what module was using it.The problem? The only module using it is the database module. So, I'm dumping this "dynamic" table, and replacing it with a normal table, the kind we have all over the place. This will make it easier to consolidate the code, and remove a few rarely-used utils like
extractRoute
.I'm also removing some cruft like the competing
endpoint
andtransaction
parameters that we don't use anymore, lifting up the data loading state, etc.