phoenix-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From 7shurik <>
Subject [GitHub] phoenix pull request: Phoenix 1759
Date Fri, 24 Apr 2015 23:13:08 GMT
GitHub user 7shurik reopened a pull request:

    Phoenix 1759

    This is a fix for There's a set of
unit tests as well as an integration test.
    Main code change is adding a PhoenixArray.toString() implementation and toString on each
primitive array type. There're also a couple of fixes for toStringLiteral not to barf on nulls.
    Please let me know if you see any problems.

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

    $ git pull PHOENIX-1759

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

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

    This closes #68
commit 9b6a4f049f084edb4d3fb03c960eb37b24d120b2
Author: Alex <>
Date:   2015-04-15T19:19:08Z

    JIRA PHOENIX-1759 - Fix implementation of PhoenixArray toString(). Includes both object
and primitive arrays.

commit 0fb879e74a3dc55211046c96f30a054a5aeacb67
Author: Alex <>
Date:   2015-04-15T19:30:32Z

    Add intergration test for array toString()

commit ca6243de97f4bc203278bd129a62e08ffea896f4
Author: Alex <>
Date:   2015-04-15T23:03:07Z

    Integration test for PhoenixArray ResultSet.getString(), also sync PDataType to latest
trunk version.

commit 1be527d737cc1ade2e121202085c42106e2ad29a
Author: Alex <>
Date:   2015-04-16T16:58:22Z

    Fix formatting, add more PVarcharArray tests.

commit fc47b68ee56e99e35cd0b96e836cb633df003993
Author: Alex <>
Date:   2015-04-16T17:55:02Z

    Unify PhoenixArray.toString() to work for all types of arrays, remove other toString()
implementations. Also create toStringLiteral(Object) to use when there's no Formatter.

commit b6de356c0fc49c90a4263e6b36dd848c0fc368b9
Author: Alex <>
Date:   2015-04-16T17:57:23Z

    Fix tabs=>spaces in PhoenixArray.toString()

commit 02a17c57fa18b74c8f97a11eb6446826a1e84b56
Author: Alex <>
Date:   2015-04-17T22:45:15Z

    Merge branch 'master' into PHOENIX-1759

commit 9220dbb4854b9448802cc4361ade5c5c8470d401
Author: Alex <>
Date:   2015-04-24T23:00:23Z

    Fix Apache license header (2=>2.0)


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 or file a JIRA ticket
with INFRA.

View raw message