accumulo-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From matthew-dailey <...@git.apache.org>
Subject [GitHub] accumulo pull request: ACCUMULO-2493: Deprecated BinaryFormatter i...
Date Mon, 04 Jan 2016 05:06:32 GMT
GitHub user matthew-dailey opened a pull request:

    https://github.com/apache/accumulo/pull/61

    ACCUMULO-2493: Deprecated BinaryFormatter in favor of DefaultFormatter

    with FormatterConfig.
    
    * New class FormatterConfig handles configuration of Formatter objects
    * FormatterConfig is taken by Formatter.initialize, which broke the
    interface for many classes throughout the non-public API
    * Added DateFormatSupplier to let Formatters use DateFormat in a
    Thread-safe way
    * Removed code from ScanCommand and Shell tied to BinaryFormatter to use
    properly configured DefaultFormatter instead
    * Fixed bug where `scan -f [num] -fm [class]` would ignore Formatter
    class used in `-fm` and be overridden with BinaryFormatter

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

    $ git pull https://github.com/matthew-dailey/accumulo ACCUMULO-2493-1.7-squashed

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

    https://github.com/apache/accumulo/pull/61.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 #61
    
----
commit 8f60b4b1cb87f6bc315479d240e9e17b212a16a9
Author: Matt Dailey <matthew.dailey1@gmail.com>
Date:   2015-12-05T02:50:58Z

    ACCUMULO-2493: Deprecated BinaryFormatter in favor of DefaultFormatter
    with FormatterConfig.
    
    * New class FormatterConfig handles configuration of Formatter objects
    * FormatterConfig is taken by Formatter.initialize, which broke the
    interface for many classes throughout the non-public API
    * Added DateFormatSupplier to let Formatters use DateFormat in a
    Thread-safe way
    * Removed code from ScanCommand and Shell tied to BinaryFormatter to use
    properly configured DefaultFormatter instead
    * Fixed bug where `scan -f [num] -fm [class]` would ignore Formatter
    class used in `-fm` and be overridden with BinaryFormatter

----


---
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