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.

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

Add the getName() method in order to obtain the applied label to the Trident stream. #2792

Closed
wants to merge 138 commits into from

Conversation

alex-87
Copy link
Contributor

@alex-87 alex-87 commented Aug 4, 2018

Being able to get the applied label of a Trident stream could be useful in case of complex Trident topology design. The getter returns the String corresponding to the _name attribute of the Stream object.

@srdo
Copy link
Contributor

srdo commented Aug 16, 2018

The changes look good to me. Please raise an issue at https://issues.apache.org/jira and rename this PR and the commit message to contain the issue number.

@HeartSaVioR
Copy link
Contributor

@alex-87 Sorry I'm late on party. Could you follow up @srdo guide?

Bipin Prasad and others added 27 commits April 9, 2021 06:06
* [STORM-3895] Add JDK-17 build

* [STORM-3895] Add command line options in maven test for JDK17 or higher - to allow reflection on java.base

* [STORM-3895] Allow reflection in external/storm-casandra when running test under JDK17

* [STORM-3895] Move introspection arguments to individual poms and make specific overrides based on JDK.

* [STORM-3895] Increase memory to avoid jdk8 test failures.
Missing dependency prevents Storm from being built, fixes apache#3940
Synchronize access to map in Multi*Metric; fixes STORM-3944
* [STORM-3884] Update calcite to 1.16.0
…ed (apache#3526)

* [STORM-3900] Update cassandra version.

* [STORM-3900] determine cassandra.version.noPatch string from cassandra.version for use in CVE suppression file url

* [STORM-3900] Add cassandra.version.noPatch property and remove derivation from cassandra.verision

* [STORM-3901] Upgrade Kryo from version 4 to 5 and remove reflectasm-shaded dependency.

* [STORM-3901] Update licenses

* [STORM-3901] Further changes to DEPENDENCY-LICENSES

* [STORM-3901] Update LICENSE-binary

* [STORM-3900] Update cassandra version.

* [STORM-3900] determine cassandra.version.noPatch string from cassandra.version for use in CVE suppression file url

* [STORM-3900] Add cassandra.version.noPatch property and remove derivation from cassandra.verision

* [STORM-3901] Upgrade Kryo from version 4 to 5 and remove reflectasm-shaded dependency.

* [STORM-3901] Update licenses

* [STORM-3901] Further changes to DEPENDENCY-LICENSES

* [STORM-3901] Update licenses

* [STORM-3901] Further changes to DEPENDENCY-LICENSES

* STORM-3901 - "kyro-shade" does not exist in Kyro 5.4.0

* STORM-3884 - Fix license files

* STORM-3901 - Fix license files

---------

Co-authored-by: Richard Zowalla <[email protected]>
[STORM-3925] Allow user resources (in WorkerTopologyContext) to be set by Worker Hooks
)

* STORM-3800 : fix resource leak due to Files.walk Files.list
jnioche and others added 28 commits November 1, 2023 08:21
[STORM-3992]Print stack trace upon Exception when getting heartbeat timeout
* STORM-3995 - commons-text 1.11.0
STORM-3998 - Guava 32.1.3-jre
STORM-3997 - Netty 4.1.100
STORM-3994 - LocalCluster init fails on Windows because of missing librocksdbjni-win64.dll
Upgrade Jedis Library to 5.x


---------

Co-authored-by: Richard Zowalla <[email protected]>
* Avoid NPE in the Values Constructor

* Fix Values Constructor as per review comments

* remove unwanted condition

* Addressing code review comments

* Fix checkstyle

* Fix JUnit 4 -> JUnit 5

---------

Co-authored-by: Kishor Patil <[email protected]>
Co-authored-by: Richard Zowalla <[email protected]>
@rzo1 rzo1 closed this Dec 4, 2023
@rzo1
Copy link
Contributor

rzo1 commented Dec 4, 2023

Fixed with c158dee

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.