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

NuoDB Producer #71

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

NuoDB Producer #71

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
d2c555a
Ignore symbolic links that make tests run first when working on them.
jazd Sep 27, 2015
3f20ba7
Use DB2.pm and 64xml-to-mysql.t as initial templates
jazd Sep 27, 2015
77a5e62
Passes test
jazd Sep 27, 2015
76bfba1
cleanup
jazd Sep 27, 2015
26105ed
Fixed index and constraint creatioin. Added reserved words and NOW.
jazd Sep 28, 2015
3b2adbf
whitespace
jazd Sep 28, 2015
f4880ea
added a couple unit tests
jazd Oct 4, 2015
238bbf3
actual nuodb reserved words
jazd Oct 4, 2015
7a05c60
Merge branch latest upstream 'master' into nuodb
jazd Nov 19, 2015
76bc9cf
Merge branch 'master' into nuodb
jazd Sep 23, 2017
2b267cc
NOW() stays NOW()
jazd Dec 16, 2017
2ddbd16
NuoDB does not support WITHOUT TIME ZONE types
jazd Dec 16, 2017
2767a07
Merge branch 'master' into nuodb
jazd Apr 7, 2019
b863fe0
Merge branch 'master' into nuodb
jazd Nov 5, 2020
aaa1ae4
Merge branch 'master' into nuodb
jazd Oct 12, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions lib/SQL/Translator/Producer/NuoDB.pm
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -192,6 +192,9 @@ sub create_field
my $data_type = uc($dt_translate{lc($field->data_type)} || $field->data_type);
my $size = $field->size();

# NuoDB does not support WITHOUT TIME ZONE types
$data_type =~ s/ WITHOUT TIME ZONE//i;

my $field_def = "$field_name $data_type";
$field_def .= $field->is_auto_increment ?
' GENERATED BY DEFAULT AS IDENTITY' : '';
Expand Down
16 changes: 16 additions & 0 deletions t/65xml-to-nuodb-types.t
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -111,6 +111,22 @@ use SQL::Translator::Producer::NuoDB;
is_deeply(@result[0], $expected, 'NOW() stays NOW()');
}

# without time zone
{
my $table = SQL::Translator::Schema::Table->new(
name => 'foo_table',
);
$table->add_field(
name => 'c',
data_type => 'timestamp WITHOUT TIME ZONE',
is_nullable => 1,
);
my $expected = "CREATE TABLE foo_table (\n c TIMESTAMP\n);";
my @result = SQL::Translator::Producer::NuoDB::create_table($table);
is_deeply(@result[0], $expected, 'Ignore WITHOUT TIME ZONE');
}


# reserved word field
{
my $table = SQL::Translator::Schema::Table->new(
Expand Down