[GitHub] [pulsar] hangc0276 commented on a change in pull request #12127: presto sample doc missing the set field method

[GitBox <gi...@apache.org>]

hangc0276 commented on a change in pull request #12127:
URL: https://github.com/apache/pulsar/pull/12127#discussion_r716120092



##########
File path: site2/docs/sql-getting-started.md
##########
@@ -117,18 +117,45 @@ You can query the mock data.
 If you want to query your own data, you need to ingest your own data first. You can write a simple producer and write custom defined data to Pulsar. The following is an example. 
 
 ```java
-public class Test {
-    
-     public static class Foo {
+public class TestProducer {
+
+    public static class Foo {

Review comment:
       Use  `@Data`  and  `@NoArgsConstructor` instead of implement get and set methods maybe better.




-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org