Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
voonhous commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3704498754 CI is green, merging this in. -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
voonhous merged PR #17740: URL: https://github.com/apache/hudi/pull/17740 -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
voonhous commented on code in PR #17740:
URL: https://github.com/apache/hudi/pull/17740#discussion_r2656859499
##
hudi-common/src/main/java/org/apache/hudi/avro/processors/DecimalLogicalTypeProcessor.java:
##
@@ -34,15 +34,15 @@ public abstract class DecimalLogicalTypeProcessor extends
JsonFieldProcessor {
/**
* Check if the given schema is a valid decimal type configuration.
*/
- protected static boolean isValidDecimalTypeConfig(Schema schema) {
-LogicalTypes.Decimal decimalType = (LogicalTypes.Decimal)
schema.getLogicalType();
+ protected static boolean isValidDecimalTypeConfig(HoodieSchema schema) {
+LogicalTypes.Decimal decimalType = (LogicalTypes.Decimal)
schema.toAvroSchema().getLogicalType();
Review Comment:
Nope nothing needed. Just adding a comment here for future reference.
##
hudi-common/src/main/java/org/apache/hudi/avro/MercifulJsonConverter.java:
##
@@ -221,20 +210,20 @@ protected JsonFieldProcessor getProcessorForSchema(Schema
schema) {
/**
* Build type processor map for each avro type.
*/
- private Map getFieldTypeProcessors() {
-Map fieldTypeProcessors = new
EnumMap<>(Schema.Type.class);
-fieldTypeProcessors.put(Type.STRING, generateStringTypeHandler());
-fieldTypeProcessors.put(Type.BOOLEAN, generateBooleanTypeHandler());
-fieldTypeProcessors.put(Type.DOUBLE, generateDoubleTypeHandler());
-fieldTypeProcessors.put(Type.FLOAT, generateFloatTypeHandler());
-fieldTypeProcessors.put(Type.INT, generateIntTypeHandler());
-fieldTypeProcessors.put(Type.LONG, generateLongTypeHandler());
-fieldTypeProcessors.put(Type.ARRAY, generateArrayTypeHandler());
-fieldTypeProcessors.put(Type.RECORD, generateRecordTypeHandler());
-fieldTypeProcessors.put(Type.ENUM, generateEnumTypeHandler());
-fieldTypeProcessors.put(Type.MAP, generateMapTypeHandler());
-fieldTypeProcessors.put(Type.BYTES, generateBytesTypeHandler());
-fieldTypeProcessors.put(Type.FIXED, generateFixedTypeHandler());
+ private Map getFieldTypeProcessors() {
+Map fieldTypeProcessors = new
EnumMap<>(HoodieSchemaType.class);
+fieldTypeProcessors.put(HoodieSchemaType.STRING,
generateStringTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.BOOLEAN,
generateBooleanTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.DOUBLE,
generateDoubleTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.FLOAT,
generateFloatTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.INT, generateIntTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.LONG, generateLongTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.ARRAY,
generateArrayTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.RECORD,
generateRecordTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.ENUM, generateEnumTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.MAP, generateMapTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.BYTES,
generateBytesTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.FIXED,
generateFixedTypeHandler());
Review Comment:
Yeap, it's okay.
--
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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
the-other-tim-brown commented on code in PR #17740:
URL: https://github.com/apache/hudi/pull/17740#discussion_r2656717800
##
hudi-common/src/main/java/org/apache/hudi/avro/MercifulJsonConverter.java:
##
@@ -221,20 +210,20 @@ protected JsonFieldProcessor getProcessorForSchema(Schema
schema) {
/**
* Build type processor map for each avro type.
*/
- private Map getFieldTypeProcessors() {
-Map fieldTypeProcessors = new
EnumMap<>(Schema.Type.class);
-fieldTypeProcessors.put(Type.STRING, generateStringTypeHandler());
-fieldTypeProcessors.put(Type.BOOLEAN, generateBooleanTypeHandler());
-fieldTypeProcessors.put(Type.DOUBLE, generateDoubleTypeHandler());
-fieldTypeProcessors.put(Type.FLOAT, generateFloatTypeHandler());
-fieldTypeProcessors.put(Type.INT, generateIntTypeHandler());
-fieldTypeProcessors.put(Type.LONG, generateLongTypeHandler());
-fieldTypeProcessors.put(Type.ARRAY, generateArrayTypeHandler());
-fieldTypeProcessors.put(Type.RECORD, generateRecordTypeHandler());
-fieldTypeProcessors.put(Type.ENUM, generateEnumTypeHandler());
-fieldTypeProcessors.put(Type.MAP, generateMapTypeHandler());
-fieldTypeProcessors.put(Type.BYTES, generateBytesTypeHandler());
-fieldTypeProcessors.put(Type.FIXED, generateFixedTypeHandler());
+ private Map getFieldTypeProcessors() {
+Map fieldTypeProcessors = new
EnumMap<>(HoodieSchemaType.class);
+fieldTypeProcessors.put(HoodieSchemaType.STRING,
generateStringTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.BOOLEAN,
generateBooleanTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.DOUBLE,
generateDoubleTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.FLOAT,
generateFloatTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.INT, generateIntTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.LONG, generateLongTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.ARRAY,
generateArrayTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.RECORD,
generateRecordTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.ENUM, generateEnumTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.MAP, generateMapTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.BYTES,
generateBytesTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.FIXED,
generateFixedTypeHandler());
Review Comment:
Is this ok as it is then?
--
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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3704228795 ## CI report: * 5161f8f1e07d68fd09c2f546001bd4e1a67e024b Azure: [SUCCESS](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10728) Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3704136088 ## CI report: * c70a3e19b7dccb2df11e8881395097ecf7c5aeb6 Azure: [SUCCESS](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10685) * 5161f8f1e07d68fd09c2f546001bd4e1a67e024b Azure: [PENDING](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10728) Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3704134952 ## CI report: * c70a3e19b7dccb2df11e8881395097ecf7c5aeb6 Azure: [SUCCESS](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10685) * 5161f8f1e07d68fd09c2f546001bd4e1a67e024b UNKNOWN Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
the-other-tim-brown commented on code in PR #17740:
URL: https://github.com/apache/hudi/pull/17740#discussion_r2656623732
##
hudi-common/src/main/java/org/apache/hudi/avro/processors/DecimalLogicalTypeProcessor.java:
##
@@ -34,15 +34,15 @@ public abstract class DecimalLogicalTypeProcessor extends
JsonFieldProcessor {
/**
* Check if the given schema is a valid decimal type configuration.
*/
- protected static boolean isValidDecimalTypeConfig(Schema schema) {
-LogicalTypes.Decimal decimalType = (LogicalTypes.Decimal)
schema.getLogicalType();
+ protected static boolean isValidDecimalTypeConfig(HoodieSchema schema) {
+LogicalTypes.Decimal decimalType = (LogicalTypes.Decimal)
schema.toAvroSchema().getLogicalType();
Review Comment:
I'm not really following this comment, is there anything needed in this PR?
--
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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
voonhous commented on code in PR #17740:
URL: https://github.com/apache/hudi/pull/17740#discussion_r2656488613
##
hudi-common/src/main/java/org/apache/hudi/avro/MercifulJsonConverter.java:
##
@@ -221,20 +210,20 @@ protected JsonFieldProcessor getProcessorForSchema(Schema
schema) {
/**
* Build type processor map for each avro type.
*/
- private Map getFieldTypeProcessors() {
-Map fieldTypeProcessors = new
EnumMap<>(Schema.Type.class);
-fieldTypeProcessors.put(Type.STRING, generateStringTypeHandler());
-fieldTypeProcessors.put(Type.BOOLEAN, generateBooleanTypeHandler());
-fieldTypeProcessors.put(Type.DOUBLE, generateDoubleTypeHandler());
-fieldTypeProcessors.put(Type.FLOAT, generateFloatTypeHandler());
-fieldTypeProcessors.put(Type.INT, generateIntTypeHandler());
-fieldTypeProcessors.put(Type.LONG, generateLongTypeHandler());
-fieldTypeProcessors.put(Type.ARRAY, generateArrayTypeHandler());
-fieldTypeProcessors.put(Type.RECORD, generateRecordTypeHandler());
-fieldTypeProcessors.put(Type.ENUM, generateEnumTypeHandler());
-fieldTypeProcessors.put(Type.MAP, generateMapTypeHandler());
-fieldTypeProcessors.put(Type.BYTES, generateBytesTypeHandler());
-fieldTypeProcessors.put(Type.FIXED, generateFixedTypeHandler());
+ private Map getFieldTypeProcessors() {
+Map fieldTypeProcessors = new
EnumMap<>(HoodieSchemaType.class);
+fieldTypeProcessors.put(HoodieSchemaType.STRING,
generateStringTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.BOOLEAN,
generateBooleanTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.DOUBLE,
generateDoubleTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.FLOAT,
generateFloatTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.INT, generateIntTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.LONG, generateLongTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.ARRAY,
generateArrayTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.RECORD,
generateRecordTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.ENUM, generateEnumTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.MAP, generateMapTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.BYTES,
generateBytesTypeHandler());
+fieldTypeProcessors.put(HoodieSchemaType.FIXED,
generateFixedTypeHandler());
Review Comment:
~~Is it possible to shift some of the KV pairs in
`getLogicalFieldTypeProcessors` to this scope?
e.g. DECIMAL, TIME, DATE, etc.~~
Edit: On second thoughts, ignore my comments above. We are in the Avro
scope. So, concepts should be `Avro`-first instead of `HoodieSchema`-first.
--
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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
voonhous commented on code in PR #17740:
URL: https://github.com/apache/hudi/pull/17740#discussion_r2656479003
##
hudi-common/src/main/java/org/apache/hudi/avro/processors/DecimalLogicalTypeProcessor.java:
##
@@ -34,15 +34,15 @@ public abstract class DecimalLogicalTypeProcessor extends
JsonFieldProcessor {
/**
* Check if the given schema is a valid decimal type configuration.
*/
- protected static boolean isValidDecimalTypeConfig(Schema schema) {
-LogicalTypes.Decimal decimalType = (LogicalTypes.Decimal)
schema.getLogicalType();
+ protected static boolean isValidDecimalTypeConfig(HoodieSchema schema) {
+LogicalTypes.Decimal decimalType = (LogicalTypes.Decimal)
schema.toAvroSchema().getLogicalType();
Review Comment:
Since this is still under the Avro package. I think converting everything to
avro make sense.
During my test, IIRC, via the `HoodieSchema#createDecimal` helper, but
creating a decimal that is backed by fix with an improper fixed size will fail
silently.
A `HoodieSchema` will still be created, but it will not be an instance of
`HoodieSchema.Decimal`. This might be a validation that we need to add in that
helper method. Just something orthogonal that i happened to think of when
reading this part of the code.
##
hudi-common/src/main/java/org/apache/hudi/avro/processors/DecimalLogicalTypeProcessor.java:
##
@@ -53,25 +53,23 @@ protected static boolean isValidDecimalTypeConfig(Schema
schema) {
* @return Pair object, with left as boolean indicating if the parsing was
successful and right as the
* BigDecimal value.
*/
- protected static Pair parseObjectToBigDecimal(Object
obj, Schema schema) {
+ protected static Pair parseObjectToBigDecimal(Object
obj, HoodieSchema.Decimal schema) {
BigDecimal bigDecimal = null;
-LogicalTypes.Decimal logicalType = (LogicalTypes.Decimal)
schema.getLogicalType();
try {
if (obj instanceof BigDecimal) {
-bigDecimal = ((BigDecimal) obj).setScale(logicalType.getScale(),
RoundingMode.UNNECESSARY);
+bigDecimal = ((BigDecimal) obj).setScale(schema.getScale(),
RoundingMode.UNNECESSARY);
} else if (obj instanceof String) {
// Case 2: Object is a number in String format.
try {
//encoded big decimal
- bigDecimal =
HoodieAvroUtils.convertBytesToBigDecimal(decodeStringToBigDecimalBytes(obj),
- (LogicalTypes.Decimal) schema.getLogicalType());
+ bigDecimal =
HoodieSchemaUtils.convertBytesToBigDecimal(decodeStringToBigDecimalBytes(obj),
schema);
} catch (IllegalArgumentException e) {
//no-op
}
}
// None fixed byte or fixed byte conversion failure would end up here.
if (bigDecimal == null) {
-bigDecimal = new BigDecimal(obj.toString(), new
MathContext(logicalType.getPrecision(),
RoundingMode.UNNECESSARY)).setScale(logicalType.getScale(),
RoundingMode.UNNECESSARY);
+bigDecimal = new BigDecimal(obj.toString(), new
MathContext(schema.getPrecision(),
RoundingMode.UNNECESSARY)).setScale(schema.getScale(),
RoundingMode.UNNECESSARY);
}
} catch (java.lang.NumberFormatException | ArithmeticException ignored) {
Review Comment:
Nit: Possible to import this?
##
hudi-utilities/src/main/java/org/apache/hudi/utilities/sources/helpers/ProtoConversionUtil.java:
##
@@ -474,26 +466,26 @@ private static Object convertObject(HoodieSchema schema,
Object value) {
Map mapValue = (Map) value;
Map mapCopy = new HashMap<>(mapValue.size());
for (Map.Entry entry : mapValue.entrySet()) {
-
mapCopy.put(convertObject(HoodieSchema.fromAvroSchema(STRING_SCHEMA),
entry.getKey()), convertObject(schema.getValueType(), entry.getValue()));
+mapCopy.put(convertObject(STRING_SCHEMA, entry.getKey()),
convertObject(schema.getValueType(), entry.getValue()));
}
return mapCopy;
case NULL:
return null;
case RECORD:
GenericData.Record newRecord = new
GenericData.Record(schema.toAvroSchema());
Message messageValue = (Message) value;
- Descriptors.FieldDescriptor[] orderedFields =
getOrderedFields(schema.toAvroSchema(), messageValue);
- for (Schema.Field field : schema.toAvroSchema().getFields()) {
+ Descriptors.FieldDescriptor[] orderedFields =
getOrderedFields(schema, messageValue);
+ for (HoodieSchemaField field : schema.getFields()) {
int position = field.pos();
Descriptors.FieldDescriptor fieldDescriptor =
orderedFields[position];
Object convertedValue;
-Schema fieldSchema = field.schema();
+HoodieSchema fieldSchema = field.schema();
// if incoming message does not contain the field, fieldDescriptor
will be null
// if the field schema is a union, it is null
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3700359085 ## CI report: * c70a3e19b7dccb2df11e8881395097ecf7c5aeb6 Azure: [SUCCESS](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10685) Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3700149861 ## CI report: * 229119adbb14cf6d296f122e1118e81c0551add1 Azure: [FAILURE](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10658) * c70a3e19b7dccb2df11e8881395097ecf7c5aeb6 Azure: [PENDING](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10685) Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3700147262 ## CI report: * 229119adbb14cf6d296f122e1118e81c0551add1 Azure: [FAILURE](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10658) * c70a3e19b7dccb2df11e8881395097ecf7c5aeb6 UNKNOWN Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3698277055 ## CI report: * 229119adbb14cf6d296f122e1118e81c0551add1 Azure: [FAILURE](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10658) Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3698161468 ## CI report: * 820fe16c9bcb97ff382eedc02822b88777b18f19 Azure: [FAILURE](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10653) * 229119adbb14cf6d296f122e1118e81c0551add1 Azure: [PENDING](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10658) Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3698159776 ## CI report: * 820fe16c9bcb97ff382eedc02822b88777b18f19 Azure: [FAILURE](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10653) * 229119adbb14cf6d296f122e1118e81c0551add1 UNKNOWN Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3698118027 ## CI report: * 820fe16c9bcb97ff382eedc02822b88777b18f19 Azure: [FAILURE](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10653) Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3697953693 ## CI report: * 9f40e5f86e9c3d652bff4b3e4ef891274991ccee Azure: [FAILURE](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10648) * 820fe16c9bcb97ff382eedc02822b88777b18f19 Azure: [PENDING](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10653) Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3697952182 ## CI report: * 9f40e5f86e9c3d652bff4b3e4ef891274991ccee Azure: [FAILURE](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10648) * 820fe16c9bcb97ff382eedc02822b88777b18f19 UNKNOWN Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3697796547 ## CI report: * 9f40e5f86e9c3d652bff4b3e4ef891274991ccee Azure: [FAILURE](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10648) Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3697672928 ## CI report: * 9acbbbd69b78a317bd1be0990f783656df39ed80 Azure: [FAILURE](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10645) * 9f40e5f86e9c3d652bff4b3e4ef891274991ccee Azure: [PENDING](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10648) Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3697620465 ## CI report: * 9acbbbd69b78a317bd1be0990f783656df39ed80 Azure: [FAILURE](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10645) * 9f40e5f86e9c3d652bff4b3e4ef891274991ccee UNKNOWN Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3697604456 ## CI report: * 9acbbbd69b78a317bd1be0990f783656df39ed80 Azure: [FAILURE](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10645) Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3697581090 ## CI report: * ba25f4190785e0d4b8bd7235dfc25a195b7a68ae Azure: [FAILURE](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10641) * 9acbbbd69b78a317bd1be0990f783656df39ed80 Azure: [PENDING](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10645) Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3697529086 ## CI report: * ba25f4190785e0d4b8bd7235dfc25a195b7a68ae Azure: [FAILURE](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10641) * 9acbbbd69b78a317bd1be0990f783656df39ed80 UNKNOWN Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3697507764 ## CI report: * ba25f4190785e0d4b8bd7235dfc25a195b7a68ae Azure: [FAILURE](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10641) Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3697481937 ## CI report: * ba25f4190785e0d4b8bd7235dfc25a195b7a68ae Azure: [PENDING](https://dev.azure.com/apachehudi/a1a51da7-8592-47d4-88dc-fd67bed336bb/_build/results?buildId=10641) Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
Re: [PR] feat(schema): Migrate json and proto converters to use HoodieSchema [hudi]
hudi-bot commented on PR #17740: URL: https://github.com/apache/hudi/pull/17740#issuecomment-3697478166 ## CI report: * ba25f4190785e0d4b8bd7235dfc25a195b7a68ae UNKNOWN Bot commands @hudi-bot supports the following commands: - `@hudi-bot run azure` re-run the last Azure build -- 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]
