Skip to content

Commit

Permalink
Create system tables for JDBC metadata
Browse files Browse the repository at this point in the history
This fixes a number of issues with JDBC metadata.
  • Loading branch information
electrum committed Sep 20, 2015
1 parent 3302c9a commit 615d8bd
Show file tree
Hide file tree
Showing 19 changed files with 1,356 additions and 242 deletions.
1 change: 1 addition & 0 deletions presto-docs/src/main/sphinx/release/release-0.119.rst
Expand Up @@ -9,6 +9,7 @@ General Changes
* Add :func:`geometric_mean` function.
* Fix restoring interrupt status in ``StatementClient``.
* Support getting server version in JDBC driver.
* Improve correctness and compliance of JDBC ``DatabaseMetaData``.
* Execute views with the permissions of the view owner.
* Add owner to view metadata.

Expand Down

Large diffs are not rendered by default.

227 changes: 162 additions & 65 deletions presto-jdbc/src/test/java/com/facebook/presto/jdbc/TestDriver.java

Large diffs are not rendered by default.

Expand Up @@ -14,6 +14,18 @@
package com.facebook.presto.connector.system;

import com.facebook.presto.connector.ConnectorManager;
import com.facebook.presto.connector.system.jdbc.AttributeJdbcTable;
import com.facebook.presto.connector.system.jdbc.CatalogJdbcTable;
import com.facebook.presto.connector.system.jdbc.ColumnJdbcTable;
import com.facebook.presto.connector.system.jdbc.ProcedureColumnJdbcTable;
import com.facebook.presto.connector.system.jdbc.ProcedureJdbcTable;
import com.facebook.presto.connector.system.jdbc.PseudoColumnJdbcTable;
import com.facebook.presto.connector.system.jdbc.SchemaJdbcTable;
import com.facebook.presto.connector.system.jdbc.SuperTableJdbcTable;
import com.facebook.presto.connector.system.jdbc.SuperTypeJdbcTable;
import com.facebook.presto.connector.system.jdbc.TableJdbcTable;
import com.facebook.presto.connector.system.jdbc.TableTypeJdbcTable;
import com.facebook.presto.connector.system.jdbc.UdtJdbcTable;
import com.facebook.presto.spi.SystemTable;
import com.google.inject.Binder;
import com.google.inject.Module;
Expand All @@ -35,6 +47,19 @@ public void configure(Binder binder)
globalTableBinder.addBinding().to(CatalogSystemTable.class).in(Scopes.SINGLETON);
globalTableBinder.addBinding().to(TablePropertiesSystemTable.class).in(Scopes.SINGLETON);

globalTableBinder.addBinding().to(AttributeJdbcTable.class).in(Scopes.SINGLETON);
globalTableBinder.addBinding().to(CatalogJdbcTable.class).in(Scopes.SINGLETON);
globalTableBinder.addBinding().to(ColumnJdbcTable.class).in(Scopes.SINGLETON);
globalTableBinder.addBinding().to(ProcedureColumnJdbcTable.class).in(Scopes.SINGLETON);
globalTableBinder.addBinding().to(ProcedureJdbcTable.class).in(Scopes.SINGLETON);
globalTableBinder.addBinding().to(PseudoColumnJdbcTable.class).in(Scopes.SINGLETON);
globalTableBinder.addBinding().to(SchemaJdbcTable.class).in(Scopes.SINGLETON);
globalTableBinder.addBinding().to(SuperTableJdbcTable.class).in(Scopes.SINGLETON);
globalTableBinder.addBinding().to(SuperTypeJdbcTable.class).in(Scopes.SINGLETON);
globalTableBinder.addBinding().to(TableJdbcTable.class).in(Scopes.SINGLETON);
globalTableBinder.addBinding().to(TableTypeJdbcTable.class).in(Scopes.SINGLETON);
globalTableBinder.addBinding().to(UdtJdbcTable.class).in(Scopes.SINGLETON);

