Skip to content

Commit 342b860

Browse files
#90: Removed JsonHelper class. (#146)
1 parent 4cdc454 commit 342b860

File tree

6 files changed

+9
-118
lines changed

6 files changed

+9
-118
lines changed

doc/changes/changelog.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
11
# Changes
22

33
* [10.1.0](changes-10.1.0.md)
4-
* [10.2.0](changes-10.2.0.md)
4+
* [11.0.0](changes-11.0.0.md)
Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
1-
# Virtual Schema Common Java 10.2.0, released 2020.07.28
1+
# Virtual Schema Common Java 11.0.0, released 2020.07.28
22

33
## Features / Enhancements
44

5+
* #90: Removed `JsonHelper` class.
56
* #126: Added capabilities for new scalar functions HASHTYPE_*.
67
* #144: Updated .travis.yml, added upload_release_assets.yml for Release Robot.
78

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
<modelVersion>4.0.0</modelVersion>
55
<groupId>com.exasol</groupId>
66
<artifactId>virtual-schema-common-java</artifactId>
7-
<version>10.2.0</version>
7+
<version>11.0.0</version>
88
<name>Common module of Exasol Virtual Schemas Adapters</name>
99
<description>This is one of the modules of Virtual Schemas Adapters. The libraries provided by this project are the
1010
foundation of the adapter development, i.e. adapters must be implemented on top of them.

src/main/java/com/exasol/adapter/metadata/converter/SchemaMetadataJsonConverter.java

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -15,14 +15,13 @@ public final class SchemaMetadataJsonConverter {
1515
private static final SchemaMetadataJsonConverter instance = new SchemaMetadataJsonConverter();
1616
private static final String ADAPTER_NOTES_KEY = "adapterNotes";
1717
private static final String TABLES_KEY = "tables";
18-
private static final String TABLE_TYPE_KEY = "type";
18+
private static final String TYPE_KEY = "type";
1919
private static final String TABLE_NAME_KEY = "name";
2020
private static final String DATA_TYPE_KEY = "dataType";
2121
private static final String NULLABLE_KEY = "isNullable";
2222
private static final String COMMENT_KEY = "comment";
2323
private static final String IDENTITY_KEY = "isIdentity";
2424
private static final String DEFAULT_KEY = "default";
25-
private static final String DATA_TYPEKEY = "type";
2625
private final JsonBuilderFactory factory = Json.createBuilderFactory(Collections.emptyMap());
2726

2827
/**
@@ -45,12 +44,12 @@ private SchemaMetadataJsonConverter() {
4544
*/
4645
public JsonObject convert(final SchemaMetadata schemaMetadata) {
4746
final JsonObjectBuilder root = this.factory.createObjectBuilder();
48-
root.add(TABLES_KEY, converTables(schemaMetadata));
47+
root.add(TABLES_KEY, convertTables(schemaMetadata));
4948
root.add(ADAPTER_NOTES_KEY, schemaMetadata.getAdapterNotes());
5049
return root.build();
5150
}
5251

53-
public JsonArrayBuilder converTables(final SchemaMetadata schemaMetadata) {
52+
private JsonArrayBuilder convertTables(final SchemaMetadata schemaMetadata) {
5453
final JsonArrayBuilder tablesBuilder = this.factory.createArrayBuilder();
5554
for (final TableMetadata table : schemaMetadata.getTables()) {
5655
tablesBuilder.add(convertTableMetadata(table));
@@ -60,7 +59,7 @@ public JsonArrayBuilder converTables(final SchemaMetadata schemaMetadata) {
6059

6160
private JsonObjectBuilder convertTableMetadata(final TableMetadata table) {
6261
final JsonObjectBuilder tableBuilder = this.factory.createObjectBuilder();
63-
tableBuilder.add(TABLE_TYPE_KEY, "table");
62+
tableBuilder.add(TYPE_KEY, "table");
6463
tableBuilder.add(TABLE_NAME_KEY, table.getName());
6564
final JsonArrayBuilder columnsBuilder = Json.createArrayBuilder();
6665
for (final ColumnMetadata column : table.getColumns()) {
@@ -104,7 +103,7 @@ private JsonObjectBuilder convertColumnMetadata(final ColumnMetadata column) {
104103
*/
105104
public JsonObject convertType(final DataType dataType) {
106105
final JsonObjectBuilder typeAsJson = this.factory.createObjectBuilder();
107-
typeAsJson.add(DATA_TYPEKEY, getExasolDataTypeName(dataType.getExaDataType()));
106+
typeAsJson.add(TYPE_KEY, getExasolDataTypeName(dataType.getExaDataType()));
108107
switch (dataType.getExaDataType()) {
109108
case UNSUPPORTED:
110109
throw new IllegalArgumentException("Unsupported data type found trying to serialize schema metadata. "

src/main/java/com/exasol/utils/JsonHelper.java

Lines changed: 0 additions & 48 deletions
This file was deleted.

src/test/java/com/exasol/utils/JsonHelperTest.java

Lines changed: 0 additions & 61 deletions
This file was deleted.

0 commit comments

Comments
 (0)