Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CARBONDATA-1091] tupleId is not working with vector reader in spark2x #946

Merged
merged 1 commit into from
May 25, 2017

Conversation

nareshpr
Copy link
Contributor

tupleId is an implicit column used in Update/Delete Implementation.

When VectorReader is enabled, tupleId is not returning any output.

@nareshpr
Copy link
Contributor Author

nareshpr commented May 24, 2017

@ravipesala, can you please review this PR ? i tested this with unsafe in query processing enabled/disabled & with vectorReader enabled/disabled.

@asfbot
Copy link

asfbot commented May 24, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-2.1/104/

@asfbot
Copy link

asfbot commented May 24, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-1.6/102/

@ravipesala
Copy link
Contributor

@nareshpr LGTM, please add jira id to it.

@nareshpr nareshpr changed the title [WIP] tupleId is not working with vector reader in spark2x [CARBONDATA-1091] tupleId is not working with vector reader in spark2x May 25, 2017
@nareshpr nareshpr force-pushed the tupleId_spark2_vectorreader branch from 902e98b to dc0a296 Compare May 25, 2017 04:51
@asfbot
Copy link

asfbot commented May 25, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-1.6/105/

@asfbot
Copy link

asfbot commented May 25, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-2.1/108/

@asfbot
Copy link

asfbot commented May 25, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-1.6/106/

@nareshpr
Copy link
Contributor Author

@ravipesala, i added jira id.

@asfbot
Copy link

asfbot commented May 25, 2017

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/carbondata-pr-spark-2.1/107/

Build result: ABORTED

[...truncated 649.69 KB...] at java.util.concurrent.FutureTask.run(FutureTask.java:266) at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142) at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617) at java.lang.Thread.run(Thread.java:745)Caused by: org.apache.maven.plugin.MojoFailureException: Timed out after 0 seconds waiting for forked process to complete. at org.scalatest.tools.maven.AbstractScalaTestMojo.runForkingOnce(AbstractScalaTestMojo.java:319) at org.scalatest.tools.maven.AbstractScalaTestMojo.runScalaTest(AbstractScalaTestMojo.java:242) at org.scalatest.tools.maven.TestMojo.execute(TestMojo.java:106) at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo(DefaultBuildPluginManager.java:134) at org.apache.maven.lifecycle.internal.MojoExecutor.execute(MojoExecutor.java:207) ... 31 more[ERROR] [ERROR] Re-run Maven using the -X switch to enable full debug logging.[ERROR] [ERROR] For more information about the errors and possible solutions, please read the following articles:[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException[ERROR] [ERROR] After correcting the problems, you can resume the build with the command[ERROR] mvn -rf :carbondata-spark-common-testBuild was abortedchannel stoppedSetting status of 902e98b to FAILURE with url https://builds.apache.org/job/carbondata-pr-spark-2.1/107/ and message: '(Spark 2.1) 'Using context: Jenkins (Spark 2.1): Maven clean install

@ravipesala
Copy link
Contributor

LGTM

@asfgit asfgit merged commit dc0a296 into apache:master May 25, 2017
asfgit pushed a commit that referenced this pull request May 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants