Skip to content

AWS: Cleanup warning about Lambda should be method reference#5476

Merged
jackye1995 merged 1 commit intoapache:masterfrom
amogh-jahagirdar:cleanup-lambda-warning
Aug 9, 2022
Merged

AWS: Cleanup warning about Lambda should be method reference#5476
jackye1995 merged 1 commit intoapache:masterfrom
amogh-jahagirdar:cleanup-lambda-warning

Conversation

@amogh-jahagirdar
Copy link
Contributor

Cleans up a warning I see in the build for the AWS integ tests:

/Users/jahamogh/workplace/iceberg-jahamogh/iceberg/aws/src/integration/java/org/apache/iceberg/aws/lakeformation/TestLakeFormationMetadataOperations.java:244: warning: [LambdaMethodReference] Lambda should be a method reference
          () -> updateProperties.commit());
      ^

@github-actions github-actions bot added the AWS label Aug 9, 2022
Copy link
Contributor

@jackye1995 jackye1995 left a comment

Choose a reason for hiding this comment

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

thanks for the fix!

@jackye1995
Copy link
Contributor

I will just merge this because it is a simple style fix, thanks for the work!

@jackye1995 jackye1995 merged commit 3f5f2cd into apache:master Aug 9, 2022
@amogh-jahagirdar
Copy link
Contributor Author

Thanks for the review Jack!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants