BJangir commented on a change in pull request #3375: [CARBONDATA-3512]Index 
Server Enhancement
URL: https://github.com/apache/carbondata/pull/3375#discussion_r322337555
 
 

 ##########
 File path: 
integration/spark2/src/main/scala/org/apache/carbondata/indexserver/IndexServer.scala
 ##########
 @@ -105,6 +109,11 @@ object IndexServer extends ServerInterface {
         sparkSession.sparkContext.setLocalProperty("spark.jobGroup.id", 
request.getTaskGroupId)
         sparkSession.sparkContext
           .setLocalProperty("spark.job.description", request.getTaskGroupDesc)
+        // Fire Generic Event like ACLCheck..etc
 
 Review comment:
   operationContext  will used in all method except ShowCache() Method and 
those method will be called multiple time so defined global one time object 
should be ok.

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


With regards,
Apache Git Services

Reply via email to