-
Notifications
You must be signed in to change notification settings - Fork 14
Fixed resetting subject/resource cache for new entities. #153
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
Conversation
anubhavi25
left a comment
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.
Some minor changes but besides that looks good 👍 merge after incorporating feedback and squashing commits
| } | ||
|
|
||
| private String timestampValue() { | ||
| private DateTime currentDateUTC() { |
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.
Make static (and remove any usages of this. when calling it)
| return new DateTime().withZone(DateTimeZone.UTC); | ||
| } | ||
|
|
||
| private String timestampUTC() { |
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.
Make static (and remove any usages of this. when calling it)
| } | ||
| } | ||
|
|
||
| private DateTime timestampToDateUTC(final long timestamp) { |
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.
Make static (and remove any usages of this. when calling it)
| return new DateTime(timestamp).withZone(DateTimeZone.UTC); | ||
| } | ||
|
|
||
| private DateTime timestampToDateUTC(final String timestamp) { |
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.
Make static (and remove any usages of this. when calling it)
Enabled attributes and decision caching for IT. Fixed enabling decision caching from environment variable.
No description provided.