Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[SPARK-13692][CORE][SQL] Fix trivial Coverity/Checkstyle defects #11530

Closed
wants to merge 6 commits into from
Closed

[SPARK-13692][CORE][SQL] Fix trivial Coverity/Checkstyle defects #11530

wants to merge 6 commits into from

Commits on Mar 8, 2016

  1. Fix potential bugs and Java coding style. 

      * Implement both null and type checking in equals functions.
  * Fix wrong type casting logic in SimpleJavaBean2.equals.
  * Add `implement Cloneable` to `UTF8String` and `SortedIterator`.
  * Remove dereferencing before null check in `AbstractBytesToBytesMapSuite`.
  * Fix coding style: Add '{}' to single `for` statement in mllib examples.
  * Remove unused imports in `ColumnarBatch`.
    @dongjoon-hyun
    dongjoon-hyun committed Mar 8, 2016
    Configuration menu
    Copy the full SHA
    5942135 View commit details
    Browse the repository at this point in the history

  2. Remove unused fields and add close() to prevent resource leak.

    @dongjoon-hyun
    dongjoon-hyun committed Mar 8, 2016
    Configuration menu
    Copy the full SHA
    c911f71 View commit details
    Browse the repository at this point in the history

  3. Use stop() instead of close().

    @dongjoon-hyun
    dongjoon-hyun committed Mar 8, 2016
    Configuration menu
    Copy the full SHA
    5467b90 View commit details
    Browse the repository at this point in the history

  4. Add a fix on JavaKinesisStreamSuit

    @dongjoon-hyun
    dongjoon-hyun committed Mar 8, 2016
    Configuration menu
    Copy the full SHA
    03ac28c View commit details
    Browse the repository at this point in the history

  5. Remove close() for streaming.

    @dongjoon-hyun
    dongjoon-hyun committed Mar 8, 2016
    Configuration menu
    Copy the full SHA
    cdb6077 View commit details
    Browse the repository at this point in the history

  6. Remove streaming example

    @dongjoon-hyun
    dongjoon-hyun committed Mar 8, 2016
    Configuration menu
    Copy the full SHA
    e3c4cf0 View commit details
    Browse the repository at this point in the history