Should invalid integrations transaction be recorded as dropped or discarded silently?

[krystofwoldrich]

Description

Currently, transactions discarded by our integrations are being recorded as client dropped because of sampling_rate even when the sampling rate is set to 1.

How transaction.sampled = false is used in RN SDK.

client.recordDroppedEvent('event_processor', 'transaction') better reflects the reason.

This will likely require extra property on the Transaction class that will be handled onFinish the same as sampled

[krystofwoldrich]

Should we send this information at all? List all the cases where the integrations set sampled = false.

[krystofwoldrich]

The following list captures when sampled = false is set by integration.

React Native Tracing:

• Navigation
  • no child spans back navigation
• User Interaction
  • transaction without child spans

React Native Navigation:

• new navigation initiated before the current navigation finished mounting new components (original transaction will not be sampled)
• navigation did not mount components before timeout
• navigated to the same component

React Navigation:

• new navigation dispatched before the state changed (original transaction will not be sampled)
• state did not change before timeout

React Navigation v4:

• state did not change before timeout

[kahest]

@krystofwoldrich as this recently came up, I think we should decide and prioritize this