maobaolong commented on PR #1882: URL: https://github.com/apache/incubator-uniffle/pull/1882#issuecomment-2220373894
@jerqi I see. So how about another resolution, we do not add a uniffle pacakge name, instead, we add a class prefix with Uniffle. e.g. `SparkVersionUtils` -> `UnifleSparkVersionUtils` ? BTW, I'm open to keep it like it is now, we can merge this PR after we first encounter a class conflict issue involved by this. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
