Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-2859

Return error for "SELECT a FROM T ORDER BY b" for has-chunks query

    XMLWordPrintable

    Details

    • Type: Story
    • Status: To Do
    • Resolution: Unresolved
    • Fix Version/s: None
    • Component/s: Qserv
    • Labels:

      Description

      ORDER BY field has to be in result table => it has to be in select list.
      Return clear error message to user if not.

        Attachments

          Issue Links

          is blocked by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-3109 Add support for accessing schema from QueryContext

          • Done
          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-2860 Add support for ORDER BY expressions

          • To Do

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-2894 Study how to solve "ORDER BY FUNC(field)"

          • To Do
          mentioned in

          Page Loading...

          Page Loading...

            Activity

            Ascending order - Click to sort in descending order
            21 older comments
            Hide
            Permalink
            jammes Fabrice Jammes added a comment - - edited

            Hi Jacek Becla,

            Can we please remove this one from current sprint and postpone it?
            Andy Salnikov, here's a connection pooling library:
            http://www.tildeslash.com/libzdb/#api
            I don't know if this tool is good (please note that it is packaged in Ubuntu) but I think the connection pooling feature could ease our life a lot (Java world even standardized this feature)...

            Please note that libzdb also provide a preparedstatement feature which is fine to prevent SQL-injection (i.e. prevent user to replace a query parameter which a ;dangerous SQL query , but I'm not sure Qserv need this.

            Show
            jammes Fabrice Jammes added a comment - - edited Hi Jacek Becla , Can we please remove this one from current sprint and postpone it? Andy Salnikov , here's a connection pooling library: http://www.tildeslash.com/libzdb/#api I don't know if this tool is good (please note that it is packaged in Ubuntu) but I think the connection pooling feature could ease our life a lot (Java world even standardized this feature)... Please note that libzdb also provide a preparedstatement feature which is fine to prevent SQL-injection (i.e. prevent user to replace a query parameter which a ;dangerous SQL query , but I'm not sure Qserv need this.
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Fritz Mueller what's the status of this old ticket?

            Show
            tjenness Tim Jenness added a comment - Fritz Mueller what's the status of this old ticket?
            Hide
            Permalink
            fritzm Fritz Mueller added a comment -

            Still relevant; low priority

            Show
            fritzm Fritz Mueller added a comment - Still relevant; low priority
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Should we move it back to TODO status?

            Show
            tjenness Tim Jenness added a comment - Should we move it back to TODO status?
            Hide
            Permalink
            fritzm Fritz Mueller added a comment -

            Thanks Tim, hadn't noticed status, just did do.

            Show
            fritzm Fritz Mueller added a comment - Thanks Tim, hadn't noticed status, just did do.

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              jammes Fabrice Jammes
              Watchers:
              Andy Salnikov, Fabrice Jammes, Fritz Mueller, Jacek Becla, Kian-Tat Lim, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated:

                  Jenkins

                  No builds found.