Uploaded image for project: 'Couchbase Server'
  1. Couchbase Server
  2. MB-22417

Add weekday part to N1QL datetime functions

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 4.5.1
    • Fix Version/s: 4.6.2, 5.0.0
    • Component/s: query
    • Labels:
      None

      Description

      It would be nice to update all N1QL datetime functions to accept a "weekday" part which is the English name of the day, in addition to number of the day.

        Attachments

          Issue Links

            Activity

            Hide
            Prasad.Varakur Prasad Varakur added a comment -

            thanks Sitaram Vemulapalli. What is the reason for adding new functions, instead of not adding a new part similar to bunch of other parts to use with date_part_str()/date_part_millis()?

            Show
            Prasad.Varakur Prasad Varakur added a comment - thanks Sitaram Vemulapalli . What is the reason for adding new functions, instead of not adding a new part similar to bunch of other parts to use with date_part_str()/date_part_millis()?
            Hide
            gerald Gerald Sangudi (Inactive) added a comment -

            Hi Prasad Varakur, I don't remember the reason we went with a new function, but the functionality is equivalent. You can document the feature as a new function.

            Show
            gerald Gerald Sangudi (Inactive) added a comment - Hi Prasad Varakur , I don't remember the reason we went with a new function, but the functionality is equivalent. You can document the feature as a new function.
            Hide
            gerald Gerald Sangudi (Inactive) added a comment -

            Note that WEEKDAY() returns strings, whereas DATE_PART() returns number.

            Show
            gerald Gerald Sangudi (Inactive) added a comment - Note that WEEKDAY() returns strings, whereas DATE_PART() returns number.
            Hide
            build-team Couchbase Build Team added a comment -

            Build 4.6.2-3889 contains testrunner commit a5f12d9582944b9e9335189da02bf3408e04739a with commit message:
            CBQE-4072: Add test cases for MB-22417 and MB-22577
            https://github.com/couchbase/testrunner/commit/a5f12d9582944b9e9335189da02bf3408e04739a

            Show
            build-team Couchbase Build Team added a comment - Build 4.6.2-3889 contains testrunner commit a5f12d9582944b9e9335189da02bf3408e04739a with commit message: CBQE-4072: Add test cases for MB-22417 and MB-22577 https://github.com/couchbase/testrunner/commit/a5f12d9582944b9e9335189da02bf3408e04739a
            Hide
            build-team Couchbase Build Team added a comment -

            Build 5.0.0-2670 contains testrunner commit bb3d446b1637d55787a964823dbcee34ff0905a5 with commit message:
            CBQE-4072: Add test cases for MB-22417 and MB-22577
            https://github.com/couchbase/testrunner/commit/bb3d446b1637d55787a964823dbcee34ff0905a5

            Show
            build-team Couchbase Build Team added a comment - Build 5.0.0-2670 contains testrunner commit bb3d446b1637d55787a964823dbcee34ff0905a5 with commit message: CBQE-4072: Add test cases for MB-22417 and MB-22577 https://github.com/couchbase/testrunner/commit/bb3d446b1637d55787a964823dbcee34ff0905a5

              People

              • Assignee:
                prasanna.gholap Prasanna Gholap
                Reporter:
                arungupta Arun Gupta (Inactive)
              • Votes:
                0 Vote for this issue
                Watchers:
                8 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: