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/01/15 00:58:43 UTC

[GitHub] [beam] shehzaadn-vd commented on pull request #13668: [BEAM-10961] Enabled strict dependency on Spark

shehzaadn-vd commented on pull request #13668:
URL: https://github.com/apache/beam/pull/13668#issuecomment-760574007


   > @sonam-vend shouldn't we enable strict dependencies on each module as we go, instead of waiting for every module to be fixed to start checking any of them?
    
   @ibzib Since #12938 is passing all checks, and all these PRs were split from that one, @sonam-vend and I were thinking we'd have a small-ish number of failures when we enable the switch at the global level. Since eventually we'll be turning on the global switch anyway, it'll help us avoid changes at the module level which we would then have to un do. Wdyt?


----------------------------------------------------------------
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:
users@infra.apache.org