ajantha-bhat edited a comment on pull request #3671: URL: https://github.com/apache/iceberg/pull/3671#issuecomment-986413732
> The new name of the file is: spark/v3.1/spark/src/main/java/org/apache/spark/sql/catalyst/plans.logical/SetWriteDistributionAndOrdering.scala > It's odd to me that it would be catalyst/plans.logical/SetWriteDistributionAndOrdering.scala instead of with a slash in the file path name (instead using plans.logical for just part of the path). @kbendick: That shows only in the PR. but when merged, github / codebase can understand that it is a sub package. See final results here: https://github.com/apache/iceberg/tree/master/spark/v3.2/spark/src/main/java/org/apache/spark/sql/catalyst/plans/logical -- 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]
