Skip to content

Conversation

@gthea
Copy link
Contributor

@gthea gthea commented Jan 9, 2026

Android SDK

What did you accomplish?

  • Expose same metadata in SDK ready and SDK ready from cache event listeners.
  • Rename freshInstall property to initialCacheLoad.

@gthea gthea self-assigned this Jan 9, 2026
@devops-split
Copy link

Android Diffuse Analysis Results

Source Branch: FME-11237-ready
Target Branch: FME-11237_baseline
Commit: 3584d70b3f9dd2076e7070598d378064705ab860

Script Output:

 AAR      │ old     │ new     │ diff   
──────────┼─────────┼─────────┼────────
      jar │ 1.1 MiB │ 1.1 MiB │ +781 B 
 manifest │   213 B │   213 B │    0 B 
    other │ 2.9 KiB │ 2.9 KiB │    0 B 
──────────┼─────────┼─────────┼────────
    total │ 1.1 MiB │ 1.1 MiB │ +781 B 

 JAR     │ old  │ new  │ diff          
─────────┼──────┼──────┼───────────────
 classes │  913 │  913 │   0 (+1 -1)   
 methods │ 5642 │ 5652 │ +10 (+31 -21) 
  fields │ 2595 │ 2595 │   0 (+3 -3)   

@devops-split
Copy link

Android Diffuse Analysis Results

Source Branch: FME-11237-ready
Target Branch: FME-11237_baseline
Commit: ec6f0e7eac23c2a17c8a021f2b2c4a4d6accc7f0

Script Output:

 AAR      │ old     │ new     │ diff   
──────────┼─────────┼─────────┼────────
      jar │ 1.1 MiB │ 1.1 MiB │ +822 B 
 manifest │   213 B │   213 B │    0 B 
    other │ 2.9 KiB │ 2.9 KiB │    0 B 
──────────┼─────────┼─────────┼────────
    total │ 1.1 MiB │ 1.1 MiB │ +822 B 

 JAR     │ old  │ new  │ diff          
─────────┼──────┼──────┼───────────────
 classes │  913 │  913 │   0 (+1 -1)   
 methods │ 5642 │ 5653 │ +11 (+32 -21) 
  fields │ 2595 │ 2595 │   0 (+3 -3)   

@sonarqube-pull-requests
Copy link

sonarqube-pull-requests bot commented Jan 9, 2026

@devops-split
Copy link

SDK Readiness Test Results ✅

Metric FME-11237-ready P50 FME-11237_baseline P50 Significance
SDK_READY 465 ms 458 ms ⚪ n.s.
SDK_READY_FROM_CACHE 432 ms 424 ms ⚪ n.s.

Legend: 🟢 significant improvement | 🔴 significant regression | ⚪ inconclusive (p≥0.05)


android-test-mock

  • Mock Server: true

  • Iterations: 20

@splitio splitio deleted a comment from devops-split Jan 9, 2026
@gthea gthea marked this pull request as ready for review January 9, 2026 21:20
@gthea gthea requested a review from a team as a code owner January 9, 2026 21:20
* @param metadata the event metadata, can be null
*/
void notifyInternalEvent(SplitInternalEvent internalEvent, @Nullable EventMetadata metadata);

Copy link

Choose a reason for hiding this comment

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

Why we are using "Split" name? I thought it suppose to be ISdkEventsManager

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