[GitHub] [carbondata] ajantha-bhat opened a new pull request #3685: [HOTFIX] Fix all flink test case failure and enable UT in CI

[GitBox]

ajantha-bhat opened a new pull request #3685: [HOTFIX] Fix all flink test case failure and enable UT in CI
URL: https://github.com/apache/carbondata/pull/3685
 
 
    ### Why is this PR needed?
    After #3628, default BATCH_FILE_ORDER is wrong [it is not ASC or DSC].
   so, all the test case in flink module failed due as no order is set in stage command.
   Also flink UT is not running in CI, hence it is not caught
   
    ### What changes were proposed in this PR?
   Fix the default value to Ascending order.
   Enable UT running for flink module.
       
    ### 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

[GitBox]

ajantha-bhat commented on issue #3685: [HOTFIX] Fix all flink test case failure and enable UT in CI
URL: https://github.com/apache/carbondata/pull/3685#issuecomment-604967884
 
 
   @niuge01 , @jackylk : please check

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

[GitBox]

CarbonDataQA1 commented on issue #3685: [HOTFIX] Fix all flink test case failure and enable UT in CI
URL: https://github.com/apache/carbondata/pull/3685#issuecomment-605030448
 
 
   Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/2573/
   

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

[GitBox]

CarbonDataQA1 commented on issue #3685: [HOTFIX] Fix all flink test case failure and enable UT in CI
URL: https://github.com/apache/carbondata/pull/3685#issuecomment-605032353
 
 
   Build Failed  with Spark 2.4.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/865/
   

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

[GitBox]

CarbonDataQA1 commented on issue #3685: [HOTFIX] Fix all flink test case failure and enable UT in CI
URL: https://github.com/apache/carbondata/pull/3685#issuecomment-605267135
 
 
   Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/875/
   

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

[GitBox]

CarbonDataQA1 commented on issue #3685: [HOTFIX] Fix all flink test case failure and enable UT in CI
URL: https://github.com/apache/carbondata/pull/3685#issuecomment-605280758
 
 
   Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/2583/
   

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

[GitBox]

niuge01 commented on issue #3685: [HOTFIX] Fix all flink test case failure and enable UT in CI
URL: https://github.com/apache/carbondata/pull/3685#issuecomment-605371290
 
 
   LGTM

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

[GitBox]

asfgit closed pull request #3685: [HOTFIX] Fix all flink test case failure and enable UT in CI
URL: https://github.com/apache/carbondata/pull/3685
 
 
   

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