You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@carbondata.apache.org by GitBox <gi...@apache.org> on 2020/10/20 14:03:32 UTC
[GitHub] [carbondata] Indhumathi27 opened a new pull request #3989: [TEST] CI
Indhumathi27 opened a new pull request #3989:
URL: https://github.com/apache/carbondata/pull/3989
### Why is this PR needed?
### What changes were proposed in this PR?
### Does this PR introduce any user interface change?
- No
- Yes. (please explain the change and update document)
### Is any new testcase added?
- No
- 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:
users@infra.apache.org
[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3989: [TEST] CI
Posted by GitBox <gi...@apache.org>.
CarbonDataQA1 commented on pull request #3989:
URL: https://github.com/apache/carbondata/pull/3989#issuecomment-713148503
Build Failed with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/4541/
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3989: [HOTFIX] Fix random failure issue due to indexMerge
Posted by GitBox <gi...@apache.org>.
CarbonDataQA1 commented on pull request #3989:
URL: https://github.com/apache/carbondata/pull/3989#issuecomment-714375670
Build Failed with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/4623/
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [carbondata] QiangCai commented on pull request #3989: [TEST] CI
Posted by GitBox <gi...@apache.org>.
QiangCai commented on pull request #3989:
URL: https://github.com/apache/carbondata/pull/3989#issuecomment-713448728
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:
users@infra.apache.org
[GitHub] [carbondata] QiangCai commented on a change in pull request #3989: [HOTFIX] Fix random failure issue due to indexMerge
Posted by GitBox <gi...@apache.org>.
QiangCai commented on a change in pull request #3989:
URL: https://github.com/apache/carbondata/pull/3989#discussion_r510038068
##########
File path: integration/spark/src/test/scala/org/apache/carbondata/spark/testsuite/standardpartition/StandardPartitionTableQueryTestCase.scala
##########
@@ -21,7 +21,6 @@ import org.apache.spark.sql.{CarbonEnv, DataFrame, Row}
import org.apache.spark.sql.catalyst.InternalRow
import org.apache.spark.sql.execution.strategy.CarbonDataSourceScan
import org.apache.spark.sql.test.util.QueryTest
-import org.apache.spark.util.SparkUtil
import org.scalatest.BeforeAndAfterAll
Review comment:
line 43:
.addProperty(CarbonCommonConstants.CARBON_MERGE_INDEX_IN_SEGMENT, "true")
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3989: [HOTFIX] Fix random failure issue due to indexMerge
Posted by GitBox <gi...@apache.org>.
CarbonDataQA1 commented on pull request #3989:
URL: https://github.com/apache/carbondata/pull/3989#issuecomment-714375489
Build Failed with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/2868/
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [carbondata] Indhumathi27 commented on a change in pull request #3989: [HOTFIX] Fix random failure issue due to indexMerge
Posted by GitBox <gi...@apache.org>.
Indhumathi27 commented on a change in pull request #3989:
URL: https://github.com/apache/carbondata/pull/3989#discussion_r510046307
##########
File path: integration/spark/src/test/scala/org/apache/carbondata/spark/testsuite/standardpartition/StandardPartitionTableQueryTestCase.scala
##########
@@ -21,7 +21,6 @@ import org.apache.spark.sql.{CarbonEnv, DataFrame, Row}
import org.apache.spark.sql.catalyst.InternalRow
import org.apache.spark.sql.execution.strategy.CarbonDataSourceScan
import org.apache.spark.sql.test.util.QueryTest
-import org.apache.spark.util.SparkUtil
import org.scalatest.BeforeAndAfterAll
Review comment:
Not required. I have added this before to check another scenario
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3989: [TEST] CI
Posted by GitBox <gi...@apache.org>.
CarbonDataQA1 commented on pull request #3989:
URL: https://github.com/apache/carbondata/pull/3989#issuecomment-713161614
Build Failed with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/2787/
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [carbondata] asfgit closed pull request #3989: [HOTFIX] Fix random failure issue due to indexMerge
Posted by GitBox <gi...@apache.org>.
asfgit closed pull request #3989:
URL: https://github.com/apache/carbondata/pull/3989
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3989: [TEST] CI
Posted by GitBox <gi...@apache.org>.
CarbonDataQA1 commented on pull request #3989:
URL: https://github.com/apache/carbondata/pull/3989#issuecomment-713507326
Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/4579/
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3989: [HOTFIX] Fix random failure issue due to indexMerge
Posted by GitBox <gi...@apache.org>.
CarbonDataQA1 commented on pull request #3989:
URL: https://github.com/apache/carbondata/pull/3989#issuecomment-714435778
Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/2869/
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3989: [TEST] CI
Posted by GitBox <gi...@apache.org>.
CarbonDataQA1 commented on pull request #3989:
URL: https://github.com/apache/carbondata/pull/3989#issuecomment-713440556
Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/4570/
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [carbondata] QiangCai commented on pull request #3989: [HOTFIX] Fix random failure issue due to indexMerge
Posted by GitBox <gi...@apache.org>.
QiangCai commented on pull request #3989:
URL: https://github.com/apache/carbondata/pull/3989#issuecomment-714417384
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:
users@infra.apache.org
[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3989: [TEST] CI
Posted by GitBox <gi...@apache.org>.
CarbonDataQA1 commented on pull request #3989:
URL: https://github.com/apache/carbondata/pull/3989#issuecomment-713414165
Build Success with Spark 2.4.5, Please check CI http://121.244.95.60:12545/job/ApacheCarbon_PR_Builder_2.4.5/2815/
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [carbondata] QiangCai commented on a change in pull request #3989: [HOTFIX] Fix random failure issue due to indexMerge
Posted by GitBox <gi...@apache.org>.
QiangCai commented on a change in pull request #3989:
URL: https://github.com/apache/carbondata/pull/3989#discussion_r510090596
##########
File path: integration/spark/src/test/scala/org/apache/carbondata/spark/testsuite/standardpartition/StandardPartitionTableQueryTestCase.scala
##########
@@ -21,7 +21,6 @@ import org.apache.spark.sql.{CarbonEnv, DataFrame, Row}
import org.apache.spark.sql.catalyst.InternalRow
import org.apache.spark.sql.execution.strategy.CarbonDataSourceScan
import org.apache.spark.sql.test.util.QueryTest
-import org.apache.spark.util.SparkUtil
import org.scalatest.BeforeAndAfterAll
Review comment:
ok
----------------------------------------------------------------
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:
users@infra.apache.org
[GitHub] [carbondata] CarbonDataQA1 commented on pull request #3989: [HOTFIX] Fix random failure issue due to indexMerge
Posted by GitBox <gi...@apache.org>.
CarbonDataQA1 commented on pull request #3989:
URL: https://github.com/apache/carbondata/pull/3989#issuecomment-714430218
Build Success with Spark 2.3.4, Please check CI http://121.244.95.60:12545/job/ApacheCarbonPRBuilder2.3/4624/
----------------------------------------------------------------
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:
users@infra.apache.org