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

[STORM-3702] Change thrift exception classes to contain getMessage() method #3337

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

[STORM-3702] Change thrift exception classes to contain getMessage() method #3337

wants to merge 11 commits into from

Commits on Dec 6, 2023

  1. [STORM-3702] Separate exception classes, rename msg to message and se… 

    …t to generate camel case methods.
    @rzo1
    Bipin Prasad authored and rzo1 committed Dec 6, 2023
    Configuration menu
    Copy the full SHA
    6b07398 View commit details
    Browse the repository at this point in the history

  2. [STORM-3702] Thrift generated java and python code.

    @rzo1
    Bipin Prasad authored and rzo1 committed Dec 6, 2023
    Configuration menu
    Copy the full SHA
    72bef0a View commit details
    Browse the repository at this point in the history

  3. [STORM-3702] Remove getMessage() method from Wrapped exception classes.

    @rzo1
    Bipin Prasad authored and rzo1 committed Dec 6, 2023
    Configuration menu
    Copy the full SHA
    e25e8c8 View commit details
    Browse the repository at this point in the history

  4. [STORM-3702] Replace calls to TException.get_msg() to getMessage().

    @rzo1
    Bipin Prasad authored and rzo1 committed Dec 6, 2023
    Configuration menu
    Copy the full SHA
    20594d1 View commit details
    Browse the repository at this point in the history

  5. [STORM-3702] Change clojure test to call TException.getMessage()

    @rzo1
    Bipin Prasad authored and rzo1 committed Dec 6, 2023
    Configuration menu
    Copy the full SHA
    a1b5b37 View commit details
    Browse the repository at this point in the history

  6. [STORM-3702] DRPCExecutionException needs wrapping still.

    @rzo1
    Bipin Prasad authored and rzo1 committed Dec 6, 2023
    Configuration menu
    Copy the full SHA
    87cff88 View commit details
    Browse the repository at this point in the history

  7. [STORM-3702] Deprecate Wrapped exceptions whose base TException has b… 

    …een fixed to generate getMessage() method.
    @rzo1
    Bipin Prasad authored and rzo1 committed Dec 6, 2023
    Configuration menu
    Copy the full SHA
    bad51f3 View commit details
    Browse the repository at this point in the history

  8. [STORM-3702] Regen using thrift 0.13.

    @rzo1
    Bipin Prasad authored and rzo1 committed Dec 6, 2023
    Configuration menu
    Copy the full SHA
    4ec5ba2 View commit details
    Browse the repository at this point in the history

  9. [STORM-3702] Regen only java exceptions in camel case.

    @rzo1
    Bipin Prasad authored and rzo1 committed Dec 6, 2023
    Configuration menu
    Copy the full SHA
    b9cd4b7 View commit details
    Browse the repository at this point in the history

  10. [STORM-3702] Regen using thrift 0.18.1

    @rzo1
    rzo1 committed Dec 6, 2023
    Configuration menu
    Copy the full SHA
    fca9114 View commit details
    Browse the repository at this point in the history

Commits on Dec 7, 2023

  1. Merge remote-tracking branch 'origin/master' into storm3702 

    # Conflicts:
#	storm-client/src/jvm/org/apache/storm/generated/AlreadyAliveException.java
#	storm-client/src/jvm/org/apache/storm/generated/AuthorizationException.java
#	storm-client/src/jvm/org/apache/storm/generated/HBAuthorizationException.java
#	storm-client/src/jvm/org/apache/storm/generated/HBExecutionException.java
#	storm-client/src/jvm/org/apache/storm/generated/IllegalStateException.java
#	storm-client/src/jvm/org/apache/storm/generated/InvalidTopologyException.java
#	storm-client/src/jvm/org/apache/storm/generated/KeyAlreadyExistsException.java
#	storm-client/src/jvm/org/apache/storm/generated/KeyNotFoundException.java
#	storm-client/src/jvm/org/apache/storm/generated/NotAliveException.java
#	storm-client/src/py/storm/ttypes.py
    @rzo1
    rzo1 committed Dec 7, 2023
    Configuration menu
    Copy the full SHA
    ae808bc View commit details
    Browse the repository at this point in the history