binder.bind(SystemConnector.class).in(Scopes.SINGLETON);
binder.bind(SystemTablesRegistrar.class).asEagerSingleton();
}
Expand Down
@@ -0,0 +1,67 @@
/*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package com.facebook.presto.connector.system.jdbc;

import com.facebook.presto.spi.ConnectorSession;
import com.facebook.presto.spi.ConnectorTableMetadata;
import com.facebook.presto.spi.InMemoryRecordSet;
import com.facebook.presto.spi.RecordCursor;
import com.facebook.presto.spi.SchemaTableName;
import com.facebook.presto.spi.TupleDomain;

import static com.facebook.presto.metadata.MetadataUtil.TableMetadataBuilder.tableMetadataBuilder;
import static com.facebook.presto.spi.type.BigintType.BIGINT;
import static com.facebook.presto.spi.type.VarcharType.VARCHAR;

public class AttributeJdbcTable
extends JdbcTable
{
public static final SchemaTableName NAME = new SchemaTableName("jdbc", "attributes");

public static final ConnectorTableMetadata METADATA = tableMetadataBuilder(NAME)
.column("type_cat", VARCHAR)
.column("type_schem", VARCHAR)
.column("type_name", VARCHAR)
.column("attr_name", VARCHAR)
.column("data_type", BIGINT)
.column("attr_type_name", VARCHAR)
.column("attr_size", BIGINT)
.column("decimal_digits", BIGINT)
.column("num_prec_radix", BIGINT)
.column("nullable", BIGINT)
.column("remarks", VARCHAR)
.column("attr_def", VARCHAR)
.column("sql_data_type", BIGINT)
.column("sql_datetime_sub", BIGINT)
.column("char_octet_length", BIGINT)
.column("ordinal_position", BIGINT)
.column("is_nullable", VARCHAR)
.column("scope_catalog", VARCHAR)
.column("scope_schema", VARCHAR)
.column("scope_table", VARCHAR)
.column("source_data_type", BIGINT)
.build();

@Override
public ConnectorTableMetadata getTableMetadata()
{
return METADATA;
}

@Override
public RecordCursor cursor(ConnectorSession session, TupleDomain<Integer> constraint)
{
return InMemoryRecordSet.builder(METADATA).build().cursor();
}
}
@@ -0,0 +1,63 @@
/*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package com.facebook.presto.connector.system.jdbc;

import com.facebook.presto.metadata.Metadata;
import com.facebook.presto.spi.ConnectorSession;
import com.facebook.presto.spi.ConnectorTableMetadata;
import com.facebook.presto.spi.InMemoryRecordSet;
import com.facebook.presto.spi.InMemoryRecordSet.Builder;
import com.facebook.presto.spi.RecordCursor;
import com.facebook.presto.spi.SchemaTableName;
import com.facebook.presto.spi.TupleDomain;

import javax.inject.Inject;

import static com.facebook.presto.metadata.MetadataUtil.TableMetadataBuilder.tableMetadataBuilder;
import static com.facebook.presto.spi.type.VarcharType.VARCHAR;
import static java.util.Objects.requireNonNull;

public class CatalogJdbcTable
extends JdbcTable
{
public static final SchemaTableName NAME = new SchemaTableName("jdbc", "catalogs");

public static final ConnectorTableMetadata METADATA = tableMetadataBuilder(NAME)
.column("table_cat", VARCHAR)
.build();

private final Metadata metadata;

@Inject
public CatalogJdbcTable(Metadata metadata)
{
this.metadata = requireNonNull(metadata);
}

@Override
public ConnectorTableMetadata getTableMetadata()
{
return METADATA;
}

@Override
public RecordCursor cursor(ConnectorSession session, TupleDomain<Integer> constraint)
{
Builder table = InMemoryRecordSet.builder(METADATA);
for (String name : metadata.getCatalogNames().keySet()) {
table.addRow(name);
}
return table.build().cursor();
}
}
@@ -0,0 +1,200 @@
/*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
* http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License.
*/
package com.facebook.presto.connector.system.jdbc;

import com.facebook.presto.Session;
import com.facebook.presto.metadata.Metadata;
import com.facebook.presto.metadata.QualifiedTableName;
import com.facebook.presto.metadata.QualifiedTablePrefix;
import com.facebook.presto.spi.ColumnMetadata;
import com.facebook.presto.spi.ConnectorSession;
import com.facebook.presto.spi.ConnectorTableMetadata;
import com.facebook.presto.spi.InMemoryRecordSet;
import com.facebook.presto.spi.InMemoryRecordSet.Builder;
import com.facebook.presto.spi.RecordCursor;
import com.facebook.presto.spi.SchemaTableName;
import com.facebook.presto.spi.TupleDomain;
import com.facebook.presto.spi.type.Type;
import com.facebook.presto.type.ArrayType;

import javax.inject.Inject;

import java.sql.DatabaseMetaData;
import java.sql.Types;
import java.util.List;
import java.util.Map.Entry;
import java.util.Optional;

import static com.facebook.presto.connector.system.jdbc.FilterUtil.filter;
import static com.facebook.presto.connector.system.jdbc.FilterUtil.stringFilter;
import static com.facebook.presto.connector.system.jdbc.FilterUtil.toSession;
import static com.facebook.presto.metadata.MetadataUtil.TableMetadataBuilder.tableMetadataBuilder;
import static com.facebook.presto.spi.type.BigintType.BIGINT;
import static com.facebook.presto.spi.type.BooleanType.BOOLEAN;
import static com.facebook.presto.spi.type.DateType.DATE;
import static com.facebook.presto.spi.type.DoubleType.DOUBLE;
import static com.facebook.presto.spi.type.TimeType.TIME;
import static com.facebook.presto.spi.type.TimeWithTimeZoneType.TIME_WITH_TIME_ZONE;
import static com.facebook.presto.spi.type.TimestampType.TIMESTAMP;
import static com.facebook.presto.spi.type.TimestampWithTimeZoneType.TIMESTAMP_WITH_TIME_ZONE;
import static com.facebook.presto.spi.type.VarbinaryType.VARBINARY;
import static com.facebook.presto.spi.type.VarcharType.VARCHAR;
import static java.util.Objects.requireNonNull;

