nsivabalan commented on a change in pull request #2963: URL: https://github.com/apache/hudi/pull/2963#discussion_r643879466
########## File path: hudi-client/hudi-client-common/src/main/java/org/apache/hudi/schema/SchemaProvider.java ########## @@ -34,18 +32,9 @@ @PublicAPIClass(maturity = ApiMaturityLevel.STABLE) public abstract class SchemaProvider implements Serializable { - protected TypedProperties config; + protected Schema sourceSchema; - protected JavaSparkContext jssc; - - public SchemaProvider(TypedProperties props) { Review comment: just now read other comments. I understand the intent to make it agnostic to engines, but not gonna be easy to make it backwards compatible. One more thought: we might need to make the base abstract class generic with two types (a config class and engine context may be). But this def needs more thought. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org