-
Notifications
You must be signed in to change notification settings - Fork 15
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
Create CLOB column type #11
Comments
I was waiting for this to become an issue. The singer spec has no distinction between large and small strings, and certainly not on the arbitrary 4000 char limit in Oracle. The best we can hope for is for the tap to specify a max-length - but that is mostly just for database taps. Two things we can do:
What do you think? |
The first options was the first idea I tried and it worked changing it in my local, but it is true that it reduces flexibility. I think the second one is the most complete but I guess it depends on the amount of work required to complete it. Maybe an approach would be to add the first option to unblock it for the moment and add the second one later? If I'm not missing anything, the first option would be to add something like this in sinks.py:97, right? maxlength = jsonschema_type.get("maxLength", 4000)
if maxlength > 4000:
return cast(
sqlalchemy.types.TypeEngine, sqlalchemy.types.CLOB()
)
else:
return cast(
sqlalchemy.types.TypeEngine, sqlalchemy.types.VARCHAR(maxlength)
) |
Hello! Any thoughts on this? Thank you :) |
Hi @gallejesus, thanks for the reminder. I have created a new branch, I have tested the |
Hi,
I'm trying to create a column for a string that exceeds 4k characters. Sadly, I cannot increase max string size in my target oracle db so the alternative would be to store this field in a CLOB type.
I have been taking a look about the types supported and cannot find a way to create this column.
Thank you!
The text was updated successfully, but these errors were encountered: