You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2021/08/02 14:45:27 UTC

[GitHub] [beam] echauchot commented on pull request #15218: [BEAM-7093] Migrate spark structured streaming runner to spark 3

echauchot commented on pull request #15218:
URL: https://github.com/apache/beam/pull/15218#issuecomment-891085988


   > @ibzib fixed everything except the package-info.java missing. I don't know how it is supposed to work with per-version source overrides. These overrides have the same java packages as the main source so the package-info file should be only in the main source packages otherwise they will be duplicated. Why is checkstyle complaining about not having package-info in source overrides whereas there are in corresponding main java packages?
   
   Found it:
   ```
    <!-- Checkstyle does not correctly detect package files across multiple source directories. -->
     <suppress checks="JavadocPackage" 
   ```


-- 
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: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org