[GitHub] [carbondata] ajantha-bhat opened a new pull request #3731: [WIP] presto carbon reader should use tablePath from hive catalog

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

[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3731: [CARBONDATA-3786] presto carbon reader should use tablePath from hive catalog

GitBox

CarbonDataQA1 commented on pull request #3731:
URL: https://github.com/apache/carbondata/pull/3731#issuecomment-625902845


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


----------------------------------------------------------------
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] CarbonDataQA1 commented on pull request #3731: [CARBONDATA-3786] presto carbon reader should use tablePath from hive catalog

GitBox
In reply to this post by GitBox

CarbonDataQA1 commented on pull request #3731:
URL: https://github.com/apache/carbondata/pull/3731#issuecomment-625903215


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


----------------------------------------------------------------
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] QiangCai commented on pull request #3731: [CARBONDATA-3786] presto carbon reader should use tablePath from hive catalog

GitBox
In reply to this post by GitBox

QiangCai commented on pull request #3731:
URL: https://github.com/apache/carbondata/pull/3731#issuecomment-626113588


   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]


Reply | Threaded
Open this post in threaded view
|

[GitHub] [carbondata] QiangCai commented on pull request #3731: [CARBONDATA-3786] presto carbon reader should use tablePath from hive catalog

GitBox
In reply to this post by GitBox

QiangCai commented on pull request #3731:
URL: https://github.com/apache/carbondata/pull/3731#issuecomment-626514553


   merge this PR, but we still need to fix the storage location in another PR.


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


12