Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Kernel] Fix typo in TableFeaturesSuite #4170

Merged
merged 1 commit into from
Feb 20, 2025

Conversation

anoopj
Copy link
Collaborator

@anoopj anoopj commented Feb 18, 2025

Which Delta project/connector is this regarding?

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

Description

Fix a minor typo in TableFeaturesSuite

How was this patch tested?

$ build/sbt kernelGroup/test

Does this PR introduce any user-facing changes?

No

@anoopj anoopj requested a review from scottsand-db February 18, 2025 21:46
@anoopj anoopj closed this Feb 20, 2025
@anoopj anoopj reopened this Feb 20, 2025
@vkorukanti vkorukanti changed the title Fix typo in TableFeaturesSuite [Kernel] Fix typo in TableFeaturesSuite Feb 20, 2025
@vkorukanti
Copy link
Collaborator

@anoopj could you please rebase and run build/sbt scalaFmtAll?

## Description
Fix typo in TableFeaturesSuite.scala in kernel

## How was this patch tested?
Existing UT
@anoopj
Copy link
Collaborator Author

anoopj commented Feb 20, 2025

@anoopj could you please rebase and run build/sbt scalaFmtAll?

Done.

@scottsand-db scottsand-db merged commit 71cf788 into delta-io:master Feb 20, 2025
21 checks passed
anoopj added a commit to anoopj/delta that referenced this pull request Feb 24, 2025
#### Which Delta project/connector is this regarding?
- [ ] Spark
- [ ] Standalone
- [ ] Flink
- [X] Kernel
- [ ] Other (fill in here)

## Description

Fix a minor typo in TableFeaturesSuite

## How was this patch tested?

$ build/sbt kernelGroup/test

## Does this PR introduce _any_ user-facing changes?

No
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