Skip to content

Conversation

@yyanyy
Copy link
Contributor

@yyanyy yyanyy commented Jan 22, 2026

  • Also improved thread safety with filter() method for updating state mutations

Which Delta project/connector is this regarding?

  • Spark
  • Standalone
  • Flink
  • Kernel
  • Other (fill in here)

Description

How was this patch tested?

Additional unit test

Does this PR introduce any user-facing changes?

No

…arkTable equality check

- Also improved thread safety with filter() method for updating state mutations
assertEquals(scan1, scan2);
assertEquals(scan1.hashCode(), scan2.hashCode());

// Test case 2: Equivalent filters (different instances)
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: let's break these test cases into differnt test methods

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