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

Move core to use Immutables instead of ImmutableBeans

Rank to TopRank to BottomBulk Copy AttachmentsBulk Move AttachmentsVotersWatch issueWatchersConvert to sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    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

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            jnadeau Jacques Nadeau
            jnadeau Jacques Nadeau
            Votes:
            1 Vote for this issue
            Watchers:
            10 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

                Slack

                  Issue deployment