Skip to content

Refactor split name filtering for case insensitivity - #849

Open
danibachar wants to merge 1 commit into
splitio:masterfrom
danibachar:patch-2
Open

Refactor split name filtering for case insensitivity#849
danibachar wants to merge 1 commit into
splitio:masterfrom
danibachar:patch-2

Conversation

@danibachar

Copy link
Copy Markdown

iOS SDK

What did you accomplish?

Small code improvements

  • Updated API for string comparison
  • Small performance improvement avoiding converting the string to lower case
  • Avoid filtering - go with greedy API of first(where:)

How do we test the changes introduced in this PR?

Regular tests should pass as is

@danibachar
danibachar requested a review from a team as a code owner August 13, 2026 16:00
@devops-split

Copy link
Copy Markdown

SDK Readiness Test Results ✅

Metric patch-2 P50 master P50 Significance
SDK_READY 128 ms 124 ms -
SDK_READY_FROM_CACHE 91 ms 92 ms -

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


ios-test-mock

  • Mock Server: true

  • Iterations: 40

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.

2 participants