Uploaded image for project: 'Beam'
  1. Beam
  2. BEAM-13430 Upgrade Gradle version to 7.3
  3. BEAM-13504

Remove provided/compileOnly deps not intended for external use

Details

    • Sub-task
    • Status: Open
    • P3
    • Resolution: Unresolved
    • None
    • None
    • build-system
    • None

    Description

      There are a lot of compileOnly deps in place to make certain libraries lighter. In many cases, these don't actually reduce the size of the resulting binary, and are a result of an overriding strategy where the types of the implementation are needed in the loading library.

      In the short term, these can all be replaced by `implementation`, with the downside that jars will have extra explicit dependencies, and the upside that users will not experience runtime errors due to missing dependencies. In the longer term, we should restructure these libraries (sql being the worst offender) to use serviceLoaders or other strategies that do not require naming the types of the overriders within the same package that the interface is declared.

      The one exception to this is libraries that we intend to use whatever version the user puts on the classpath at runtime without expressing a version preference, such as hadoop or kafka. We should still consider if this has value though, as users are still free to explicitly override the dependency version we choose, even if we declare an `implementation` dependency on it.

      Attachments

        Issue Links

          Activity

            People

              Unassigned Unassigned
              dpcollins-google Daniel Collins
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

                Created:
                Updated:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 40m
                  40m