Browse Source

feat(profiling): add p50 column (#66099)

User complaint, not sure why they thought p50 disappeared. I think many
other pages display p50 by default so we'll just do the same here
Jonas 1 year ago
parent
commit
56a696c2c2

+ 7 - 0
static/app/components/profiling/profileEventsTable.tsx

@@ -310,6 +310,7 @@ const FIELDS = [
   'os.name',
   'os.version',
   'last_seen()',
+  'p50()',
   'p75()',
   'p95()',
   'p99()',
@@ -322,6 +323,7 @@ type FieldType = (typeof FIELDS)[number];
 const RIGHT_ALIGNED_FIELDS = new Set<FieldType>([
   'transaction.duration',
   'profile.duration',
+  'p50()',
   'p75()',
   'p95()',
   'p99()',
@@ -445,6 +447,11 @@ const COLUMN_ORDERS: Record<FieldType, GridColumnOrder<FieldType>> = {
     name: t('Last Seen'),
     width: COL_WIDTH_UNDEFINED,
   },
+  'p50()': {
+    key: 'p50()',
+    name: t('P50()'),
+    width: COL_WIDTH_UNDEFINED,
+  },
   'p75()': {
     key: 'p75()',
     name: t('P75()'),

+ 1 - 0
static/app/views/profiling/content.tsx

@@ -321,6 +321,7 @@ const BASE_FIELDS = [
   'transaction',
   'project.id',
   'last_seen()',
+  'p50()',
   'p75()',
   'p95()',
   'p99()',