[GitHub] [carbondata] VenuReddy2103 opened a new pull request #3850: [CARBONDATA-3907]Reuse firePreLoadEvents and firePostLoadEvents methods from CommonLoadUtils to trigger LoadTablePreExecutionEvent and LoadTablePostExecutionEvent respectively in alter table add segment flow

Thu, 16 Jul 2020 04:27:10 -0700 


VenuReddy2103 opened a new pull request #3850:
URL: https://github.com/apache/carbondata/pull/3850


    ### Why is this PR needed?
   Currently we have 2 different ways of firing LoadTablePreExecutionEvent and 
LoadTablePostExecutionEvent. We can reuse firePreLoadEvents and 
firePostLoadEvents methods from CommonLoadUtils to trigger 
LoadTablePreExecutionEvent and LoadTablePostExecutionEvent respectively in 
alter table add segment flow as well. So that we can have single flow to fire 
these events
    
    ### What changes were proposed in this PR?
   Reuse firePreLoadEvents and firePostLoadEvents methods from CommonLoadUtils 
to trigger LoadTablePreExecutionEvent and LoadTablePostExecutionEvent 
respectively in alter table add segment flow.
       
    ### Does this PR introduce any user interface change?
    - No
   
    ### Is any new testcase added?
    - No
   


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

Reply via email to