[GitHub] [carbondata] nihal0107 opened a new pull request #4041: [CARBONDATA-4068] handled set long string column on SI table.

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

[GitHub] [carbondata] nihal0107 opened a new pull request #4041: [CARBONDATA-4068] handled set long string column on SI table.

GitBox

nihal0107 opened a new pull request #4041:
URL: https://github.com/apache/carbondata/pull/4041


    ### Why is this PR needed?
    Currently  set column as long string on which SI is already created is allowed. This operation should not be allowed because SI doesn't support long strings.
   
    ### What changes were proposed in this PR?
   Handled set long string on SI columns and throws the exception.
       
    ### Does this PR introduce any user interface change?
    - No
   
    ### Is any new testcase added?
    - Yes
   
       
   


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


Reply | Threaded
Open this post in threaded view
|

[GitHub] [carbondata] CarbonDataQA2 commented on pull request #4041: [CARBONDATA-4068] handled set long string on MT for column on which SI is already created.

GitBox

CarbonDataQA2 commented on pull request #4041:
URL: https://github.com/apache/carbondata/pull/4041#issuecomment-738632807


   Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/5064/
   


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


Reply | Threaded
Open this post in threaded view
|

[GitHub] [carbondata] CarbonDataQA2 commented on pull request #4041: [CARBONDATA-4068] handled set long string on MT for column on which SI is already created.

GitBox
In reply to this post by GitBox

CarbonDataQA2 commented on pull request #4041:
URL: https://github.com/apache/carbondata/pull/4041#issuecomment-738636337


   Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/3305/
   


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


Reply | Threaded
Open this post in threaded view
|

[GitHub] [carbondata] CarbonDataQA2 commented on pull request #4041: [CARBONDATA-4068] handled set long string on MT for column on which SI is already created.

GitBox
In reply to this post by GitBox

CarbonDataQA2 commented on pull request #4041:
URL: https://github.com/apache/carbondata/pull/4041#issuecomment-739468660


   Build Failed  with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/3291/
   


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


Reply | Threaded
Open this post in threaded view
|

[GitHub] [carbondata] CarbonDataQA2 commented on pull request #4041: [CARBONDATA-4068] handled set long string on MT for column on which SI is already created.

GitBox
In reply to this post by GitBox

CarbonDataQA2 commented on pull request #4041:
URL: https://github.com/apache/carbondata/pull/4041#issuecomment-739480242


   Build Failed  with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/5049/
   


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


Reply | Threaded
Open this post in threaded view
|

[GitHub] [carbondata] brijoobopanna commented on pull request #4041: [CARBONDATA-4068] handled set long string on MT for column on which SI is already created.

GitBox
In reply to this post by GitBox

brijoobopanna commented on pull request #4041:
URL: https://github.com/apache/carbondata/pull/4041#issuecomment-739941027


   retest this please
   


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


Reply | Threaded
Open this post in threaded view
|

[GitHub] [carbondata] CarbonDataQA2 commented on pull request #4041: [CARBONDATA-4068] handled set long string on MT for column on which SI is already created.

GitBox
In reply to this post by GitBox

CarbonDataQA2 commented on pull request #4041:
URL: https://github.com/apache/carbondata/pull/4041#issuecomment-740010043


   Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12444/job/ApacheCarbonPRBuilder2.3/5097/
   


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


Reply | Threaded
Open this post in threaded view
|

[GitHub] [carbondata] CarbonDataQA2 commented on pull request #4041: [CARBONDATA-4068] handled set long string on MT for column on which SI is already created.

GitBox
In reply to this post by GitBox

CarbonDataQA2 commented on pull request #4041:
URL: https://github.com/apache/carbondata/pull/4041#issuecomment-740014376


   Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12444/job/ApacheCarbon_PR_Builder_2.4.5/3336/
   


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


Reply | Threaded
Open this post in threaded view
|

[GitHub] [carbondata] nihal0107 closed pull request #4041: [CARBONDATA-4068] handled set long string on MT for column on which SI is already created.

GitBox
In reply to this post by GitBox

nihal0107 closed pull request #4041:
URL: https://github.com/apache/carbondata/pull/4041


   


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