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

Add a public API for evaluating linq4j Expression nodes

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • New Feature
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 1.21.0
    • None

    Description

      Currently, the public method evaluate of AbstractNode need a Evaluator object as parameter, but Evaluator class has default access control. This limit the access of evaluate method. So may be we can add an overload evaluate method with default Evaluator, thus, allow to evaluate Expression directly.

      Attachments

        Issue Links

        is duplicated by

        Improvement - An improvement or enhancement to an existing feature or task. CALCITE-3290 evaluate in AbstractNode

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

        Web Link GitHub Pull Request #1387

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

          People

            julianhyde Julian Hyde
            yanlin-Lynn Wang Yanlin
            Votes:
            0 Vote for this issue
            Watchers:
            3 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 - 20m
                20m

                Slack

                  Issue deployment