Details

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

      Description

      As an end user I should be able to interact with drillbits using a REST API.
      This API will provide RESTful resources for query submission, status polling, data source configuration and result pagination.
      Link to the design doc -
      https://docs.google.com/document/d/1RF0v05l0lWy1OwSXxHuboqJNLRkS-YzshrEVmUSby5I/edit?usp=sharing

      Task 1 - Embed jetty into drill bit.
      – First task is to embed jetty. The version of jetty will be the latest i.e version 9.x
      – By default it will run on port 50000 (no particular reason why this port...just simple to remember)
      – Will have just one jetty connector assembled which is the HTTP connector.
      – At the end of this task you'll be able to run the drill bit and view the contents of an index.json (which for now perhaps could only spit out the version of Drill)

      Task 2 - Integrate Jersey to provide a platform for RESTful interactions

      Task 3 - Provide an API to submit a new query.

      Remaining tasks - TBD

      1. DRILL-77.patch
        62 kB
        Sudheesh Katkam

        Issue Links

          Activity

          Hide
          Michael Hausenblas added a comment -

          See also DRILL-58 for a Web UI that uses this REST API.

          Show
          Michael Hausenblas added a comment - See also DRILL-58 for a Web UI that uses this REST API.
          Hide
          Michael Hausenblas added a comment -

          I think the round of feedback we've received concerning the v1 API are very valuable, thanks everyone so far. Hari, we might want to set ourselves a deadline when we want to have a stable design (that is, when we consider the v1 API design as done) and move over to implementation. Any ideas?

          Show
          Michael Hausenblas added a comment - I think the round of feedback we've received concerning the v1 API are very valuable, thanks everyone so far. Hari, we might want to set ourselves a deadline when we want to have a stable design (that is, when we consider the v1 API design as done) and move over to implementation. Any ideas?
          Hide
          Jacques Nadeau added a comment -

          For task 1, two quick requests:
          1. please use the ExecConstants and drill.conf files to hold the port number
          2. please add an additional configuration option to disable/enable whether or not the jetty server is activated (using the same method as #1).

          Show
          Jacques Nadeau added a comment - For task 1, two quick requests: 1. please use the ExecConstants and drill.conf files to hold the port number 2. please add an additional configuration option to disable/enable whether or not the jetty server is activated (using the same method as #1).
          Hide
          Sudheesh Katkam added a comment - - edited

          Basic REST API implemented.

          Here's the link to the API: docs.google.com/a/maprtech.com/document/d/1oE-vGsl2K3jXed1srK_6yKm6bYANsW2MJVLjlIBkxZY/edit?usp=sharing

          Show
          Sudheesh Katkam added a comment - - edited Basic REST API implemented. Here's the link to the API: docs.google.com/a/maprtech.com/document/d/1oE-vGsl2K3jXed1srK_6yKm6bYANsW2MJVLjlIBkxZY/edit?usp=sharing
          Hide
          Sudheesh Katkam added a comment -

          Updated the patch to include DRILL-1079 and submitted the patch to review board: https://reviews.apache.org/r/23217/

          Show
          Sudheesh Katkam added a comment - Updated the patch to include DRILL-1079 and submitted the patch to review board: https://reviews.apache.org/r/23217/
          Show
          Sudheesh Katkam added a comment - Here's the updated doc link: https://docs.google.com/document/d/1mRsuWk4Dpt6ts-jQ6ke3bB30PIwanRiCPfGxRwZEQME/edit?usp=sharing
          Hide
          Sudheesh Katkam added a comment -

          Updated the patch to include error messages and logs (updated review board).

          Show
          Sudheesh Katkam added a comment - Updated the patch to include error messages and logs (updated review board).
          Hide
          Sudheesh Katkam added a comment -

          Removed parser from this patch.

          Show
          Sudheesh Katkam added a comment - Removed parser from this patch.
          Hide
          Sudheesh Katkam added a comment -

          Updated patch to include graph viz and rebased on master to resolve conflicts (699851b8d7).

          Show
          Sudheesh Katkam added a comment - Updated patch to include graph viz and rebased on master to resolve conflicts (699851b8d7).
          Hide
          Jacques Nadeau added a comment -

          fixed by e5c2da0 or earlier

          Show
          Jacques Nadeau added a comment - fixed by e5c2da0 or earlier

            People

            • Assignee:
              Jacques Nadeau
              Reporter:
              Srihari Srinivasan
            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development