[
https://issues.apache.org/jira/browse/BEAM-6240?focusedWorklogId=182184&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-182184
]
ASF GitHub Bot logged work on BEAM-6240:
----------------------------------------
Author: ASF GitHub Bot
Created on: 08/Jan/19 06:02
Start Date: 08/Jan/19 06:02
Worklog Time Spent: 10m
Work Description: reuvenlax commented on pull request #7289: [BEAM-6240]
Add a library of schema annotations for POJO and JavaBeans
URL: https://github.com/apache/beam/pull/7289#discussion_r245885269
##########
File path:
sdks/java/core/src/main/java/org/apache/beam/sdk/schemas/utils/FieldValueTypeSupplier.java
##########
@@ -28,8 +29,10 @@
*/
public interface FieldValueTypeSupplier extends Serializable {
/**
- * Return all the FieldValueTypeInformations. The returned list must be in
the same order as
+ * Return all the FieldValueTypeInformations.
+ *
+ * <p>If the schema parameter is not null, then the returned list must be in
the same order as
* fields in the schema.
*/
- List<FieldValueTypeInformation> get(Class<?> clazz, Schema schema);
+ List<FieldValueTypeInformation> get(Class<?> clazz, @Nullable Schema schema);
Review comment:
Since every single call site except for one needed this, moving the sorting
to all call sites seems undesirable. Instead I split into two separate methods,
one taking Schema one not, so that we don't need a @Nullable parameter
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 182184)
Time Spent: 5.5h (was: 5h 20m)
> Allow users to annotate POJOs and JavaBeans for richer functionality
> --------------------------------------------------------------------
>
> Key: BEAM-6240
> URL: https://issues.apache.org/jira/browse/BEAM-6240
> Project: Beam
> Issue Type: Sub-task
> Components: sdk-java-core
> Reporter: Reuven Lax
> Assignee: Reuven Lax
> Priority: Major
> Time Spent: 5.5h
> Remaining Estimate: 0h
>
> Desired annotations:
> * SchemaIgnore - ignore this field
> * FieldName - allow the user to explicitly specify a field name
> * SchemaCreate - register a function to be used to create an object (so
> fields can be final, and no default constructor need be assumed).
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
