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

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

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



##########
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:
       That's a demo program for others to copy. They may not use `lombok`, so i think use `set get` more friendly




-- 
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