Skip to content

[WIP] Doc 81 teradata sql connector#857

Open
andrea-unstructured wants to merge 5 commits intomainfrom
DOC-81-teradata-sql-connector
Open

[WIP] Doc 81 teradata sql connector#857
andrea-unstructured wants to merge 5 commits intomainfrom
DOC-81-teradata-sql-connector

Conversation

@andrea-unstructured
Copy link
Collaborator

Updates to the teradata sql connector to include auto-created table name information.

</Warning>

Here is an example table schema that is compatible with Unstructured. It includes all of the required and recommended columns, as
If you specify a table name and it does not exist, Unstructured creates it with the standard schema. If a table name is not specified or already exists, Unstructured creates a table called `<unstructuredautocreated>`.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If a table name already exists, why would Unstructured then create a table called <unstructuredautocreated>? Why wouldn't Unstructured just use the existing table?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I understood it to mean if there's a table that's different with the same name (not one you created already for this same process) then it would create a new one. Will confirm.


Here is an example table schema that is compatible with Unstructured. It includes all of the required and recommended columns, as
If you specify a table name and it does not exist, Unstructured creates it with the standard schema. If a table name is not specified or already exists, Unstructured creates a table called `<unstructuredautocreated>`.
If you leave the table name blank, you must check the **Metadata as JSON** option in the UI or set `metadata_as_json` to **true** in the API.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What happens in this case if you forget to check this option?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will ask, I think you get an error?

PRIMARY INDEX ("id");
```

Following is a legacy example table schema that is compatible with Unstructured. It includes all of the required and recommended columns, as
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When would I want to use this legacy table schema instead of the standard schema?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll check with Potter. He said to leave it in, but you're right, we might need to explain better. Above it says you CAN use the columns if you need them.

@@ -0,0 +1,7 @@
{
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this file intended to be checked in to this PR?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think it's an artifact that somehow got mixed in, will delete.

Copy link
Collaborator Author

@andrea-unstructured andrea-unstructured left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Addressed questions from Paul.

PRIMARY INDEX ("id");
```

The following legacy example table schema is used if you leave **Metadata as JSON** blank. It includes all of the required and recommended columns, as
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It it safe to assume that this is also the behavior if you set metadata_as_json to false or don't specify it altogether?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made a couple small updates.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants