Uploaded image for project: 'Calcite'
  1. Calcite
  2. CALCITE-1461

Hard-coded class name in JaninoRelMetadataProvider breaks shading

          Details

          • Type: Bug
          • Status: Closed
          • Priority: Major
          • Resolution: Fixed
          • Affects Version/s: None
          • Fix Version/s: 1.11.0
          • Component/s: None
          • Labels:
            None

            Description

            In JaninoRelMetadataProvider.load3, the generated class string contains some hard coded Calcite's classes, such as org.apache.calcite.rel.metadata.MetadataDef. After shading Calcite, the MetadataDef can not be found in the shaded classes. And a compile error will be thrown.

            So it's better to use MetadataDef.class.getName() to replace the hard code string.

            I'm appreciate to make a PR if you want.

              Attachments

                Issue Links

                is depended upon by

                Bug - A problem which impairs or prevents the functions of the product. CALCITE-1496 Apache Calcite is not working in Web Application

                • Critical - Crashes, loss of data, severe memory leak.
                • Closed

                  Activity

                    People

                    • Assignee:
                      julianhyde Julian Hyde
                      Reporter:
                      jark Jark Wu
                    • Votes:
                      0 Vote for this issue
                      Watchers:
                      2 Start watching this issue

                      Dates

                      • Created:
                        Updated:
                        Resolved: