acezen commented on PR #514: URL: https://github.com/apache/incubator-graphar/pull/514#issuecomment-2151747312
> > > @acezen What do you think about this? For me it looks like there are no changes for public API > > > > > > Hi, Sem, thanks for the draft, change to `org.apache.spark.sql` is good to me. But if we upload the binary package to maven, is that possible to upload all the binary packages to `org.apache.graphar`? > > Yes, it should be. For example, check this: > > 1. Download delta-saprk: https://mvnrepository.com/artifact/io.delta/delta-spark_2.13/3.2.0 > 2. Open it with archive manager > 3. You will see org-apache-spark classes inside: >  > > but coordinates are still `io.delta:delta-spark_2.13:3.2.0` > > So, if you OK with it I will update 32 too and fix license headers. Looks NICE! The change is OK for me. Thanks for the work. -- 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]
