-
Notifications
You must be signed in to change notification settings - Fork 296
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
[#1908] Fix: In HadoopCatalogOperations.java field entity is not used #4350
base: main
Are you sure you want to change the base?
Conversation
@poojatalele |
@yuqi1129 |
You ned to run |
Pls check now once |
Any update? |
Entity.EntityType.SCHEMA, | ||
schemaEntity -> updateSchemaEntity(ident, schemaEntity, changes)); | ||
|
||
SchemaEntity entity = store.get(ident, Entity.EntityType.SCHEMA, SchemaEntity.class); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure why do you add this line of code here, can you explain more?
What changes were proposed in this pull request?
Removed the unused entity variable in the alterSchema method of HadoopCatalogOperations.java. The refactored code now directly updates the schema and retrieves the updated schema entity for returning.
Why are the changes needed?
The entity variable was assigned but never used, which was flagged as a possible bug.
Removing this unused variable makes the code cleaner and easier to maintain.
Fix: #1908
Does this PR introduce any user-facing change?
No, this PR does not introduce any user-facing changes.
How was this patch tested?