fx19880617 opened a new issue #6573:
URL: https://github.com/apache/incubator-pinot/issues/6573


   Reported by the community:
   When user not specify the config `region`, pinot should give correct 
stacktrace in the logging for missing config: `region`.
   The code: 
`Preconditions.checkArgument(!isNullOrEmpty(config.getProperty(REGION)));` is 
not user-friendly as it's not giving an actionable and informative message.
   
   Right now: the logs is very unclear like:
   ```
   2021/02/11 11:37:06.847 ERROR [PinotFSFactory] [Start a Pinot [SERVER]] 
Could not instantiate file system for class 
org.apache.pinot.plugin.filesystem.S3PinotFS with scheme s3                     
                           
   java.lang.IllegalArgumentException: null
           at 
shaded.com.google.common.base.Preconditions.checkArgument(Preconditions.java:108)
 
~[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
                                       
           at 
org.apache.pinot.plugin.filesystem.S3PinotFS.init(S3PinotFS.java:83) 
~[pinot-s3-0.6.0-shaded.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
           at 
org.apache.pinot.spi.filesystem.PinotFSFactory.register(PinotFSFactory.java:54) 
~[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
                                         
           at 
org.apache.pinot.spi.filesystem.PinotFSFactory.init(PinotFSFactory.java:74) 
~[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
                                             
           at 
org.apache.pinot.server.starter.helix.SegmentFetcherAndLoader.<init>(SegmentFetcherAndLoader.java:71)
 
~[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
                   
           at 
org.apache.pinot.server.starter.helix.HelixServerStarter.start(HelixServerStarter.java:316)
 
~[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
                             
           at 
org.apache.pinot.tools.service.PinotServiceManager.startServer(PinotServiceManager.java:150)
 
~[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
                            
           at 
org.apache.pinot.tools.service.PinotServiceManager.startRole(PinotServiceManager.java:95)
 
~[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
                               
           at 
org.apache.pinot.tools.admin.command.StartServiceManagerCommand$1.lambda$run$0(StartServiceManagerCommand.java:260)
 
~[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
     
           at 
org.apache.pinot.tools.admin.command.StartServiceManagerCommand.startPinotService(StartServiceManagerCommand.java:286)
 
[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
   
           at 
org.apache.pinot.tools.admin.command.StartServiceManagerCommand.access$000(StartServiceManagerCommand.java:57)
 
[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
           
           at 
org.apache.pinot.tools.admin.command.StartServiceManagerCommand$1.run(StartServiceManagerCommand.java:260)
 
[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
               
   2021/02/11 11:37:06.851 ERROR [StartServiceManagerCommand] [Start a Pinot 
[SERVER]] Failed to start a Pinot [SERVER] at 0.883 since launch
   java.lang.RuntimeException: java.lang.IllegalArgumentException               
                                  
           at 
org.apache.pinot.spi.filesystem.PinotFSFactory.register(PinotFSFactory.java:58) 
~[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
                                         
           at 
org.apache.pinot.spi.filesystem.PinotFSFactory.init(PinotFSFactory.java:74) 
~[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
                                             
           at 
org.apache.pinot.server.starter.helix.SegmentFetcherAndLoader.<init>(SegmentFetcherAndLoader.java:71)
 
~[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
                   
           at 
org.apache.pinot.server.starter.helix.HelixServerStarter.start(HelixServerStarter.java:316)
 
~[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
                             
           at 
org.apache.pinot.tools.service.PinotServiceManager.startServer(PinotServiceManager.java:150)
 
~[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
                            
           at 
org.apache.pinot.tools.service.PinotServiceManager.startRole(PinotServiceManager.java:95)
 
~[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
                               
           at 
org.apache.pinot.tools.admin.command.StartServiceManagerCommand$1.lambda$run$0(StartServiceManagerCommand.java:260)
 
~[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
     
           at 
org.apache.pinot.tools.admin.command.StartServiceManagerCommand.startPinotService(StartServiceManagerCommand.java:286)
 
[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
   
           at 
org.apache.pinot.tools.admin.command.StartServiceManagerCommand.access$000(StartServiceManagerCommand.java:57)
 
[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
           
           at 
org.apache.pinot.tools.admin.command.StartServiceManagerCommand$1.run(StartServiceManagerCommand.java:260)
 
[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
               
   Caused by: java.lang.IllegalArgumentException
           at 
shaded.com.google.common.base.Preconditions.checkArgument(Preconditions.java:108)
 
~[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
                                       
           at 
org.apache.pinot.plugin.filesystem.S3PinotFS.init(S3PinotFS.java:83) 
~[pinot-s3-0.6.0-shaded.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
           at 
org.apache.pinot.spi.filesystem.PinotFSFactory.register(PinotFSFactory.java:54) 
~[pinot-all-0.6.0-jar-with-dependencies.jar:0.6.0-bb646baceafcd9b849a1ecdec7a11203c7027e21]
                                         
           ... 9 more


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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org

Reply via email to