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

Move core to use Immutables instead of ImmutableBeans

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 1.28.0
    • None

    Description

      In the creation of CALCITE-3328, Immutables was discussed as an alternative to a custom implementation. This ticket is to evaluate the impact to the codebase of changing. Ideally, introduction of immutables would both add flexibility and reduce the amount of code associated with these classes.

      Immutables works via annotation processor which means that it is should be relatively seamless to build systems and IDEs.

      The switch would also make it easier to work with these objects types in the context of aot compilation tools like GraalVM.

       

      This initial task covers key classes in the core module. Will open up follow-on tickets for other locations.

      Attachments

        Issue Links

          blocks

          Improvement - An improvement or enhancement to an existing feature or task. CALCITE-4786 Facilitate use of graalvm native-image compilation

          • Major - Major loss of function.
          • Open
          causes

          Bug - A problem which impairs or prevents the functions of the product. CALCITE-5332 Configuring PruneEmptyRules is cumbersome

          • Major - Major loss of function.
          • Closed

          Improvement - An improvement or enhancement to an existing feature or task. CALCITE-6010 RelRule.Config requires too much ceremony for trivial cases

          • Major - Major loss of function.
          • Open
          incorporates

          Task - A task that needs to be done. CALCITE-4839 Remove remnants of ImmutableBeans post 1.28 release

          • Major - Major loss of function.
          • Closed
          links to

          Web Link GitHub Pull Request #2531

          mentioned in

          Page Loading...

          Activity

            People

              jnadeau Jacques Nadeau
              jnadeau Jacques Nadeau
              Votes:
              1 Vote for this issue
              Watchers:
              9 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

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