Github user budde commented on the issue:
https://github.com/apache/spark/pull/16944
Thanks for the thorough feedback.
In the latest update I refactored ```convertToLogicalRelation()``` to
update the schema of the ```catalogTable``` field of the value it returns
whenever a schema is successfully inferred rather than only when one is
inferred and successfully saved. This allowed me to clean up the changes to
this method a bit. @ericl -- feel free to give this another look over.
Per @cloud-fan's feedback, I added an ```alterTableSchema()``` method to
the ExternalTable interface. I've added implementations of it to
```InMemoryCatalog``` and ```HiveExternalCatalog``` along with tests. I've also
addressed the feedback from @viirya
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
