trafodion-codereview mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ryzuo <...@git.apache.org>
Subject [GitHub] incubator-trafodion pull request #854: [TRAFODION - 2345] Support LOG() func...
Date Mon, 28 Nov 2016 10:28:06 GMT
GitHub user ryzuo opened a pull request:

    https://github.com/apache/incubator-trafodion/pull/854

    [TRAFODION - 2345] Support LOG() function of any base

    Like Oracle does, the function LOG() function supports 2 parameter of call, in this way,
the 1st parameter is the radix, and the second one is power, the function returns the logarithm.
    This change implement this, and still, the user can call it with only one parameter (form
LOG(X)), in which case the parameter X will be the power, and the function just returns the
natural logarithm of X. This is also compatible with MySQL.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/ryzuo/incubator-trafodion jira2345

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/incubator-trafodion/pull/854.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #854
    
----
commit 1f9071ae82ee63e6f16bddd091d3fcce1057583f
Author: ryzuo <joshuamurdoc@gmail.com>
Date:   2016-11-25T18:30:26Z

    Implement LOG() function for any base

commit e4681d139f1b3e0ffd7784d9930e9fc0d72bbcbb
Author: ryzuo <joshuamurdoc@gmail.com>
Date:   2016-11-28T08:39:45Z

    Add tests for builtin function LOG() of any base

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---

Mime
View raw message