Skip to content

Commit 32a2fe2

Browse files
committed
Revert "Revert "feat: profile concurrent transactions (#2105)" (#2225)"
This reverts commit 4c1fab4.
1 parent 4c1fab4 commit 32a2fe2

File tree

13 files changed

+721
-428
lines changed

13 files changed

+721
-428
lines changed

Sentry.xcodeproj/project.pbxproj

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -582,8 +582,10 @@
582582
7DC83100239826280043DD9A /* SentryIntegrationProtocol.h in Headers */ = {isa = PBXBuildFile; fileRef = 7DC830FF239826280043DD9A /* SentryIntegrationProtocol.h */; settings = {ATTRIBUTES = (Public, ); }; };
583583
7DC8310A2398283C0043DD9A /* SentryCrashIntegration.h in Headers */ = {isa = PBXBuildFile; fileRef = 7DC831082398283C0043DD9A /* SentryCrashIntegration.h */; };
584584
7DC8310C2398283C0043DD9A /* SentryCrashIntegration.m in Sources */ = {isa = PBXBuildFile; fileRef = 7DC831092398283C0043DD9A /* SentryCrashIntegration.m */; };
585+
8419C0C428C1889D001C8259 /* SentryProfilerSwiftTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = 8419C0C328C1889D001C8259 /* SentryProfilerSwiftTests.swift */; };
585586
8453421228BE855D00C22EEC /* SentrySampleDecision.m in Sources */ = {isa = PBXBuildFile; fileRef = 8453421128BE855D00C22EEC /* SentrySampleDecision.m */; };
586587
8453421628BE8A9500C22EEC /* SentrySpanStatus.m in Sources */ = {isa = PBXBuildFile; fileRef = 8453421528BE8A9500C22EEC /* SentrySpanStatus.m */; };
588+
84A888FD28D9B11700C51DFD /* SentryProfiler+Test.h in Headers */ = {isa = PBXBuildFile; fileRef = 84A888FC28D9B11700C51DFD /* SentryProfiler+Test.h */; };
587589
861265F92404EC1500C4AFDE /* NSArray+SentrySanitize.h in Headers */ = {isa = PBXBuildFile; fileRef = 861265F72404EC1500C4AFDE /* NSArray+SentrySanitize.h */; };
588590
861265FA2404EC1500C4AFDE /* NSArray+SentrySanitize.m in Sources */ = {isa = PBXBuildFile; fileRef = 861265F82404EC1500C4AFDE /* NSArray+SentrySanitize.m */; };
589591
8E0551E026A7A63C00400526 /* TestProtocolClient.swift in Sources */ = {isa = PBXBuildFile; fileRef = 8E0551DF26A7A63C00400526 /* TestProtocolClient.swift */; };
@@ -1325,6 +1327,7 @@
13251327
7DC830FF239826280043DD9A /* SentryIntegrationProtocol.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; name = SentryIntegrationProtocol.h; path = Public/SentryIntegrationProtocol.h; sourceTree = "<group>"; };
13261328
7DC831082398283C0043DD9A /* SentryCrashIntegration.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; name = SentryCrashIntegration.h; path = include/SentryCrashIntegration.h; sourceTree = "<group>"; };
13271329
7DC831092398283C0043DD9A /* SentryCrashIntegration.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = SentryCrashIntegration.m; sourceTree = "<group>"; };
1330+
8419C0C328C1889D001C8259 /* SentryProfilerSwiftTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = SentryProfilerSwiftTests.swift; sourceTree = "<group>"; };
13281331
844A34C3282B278500C6D1DF /* .github */ = {isa = PBXFileReference; lastKnownFileType = folder; path = .github; sourceTree = "<group>"; };
13291332
844A3563282B3C9F00C6D1DF /* .sauce */ = {isa = PBXFileReference; lastKnownFileType = folder; path = .sauce; sourceTree = "<group>"; };
13301333
844DA7F6282435CD00E6B62E /* README.md */ = {isa = PBXFileReference; lastKnownFileType = net.daringfireball.markdown; path = README.md; sourceTree = "<group>"; };
@@ -1350,6 +1353,7 @@
13501353
844DA81F28246DE300E6B62E /* scripts */ = {isa = PBXFileReference; lastKnownFileType = folder; path = scripts; sourceTree = "<group>"; };
13511354
8453421128BE855D00C22EEC /* SentrySampleDecision.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = SentrySampleDecision.m; sourceTree = "<group>"; };
13521355
8453421528BE8A9500C22EEC /* SentrySpanStatus.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = SentrySpanStatus.m; sourceTree = "<group>"; };
1356+
84A888FC28D9B11700C51DFD /* SentryProfiler+Test.h */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.c.h; name = "SentryProfiler+Test.h"; path = "Sources/Sentry/include/SentryProfiler+Test.h"; sourceTree = SOURCE_ROOT; };
13531357
861265F72404EC1500C4AFDE /* NSArray+SentrySanitize.h */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.h; name = "NSArray+SentrySanitize.h"; path = "include/NSArray+SentrySanitize.h"; sourceTree = "<group>"; };
13541358
861265F82404EC1500C4AFDE /* NSArray+SentrySanitize.m */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.c.objc; path = "NSArray+SentrySanitize.m"; sourceTree = "<group>"; };
13551359
8E0551DF26A7A63C00400526 /* TestProtocolClient.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = TestProtocolClient.swift; sourceTree = "<group>"; };
@@ -1499,6 +1503,7 @@
14991503
035E73CB27D575B3005EEB11 /* SentrySamplingProfilerTests.mm */,
15001504
035E73CD27D5790A005EEB11 /* SentryThreadMetadataCacheTests.mm */,
15011505
03F9D37B2819A65C00602916 /* SentryProfilerTests.mm */,
1506+
8419C0C328C1889D001C8259 /* SentryProfilerSwiftTests.swift */,
15021507
);
15031508
path = Profiling;
15041509
sourceTree = "<group>";
@@ -2681,6 +2686,7 @@
26812686
03F84D1B27DD414C008FE43F /* SentryMachLogging.hpp */,
26822687
03F84D2C27DD4191008FE43F /* SentryMachLogging.cpp */,
26832688
03F84D1127DD414C008FE43F /* SentryProfiler.h */,
2689+
84A888FC28D9B11700C51DFD /* SentryProfiler+Test.h */,
26842690
03F84D2B27DD4191008FE43F /* SentryProfiler.mm */,
26852691
03BCC38D27E2A377003232C7 /* SentryProfilingConditionals.h */,
26862692
03F84D2927DD416B008FE43F /* SentryProfilingLogging.hpp */,
@@ -2926,6 +2932,7 @@
29262932
8EE3251C261FE33B00DC3FF2 /* SentryUIViewControllerSanitizer.h in Headers */,
29272933
7BAF3DD92440AEC8008A5414 /* SentryRequestManager.h in Headers */,
29282934
7BE3C77B2446111500A38442 /* SentryRateLimitParser.h in Headers */,
2935+
84A888FD28D9B11700C51DFD /* SentryProfiler+Test.h in Headers */,
29292936
7D0637032382B34300B30749 /* SentryScope.h in Headers */,
29302937
03F84D2727DD414C008FE43F /* SentryMachLogging.hpp in Headers */,
29312938
0356A570288B4612008BF593 /* SentryProfilesSampler.h in Headers */,
@@ -3509,6 +3516,7 @@
35093516
63FE720520DA66EC00CDBAE8 /* FileBasedTestCase.m in Sources */,
35103517
0A6EEADD28A657970076B469 /* UIViewRecursiveDescriptionTests.swift in Sources */,
35113518
63EED6C32237989300E02400 /* SentryOptionsTest.m in Sources */,
3519+
8419C0C428C1889D001C8259 /* SentryProfilerSwiftTests.swift in Sources */,
35123520
7BBD18B22451804C00427C76 /* SentryRetryAfterHeaderParserTests.swift in Sources */,
35133521
7BD337E424A356180050DB6E /* SentryCrashIntegrationTests.swift in Sources */,
35143522
7BD4E8E827FD95900086C410 /* SentryMigrateSessionInitTests.m in Sources */,

Sources/Sentry/SentryFramesTracker.m

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
#import "SentryFramesTracker.h"
22
#import "SentryDisplayLinkWrapper.h"
3+
#import "SentryProfiler.h"
34
#import "SentryProfilingConditionals.h"
45
#import "SentryTracer.h"
56
#import <SentryScreenFrames.h>
@@ -123,7 +124,7 @@ - (void)displayLinkCallback
123124
}
124125

125126
# if SENTRY_TARGET_PROFILING_SUPPORTED
126-
if (self.currentTracer.isProfiling
127+
if ([SentryProfiler isRunning]
127128
&& (self.frameRateTimestamps.count == 0
128129
|| self.frameRateTimestamps.lastObject[@"frame_rate"].doubleValue
129130
!= actualFramesPerSecond)) {
@@ -161,7 +162,7 @@ - (void)displayLinkCallback
161162
# if SENTRY_TARGET_PROFILING_SUPPORTED
162163
- (void)recordTimestampStart:(NSNumber *)start end:(NSNumber *)end
163164
{
164-
if (self.currentTracer.isProfiling) {
165+
if ([SentryProfiler isRunning]) {
165166
[self.frameTimestamps addObject:@{ @"start_timestamp" : start, @"end_timestamp" : end }];
166167
}
167168
}

0 commit comments

Comments
 (0)