public class ColumnJdbcTable
extends JdbcTable
{
public static final SchemaTableName NAME = new SchemaTableName("jdbc", "columns");

public static final ConnectorTableMetadata METADATA = tableMetadataBuilder(NAME)
.column("table_cat", VARCHAR)
.column("table_schem", VARCHAR)
.column("table_name", VARCHAR)
.column("column_name", VARCHAR)
.column("data_type", BIGINT)
.column("type_name", VARCHAR)
.column("column_size", BIGINT)
.column("buffer_length", BIGINT)
.column("decimal_digits", BIGINT)
.column("num_prec_radix", BIGINT)
.column("nullable", BIGINT)
.column("remarks", VARCHAR)
.column("column_def", VARCHAR)
.column("sql_data_type", BIGINT)
.column("sql_datetime_sub", BIGINT)
.column("char_octet_length", BIGINT)
.column("ordinal_position", BIGINT)
.column("is_nullable", VARCHAR)
.column("scope_catalog", VARCHAR)
.column("scope_schema", VARCHAR)
.column("scope_table", VARCHAR)
.column("source_data_type", BIGINT)
.column("is_autoincrement", VARCHAR)
.column("is_generatedcolumn", VARCHAR)
.build();

private final Metadata metadata;

@Inject
public ColumnJdbcTable(Metadata metadata)
{
this.metadata = requireNonNull(metadata);
}

@Override
public ConnectorTableMetadata getTableMetadata()
{
return METADATA;
}

@Override
public RecordCursor cursor(ConnectorSession connectorSession, TupleDomain<Integer> constraint)
{
Session session = toSession(connectorSession);
Optional<String> catalogFilter = stringFilter(constraint, 0);
Optional<String> schemaFilter = stringFilter(constraint, 1);
Optional<String> tableFilter = stringFilter(constraint, 2);

Builder table = InMemoryRecordSet.builder(METADATA);
for (String catalog : filter(metadata.getCatalogNames().keySet(), catalogFilter)) {
QualifiedTablePrefix prefix = FilterUtil.tablePrefix(catalog, schemaFilter, tableFilter);
for (Entry<QualifiedTableName, List<ColumnMetadata>> entry : metadata.listTableColumns(session, prefix).entrySet()) {
addColumnRows(table, entry.getKey(), entry.getValue());
}
}
return table.build().cursor();
}

private static void addColumnRows(Builder builder, QualifiedTableName tableName, List<ColumnMetadata> columns)
{
int ordinalPosition = 1;
for (ColumnMetadata column : columns) {
if (column.isHidden()) {
continue;
}
builder.addRow(
tableName.getCatalogName(),
tableName.getSchemaName(),
tableName.getTableName(),
column.getName(),
jdbcDataType(column.getType()),
column.getType().getDisplayName(),
0,
0,
decimalDigits(column.getType()),
10,
DatabaseMetaData.columnNullableUnknown,
column.getComment(),
null,
null,
null,
0,
ordinalPosition,
"",
null,
null,
null,
null,
null,
null);
ordinalPosition++;
}
}

private static int jdbcDataType(Type type)
{
if (type.equals(BOOLEAN)) {
return Types.BOOLEAN;
}
if (type.equals(BIGINT)) {
return Types.BIGINT;
}
if (type.equals(DOUBLE)) {
return Types.DOUBLE;
}
if (type.equals(VARCHAR)) {
return Types.LONGNVARCHAR;
}
if (type.equals(VARBINARY)) {
return Types.LONGVARBINARY;
}
if (type.equals(TIME)) {
return Types.TIME;
}
if (type.equals(TIME_WITH_TIME_ZONE)) {
return Types.TIME_WITH_TIMEZONE;
}
if (type.equals(TIMESTAMP)) {
return Types.TIMESTAMP;
}
if (type.equals(TIMESTAMP_WITH_TIME_ZONE)) {
return Types.TIMESTAMP_WITH_TIMEZONE;
}
if (type.equals(DATE)) {
return Types.DATE;
}
if (type instanceof ArrayType) {
return Types.ARRAY;
}
return Types.JAVA_OBJECT;
}

private static Integer decimalDigits(Type type)
{
if (type.equals(BIGINT)) {
return 19;
}
return null;
}
}

0 comments on commit 615d8bd

Please sign in to comment.