wypoon commented on pull request #2512: URL: https://github.com/apache/iceberg/pull/2512#issuecomment-844615564
> @wypoon, it's much easier for users if we can use shims to handle API compatibility issues. Is Alias the only problem? Can we use a Spark shim to instantiate it? I can handle Alias by using reflection (DynConstructors) as well, but the real issue is this: other compatibility differences that are currently addressed are all found at compile-time; the problem with Alias was found at runtime, and only because there is an integration test that exercised that code path. The integration test suite is just a smoke test. There is no telling what other compatibility issues there are. That is why my preference is to build separate runtime jars. -- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
