[GitHub] [carbondata] jackylk opened a new pull request #3596: [CARBONDATA-3673] Remove unused declare

classic Classic list List threaded Threaded
6 messages Options
Reply | Threaded
Open this post in threaded view
|

[GitHub] [carbondata] jackylk opened a new pull request #3596: [CARBONDATA-3673] Remove unused declare

GitBox
jackylk opened a new pull request #3596: [CARBONDATA-3673] Remove unused declare
URL: https://github.com/apache/carbondata/pull/3596
 
 
    ### Why is this PR needed?
   After global dictionary, custom partition feature is deprecated, some code path is not used. It is better to do refactory.
   
    ### What changes were proposed in this PR?
   This PR removes unused declarations.
       
    ### 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:
[hidden email]


With regards,
Apache Git Services
Reply | Threaded
Open this post in threaded view
|

[GitHub] [carbondata] CarbonDataQA1 commented on issue #3596: [CARBONDATA-3673] Remove unused declare

GitBox
CarbonDataQA1 commented on issue #3596: [CARBONDATA-3673] Remove unused declare
URL: https://github.com/apache/carbondata/pull/3596#issuecomment-579842075
 
 
   Build Failed  with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/1798/
   

----------------------------------------------------------------
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
Reply | Threaded
Open this post in threaded view
|

[GitHub] [carbondata] CarbonDataQA1 commented on issue #3596: [CARBONDATA-3673] Remove unused declarations

GitBox
In reply to this post by GitBox
CarbonDataQA1 commented on issue #3596: [CARBONDATA-3673] Remove unused declarations
URL: https://github.com/apache/carbondata/pull/3596#issuecomment-580138078
 
 
   Build Failed  with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/1802/
   

----------------------------------------------------------------
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
Reply | Threaded
Open this post in threaded view
|

[GitHub] [carbondata] CarbonDataQA1 commented on issue #3596: [CARBONDATA-3673] Remove unused declarations

GitBox
In reply to this post by GitBox
CarbonDataQA1 commented on issue #3596: [CARBONDATA-3673] Remove unused declarations
URL: https://github.com/apache/carbondata/pull/3596#issuecomment-580187468
 
 
   Build Failed  with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/1803/
   

----------------------------------------------------------------
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
Reply | Threaded
Open this post in threaded view
|

[GitHub] [carbondata] CarbonDataQA1 commented on issue #3596: [CARBONDATA-3673] Remove unused declarations

GitBox
In reply to this post by GitBox
CarbonDataQA1 commented on issue #3596: [CARBONDATA-3673] Remove unused declarations
URL: https://github.com/apache/carbondata/pull/3596#issuecomment-581264627
 
 
   Build Failed  with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.4/2/
   

----------------------------------------------------------------
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
Reply | Threaded
Open this post in threaded view
|

[GitHub] [carbondata] jackylk closed pull request #3596: [CARBONDATA-3673] Remove unused declarations

GitBox
In reply to this post by GitBox
jackylk closed pull request #3596: [CARBONDATA-3673] Remove unused declarations
URL: https://github.com/apache/carbondata/pull/3596
 
 
   

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