fix(QTDI-2215): Add schema/Entry pojo. #1146
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,88 @@ | ||
| /** | ||
| * Copyright (C) 2006-2025 Talend Inc. - www.talend.com | ||
| * | ||
| * 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 org.talend.sdk.component.server.front.model; | ||
|
|
||
| import java.beans.ConstructorProperties; | ||
| import java.util.LinkedHashMap; | ||
| import java.util.Map; | ||
|
|
||
| import lombok.Data; | ||
|
|
||
| @Data | ||
| public final class Entry { | ||
|
|
||
| private final String name; | ||
|
|
||
| private final String rawName; | ||
|
|
||
| private final Schema.Type type; | ||
|
|
||
| private final boolean nullable; | ||
|
|
||
| private final boolean metadata; | ||
|
|
||
| private final boolean errorCapable; | ||
|
|
||
| private final boolean valid; | ||
|
|
||
| private final Schema elementSchema; | ||
|
|
||
| private final String comment; | ||
|
|
||
| private final Map<String, String> props = new LinkedHashMap<>(0); | ||
|
|
||
| private final Object internalDefaultValue; | ||
|
|
||
| @ConstructorProperties({"name", "rawName", "type", "nullable", "metadata", "errorCapable", | ||
ypiel-talend marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| "valid", "elementSchema", "comment", "props", "internalDefaultValue"}) | ||
| public Entry( | ||
|
Check warning on line 51 in component-server-parent/component-server-model/src/main/java/org/talend/sdk/component/server/front/model/Entry.java
|
||
| final String name, | ||
| final String rawName, | ||
| final Schema.Type type, | ||
| final boolean nullable, | ||
| final boolean metadata, | ||
| final boolean errorCapable, | ||
| final boolean valid, | ||
| final Schema elementSchema, | ||
| final String comment, | ||
| final Map<String, String> props, | ||
| final Object internalDefaultValue) { | ||
| this.name = name; | ||
| this.rawName = rawName; | ||
| this.type = type; | ||
| this.nullable = nullable; | ||
| this.metadata = metadata; | ||
| this.errorCapable = errorCapable; | ||
| this.valid = valid; | ||
| this.elementSchema = elementSchema; | ||
| this.comment = comment; | ||
| this.props.putAll(props); | ||
| this.internalDefaultValue = internalDefaultValue; | ||
| } | ||
|
|
||
| public <T> T getDefaultValue(){ | ||
| return (T) this.getInternalDefaultValue(); | ||
| } | ||
|
|
||
| public String getOriginalFieldName() { | ||
| return rawName != null ? rawName : name; | ||
| } | ||
|
|
||
| public String getProp(final String key) { | ||
| return this.props.get(key); | ||
| } | ||
|
|
||
| } | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,101 @@ | ||
| /** | ||
| * Copyright (C) 2006-2025 Talend Inc. - www.talend.com | ||
| * | ||
| * 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 org.talend.sdk.component.server.front.model; | ||
|
|
||
| import java.beans.ConstructorProperties; | ||
| import java.math.BigDecimal; | ||
| import java.time.temporal.Temporal; | ||
| import java.util.Collection; | ||
| import java.util.Date; | ||
| import java.util.List; | ||
| import java.util.Map; | ||
|
|
||
| import lombok.Data; | ||
|
|
||
| @Data | ||
| public final class Schema { | ||
|
|
||
| private final Type type; | ||
|
|
||
| private final Schema elementSchema; | ||
|
|
||
| private final List<Entry> entries; | ||
|
|
||
| private final List<Entry> metadata; | ||
|
|
||
| private final Map<String, String> props; | ||
|
|
||
| @ConstructorProperties({"type", "elementSchema", "entries", "metadata", "props"}) | ||
| public Schema( | ||
|
||
| final Type type, | ||
| final Schema elementSchema, | ||
| final List<Entry> entries, | ||
| final List<Entry> metadata, | ||
| final Map<String, String> props) { | ||
| this.type = type; | ||
| this.elementSchema = elementSchema; | ||
| this.entries = entries; | ||
| this.metadata = metadata; | ||
| this.props = props; | ||
| } | ||
|
|
||
| public String getProp(final String key){ | ||
| return this.props.get(key); | ||
| } | ||
|
|
||
| public enum Type { | ||
|
|
||
| RECORD(new Class<?>[] { Record.class }), | ||
| ARRAY(new Class<?>[] { Collection.class }), | ||
| STRING(new Class<?>[] { String.class, Object.class }), | ||
| BYTES(new Class<?>[] { byte[].class, Byte[].class }), | ||
| INT(new Class<?>[] { Integer.class }), | ||
| LONG(new Class<?>[] { Long.class }), | ||
| FLOAT(new Class<?>[] { Float.class }), | ||
| DOUBLE(new Class<?>[] { Double.class }), | ||
| BOOLEAN(new Class<?>[] { Boolean.class }), | ||
| DATETIME(new Class<?>[] { Long.class, Date.class, Temporal.class }), | ||
| DECIMAL(new Class<?>[] { BigDecimal.class }); | ||
|
|
||
| /** | ||
| * All compatibles Java classes | ||
| */ | ||
| private final Class<?>[] classes; | ||
|
|
||
| Type(final Class<?>[] classes) { | ||
| this.classes = classes; | ||
| } | ||
|
|
||
| /** | ||
| * Check if input can be affected to an entry of this type. | ||
| * | ||
| * @param input : object. | ||
| * | ||
| * @return true if input is null or ok. | ||
| */ | ||
| public boolean isCompatible(final Object input) { | ||
| if (input == null) { | ||
| return true; | ||
| } | ||
| for (final Class<?> clazz : classes) { | ||
| if (clazz.isInstance(input)) { | ||
| return true; | ||
| } | ||
| } | ||
| return false; | ||
| } | ||
| } | ||
| } | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why this configuration has been added ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
because Entry need more than 10 parameters.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok, usually we can do some exception by configuration but it seems not possible for number of parameter as you probably already checked. I had the same issue few days ago, and instead of duplicate all the configuration from the parent I have simply deactivate checkstyle on the methode. In the Entry class you can do:
It is to authorize an exception, the issue with copying the whole configuration is that if we change something the parent, it will not be reflected in this module.