vinothchandar commented on a change in pull request #1150: [HUDI-288]: Add
support for ingesting multiple kafka streams in a single DeltaStreamer
deployment
URL: https://github.com/apache/incubator-hudi/pull/1150#discussion_r379215764
##########
File path:
hudi-client/src/test/java/org/apache/hudi/common/HoodieTestDataGenerator.java
##########
@@ -80,16 +79,27 @@
+ "{\"name\": \"begin_lat\", \"type\": \"double\"},{\"name\":
\"begin_lon\", \"type\": \"double\"},"
+ "{\"name\": \"end_lat\", \"type\": \"double\"},{\"name\": \"end_lon\",
\"type\": \"double\"},"
+ "{\"name\":\"fare\",\"type\": \"double\"}]}";
+ public static String GROCERY_PURCHASE_SCHEMA =
"{\"type\":\"record\",\"name\":\"purchaserec\",\"fields\":["
Review comment:
I think what @bvaradar is trying to get at it is that the current payload is
built around the trip schema.. Overall, can we just have multiple topics with
the same trips schema? I am not sure what real benefits we get from introducing
new schema here.. if we do, I would like to spend more time structuring this
class well , since it will make this class even more unclean than what it is
today..
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
With regards,
Apache Git Services
