GitHub user prateekm opened a pull request:

    https://github.com/apache/samza/pull/455

    Misc. Util cleanup

    Major changes:
    1. Broke up 'Util' class into multiple classes: 'FileUtil', 'HttpUtil', 
'CoordinatorStreamUtil'.
    2. Consolidated some Util classes: MathUtil, ScalaJavaUtil
    3. Removed redundant Util classes: ClassloaderUtil, ScalaToJavaUtil
    4. Renamed some Util classes for consistency: TimerUtils -> TimerUtil.
    5. Inlined some util classes and methods to where they're used: 
LexicographicComparator to RocksDBKeyValueStore, 
defaultSerdeFactoryFromSerdeName to SerdeManager, etc.
    
    Rest of the changes are updates to use the new classes and methods.
    
    Testing: Local build and test works. Tested with a locally deployed Samza 
job.

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

    $ git pull https://github.com/prateekm/samza util-cleanup

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

    https://github.com/apache/samza/pull/455.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 #455
    
----
commit d08168071132e26c021b7f15cda7ab470720b473
Author: Prateek Maheshwari <pmaheshw@...>
Date:   2018-03-22T19:07:10Z

    Util cleanup.

----


---

Reply via email to