[GitHub] [carbondata] xubo245 commented on a change in pull request #3213: [CARBONDATA-3380] Fix missing appName and AnalysisException bug in DirectSQLExample

[GitBox]

xubo245 commented on a change in pull request #3213: [CARBONDATA-3380] Fix missing appName and AnalysisException bug in DirectSQLExample
URL: https://github.com/apache/carbondata/pull/3213#discussion_r288055134
 
 

 ##########
 File path: examples/spark2/src/main/scala/org/apache/carbondata/examples/DirectSQLExample.scala
 ##########
 @@ -86,7 +87,7 @@ object DirectSQLExample {
     println("Running SQL on carbon files directly")
     try {
       // 2. run queries directly, no need to create table first
-      sql(s"""select * FROM  carbonfile.`$readPath` limit 10""".stripMargin).show()
+      sql(s"""select * FROM carbon.`$readPath` limit 10""".stripMargin).show()
 
 Review comment:
   Can you add this example(DirectSQLExample) to org.apache.carbondata.examplesCI.RunExamples? and do some check? which can avoid fail when other developer change the code in the future.

----------------------------------------------------------------
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:
[hidden email]


With regards,
Apache Git Services