bvaradar commented on code in PR #14314:
URL: https://github.com/apache/hudi/pull/14314#discussion_r2557306963
##########
hudi-common/src/main/java/org/apache/hudi/common/schema/HoodieSchema.java:
##########
@@ -402,12 +403,12 @@ public Option<String> getNamespace() {
}
/**
- * Returns the full name of this schema (namespace + name).
+ * Returns the full name of this schema (namespace + name) if a record, or
the type name otherwise.
*
- * @return Option containing the full schema name, or Option.empty() if none
+ * @return The full schema name, or name of the type if not a record
*/
- public Option<String> getFullName() {
- return Option.ofNullable(avroSchema.getFullName());
+ public String getFullName() {
Review Comment:
Can you add a precondition in the constructor of HoodieSchema to verify this
assumption
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
