-
Notifications
You must be signed in to change notification settings - Fork 38
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
Experimental DB2 support #274
Conversation
Long propsTimeZoneOffsetHours = HOURS.convert( | ||
TimeZone.getTimeZone(property(env, "timezone")).getOffset(currentTimeMillis()), MILLISECONDS); | ||
if (!Objects.equals(dbTimeZoneOffsetHours, propsTimeZoneOffsetHours)) { | ||
throw new RuntimeException("Database has unexpected time zone - hour offset DB2: " + dbTimeZoneOffsetHours + |
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.
add expected offset to exception message
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.
Added
|
||
-- Archive tables | ||
-- - no default values | ||
-- - no default values |
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.
duplicate line
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.
Removed
constraint nflow_workflow_uniq unique (type, external_id, executor_group) | ||
); | ||
|
||
-- create or replace function update_modified() returns trigger language plpgsql as ' |
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.
This commented stuff is Oracle related? Could be removed?
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.
It was just missing :). Integration tests didn't cover checking that modified-column auto-updates. Now they include at least one check.
|
||
/** | ||
* Configuration for DB2 database. Note: tested only using DB2 Express-C (Docker: ibmcom/db2express-c). | ||
* DB2 database must be configured to UTC time zone, otherwise nFlow will fail to function correctly. |
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.
remove line about UTC
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.
Done
No description provided.