Uploaded image for project: 'IMPALA'
  1. IMPALA
  2. IMPALA-3973

add position and occurrence to instr()

    Details

      Description

      currently instr() is defined as

      instr(string str, string substr)

      but other systems (Vertica, Oracle, etc) define it as

      INSTR ( string , substring [, position [, occurrence ] ] )

      Request is to add position and occurrence.

      References:

      See also: https://jira.cloudera.com/browse/CDH-42948

        Issue Links

          Activity

          Hide
          scotts_impala_39e4 Scott Sappenfield added a comment -

          Sorry, forgot to specify, the ask is also specifically for the position and occurrence capability/functionality.

          Show
          scotts_impala_39e4 Scott Sappenfield added a comment - Sorry, forgot to specify, the ask is also specifically for the position and occurrence capability/functionality.
          Hide
          scotts_impala_39e4 Scott Sappenfield added a comment -

          Hey Zoltan, does this change in status from open to patch available mean this has already been coded upstream in Impala?

          Show
          scotts_impala_39e4 Scott Sappenfield added a comment - Hey Zoltan, does this change in status from open to patch available mean this has already been coded upstream in Impala?
          Hide
          zi Zoltan Ivanfi added a comment -

          Hey Scott,

          No, the meaning I intended is that there is a patchset uploaded for review at https://gerrit.cloudera.org/4094

          It is possible that I misunderstood the purpose of the "patch available" status, in this case sorry for the confusion.

          Show
          zi Zoltan Ivanfi added a comment - Hey Scott, No, the meaning I intended is that there is a patchset uploaded for review at https://gerrit.cloudera.org/4094 It is possible that I misunderstood the purpose of the "patch available" status, in this case sorry for the confusion.
          Hide
          scotts_impala_39e4 Scott Sappenfield added a comment -

          Hey Zoltan, all good, thanks for the update. No, I think it was probably my misunderstanding, not yours.

          If I had seen the gerrit review, with the link you just pasted, it would have been obvious to me. But I couldn't see it anywhere in this JIRA so I wasn't clear.

          Thanks!

          Show
          scotts_impala_39e4 Scott Sappenfield added a comment - Hey Zoltan, all good, thanks for the update. No, I think it was probably my misunderstanding, not yours. If I had seen the gerrit review, with the link you just pasted, it would have been obvious to me. But I couldn't see it anywhere in this JIRA so I wasn't clear. Thanks!
          Show
          zi Zoltan Ivanfi added a comment - https://gerrit.cloudera.org/4094
          Hide
          grahn Greg Rahn added a comment -

          John Russell - Docs work required.

          Show
          grahn Greg Rahn added a comment - John Russell - Docs work required.
          Hide
          scotts_impala_39e4 Scott Sappenfield added a comment -

          Hey Greg, is this being targeted for 5.10 or some other release as far as you know/can share today?

          Show
          scotts_impala_39e4 Scott Sappenfield added a comment - Hey Greg, is this being targeted for 5.10 or some other release as far as you know/can share today?

            People

            • Assignee:
              jrussell John Russell
              Reporter:
              scotts_impala_39e4 Scott Sappenfield
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development