Skip to content

Conversation

@gthea
Copy link
Contributor

@gthea gthea commented Jan 11, 2026

Android SDK

What did you accomplish?

  • Added modifications to only expose flag names in updated flags SDK_UPDATE events. Segments changes are informed but no names are exposed.

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

Android Diffuse Analysis Results

Source Branch: FME-12217-segments_3
Target Branch: FME-11237_baseline
Commit: d14b69b96bb1a98aec0792976b0c57d1b69d9b29

Script Output:

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

 JAR     │ old  │ new  │ diff       
─────────┼──────┼──────┼────────────
 classes │  913 │  913 │  0 (+0 -0) 
 methods │ 5642 │ 5641 │ -1 (+1 -2) 
  fields │ 2595 │ 2595 │  0 (+0 -0) 

@devops-split
Copy link

SDK Readiness Test Results 🔴

Metric FME-12217-segments_3 P50 FME-11237_baseline P50 Significance
SDK_READY 443 ms 428 ms 🔴 p=0.008
SDK_READY_FROM_CACHE 418 ms 416 ms ⚪ n.s.

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


android-test-mock

  • Mock Server: true

  • Iterations: 40

@gthea gthea marked this pull request as ready for review January 12, 2026 13:44
@gthea gthea requested a review from a team as a code owner January 12, 2026 13:44
@gthea gthea marked this pull request as draft January 12, 2026 15:17
@devops-split
Copy link

Android Diffuse Analysis Results

Source Branch: FME-12217-segments_3
Target Branch: FME-11237_baseline
Commit: b078793fb575f6434e222902c42883a5cfd72d1d

Script Output:

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

 JAR     │ old  │ new  │ diff       
─────────┼──────┼──────┼────────────
 classes │  913 │  913 │  0 (+0 -0) 
 methods │ 5642 │ 5635 │ -7 (+2 -9) 
  fields │ 2595 │ 2595 │  0 (+0 -0) 

@devops-split
Copy link

SDK Readiness Test Results 🔴

Metric FME-12217-segments_3 P50 FME-11237_baseline P50 Significance
SDK_READY 415 ms 398 ms 🔴 p=0.012
SDK_READY_FROM_CACHE 404 ms 387 ms 🔴 p=0.000

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


android-test-mock

  • Mock Server: true

  • Iterations: 40

@devops-split
Copy link

Android Diffuse Analysis Results

Source Branch: FME-12217-segments_3
Target Branch: FME-11237_baseline
Commit: 3bb355612bab61fee1b074b14a7242be210c4366

Script Output:

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

 JAR     │ old  │ new  │ diff       
─────────┼──────┼──────┼────────────
 classes │  913 │  913 │  0 (+0 -0) 
 methods │ 5642 │ 5635 │ -7 (+2 -9) 
  fields │ 2595 │ 2595 │  0 (+0 -0) 

@devops-split
Copy link

SDK Readiness Test Results ✅

Metric FME-12217-segments_3 P50 FME-11237_baseline P50 Significance
SDK_READY 440 ms 441 ms ⚪ n.s.
SDK_READY_FROM_CACHE 420 ms 416 ms ⚪ n.s.

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


android-test-mock

  • Mock Server: true

  • Iterations: 40

@devops-split
Copy link

Android Diffuse Analysis Results

Source Branch: FME-12217-segments_3
Target Branch: FME-11237_baseline
Commit: 440e780259aa5d7be58c5329f233e07d20ac1546

Script Output:

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

 JAR     │ old  │ new  │ diff       
─────────┼──────┼──────┼────────────
 classes │  913 │  913 │  0 (+0 -0) 
 methods │ 5653 │ 5646 │ -7 (+2 -9) 
  fields │ 2595 │ 2595 │  0 (+0 -0) 

@sonarqube-pull-requests
Copy link

sonarqube-pull-requests bot commented Jan 13, 2026

@devops-split
Copy link

SDK Readiness Test Results 🔴

Metric FME-12217-segments_3 P50 FME-11237_baseline P50 Significance
SDK_READY 429 ms 381 ms 🔴 p=0.000
SDK_READY_FROM_CACHE 400 ms 375 ms 🔴 p=0.000

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


android-test-mock

  • Mock Server: true

  • Iterations: 40

@gthea gthea marked this pull request as ready for review January 13, 2026 14:59
@gthea gthea changed the title Add changed memberships metadata to event Expose flag names in metadata Jan 13, 2026
@gthea gthea merged commit bf6701e into FME-11237_baseline Jan 14, 2026
5 checks passed
@gthea gthea deleted the FME-12217-segments_3 branch January 14, 2026 11:19
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