Apache Drill
  1. Apache Drill
  2. DRILL-294

simple_plan.json needs to be written correctly

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.0.0-rc1
    • Fix Version/s: None
    • Labels:

      Description

      simple_plan.json is written in old style and is being referred to on the WIKI. Its giving problems to all new drillers. It has to be written correctly.

        Activity

        Timothy Chen made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Timothy Chen made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Yash Sharma added a comment -

        This is fixed as part of commit 3694542.

        Show
        Yash Sharma added a comment - This is fixed as part of commit 3694542.
        Hide
        Yash Sharma added a comment -

        @Kai: There have been few changes in structure of the plan.
        Apart from that the new plan syntax uses 'id' and 'input' elements. The 'ref' element needs a prefix of 'output.' in syntax.
        You can have a look into other plans and drop a note at drill-dev if you stumble at any part.

        Show
        Yash Sharma added a comment - @Kai: There have been few changes in structure of the plan. Apart from that the new plan syntax uses 'id' and 'input' elements. The 'ref' element needs a prefix of 'output.' in syntax. You can have a look into other plans and drop a note at drill-dev if you stumble at any part.
        Hide
        Kai Wei added a comment -

        Thank you Yash, the patch is really useful to new drillers like me. Could you give more information about the change of syntax style?

        Show
        Kai Wei added a comment - Thank you Yash, the patch is really useful to new drillers like me. Could you give more information about the change of syntax style?
        Yash Sharma made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Labels patch
        Hide
        Yash Sharma added a comment -

        Patch submitted

        Show
        Yash Sharma added a comment - Patch submitted
        Yash Sharma made changes -
        Field Original Value New Value
        Attachment DRILL-294.patch [ 12613548 ]
        Hide
        Yash Sharma added a comment -

        Fixed the simple_plan.json logical plan as per new plan style.
        Review Borad: https://reviews.apache.org/r/15474/

        Show
        Yash Sharma added a comment - Fixed the simple_plan.json logical plan as per new plan style. Review Borad: https://reviews.apache.org/r/15474/
        Yash Sharma created issue -

          People

          • Assignee:
            Unassigned
            Reporter:
            Yash Sharma
          • Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development