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
Expose creation and modification date for subscription #1158
Expose creation and modification date for subscription #1158
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SGTM
T t = supplier.read(data); | ||
statDecorator.accept(t, stat); | ||
return Optional.of(t); | ||
} else { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
else
isn't needed :)
@cristaloleg what date format do you expect? |
For me 2nd option is more preferable, 'cause it's mostly dev-oriented console, but let's ping @dankraw @druminski @piotrrzysko @faderskd 😉 |
@cristaloleg I pushed changes in hermes-console. Dates have format of timestamp. You can do code review. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sgtm
* Expose creation and modification date for topic and subscription
* Expose creation and modification date for topic and subscription
#1123
For now I implemented it only for subscription. Please check it out and give me a feedback, then I will continue working.
What format of creation and modification date do you expect? Timestamp (
Long
) or human-readable string ("yyyy-MM-dd'T'HH:mm:ss.SSS'Z'")?