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.

Sign up for GitHub

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

Jump to bottom

[CARBONDATA-1832] Table cache should be cleared when dropping table #1585

Closed
wants to merge 3 commits into from
Closed

[CARBONDATA-1832] Table cache should be cleared when dropping table #1585

wants to merge 3 commits into from

Conversation

jackylk
Copy link
Contributor

@jackylk jackylk commented Nov 29, 2017

Table cache should be cleared when dropping table. This PR drop the index and dictionary cache in drive side.

  • Any interfaces changed?
    No
  • Any backward compatibility impacted?
    No
  • Document update required?
    No
  • Testing done
    Yes
  • For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.
    NA

@CarbonDataQA
Copy link

Build Failed with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1552/

@ravipesala
Copy link
Contributor

SDV Build Fail , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/1951/

@jackylk
Copy link
Contributor Author

jackylk commented Nov 29, 2017

retest this please

@CarbonDataQA
Copy link

Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1559/

@jackylk
clear cache when drop table
2264db4 
fix style

fix style
@CarbonDataQA
Copy link

Build Failed with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1565/

@CarbonDataQA
Copy link

Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1569/

@chenliang613
Copy link
Contributor

LGTM

@asfgit asfgit closed this in f2dcb6d Nov 29, 2017
@ravipesala
Copy link
Contributor

SDV Build Success , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/1967/

@CarbonDataQA
Copy link

Build Success with Spark 2.1.0, Please check CI http://136.243.101.176:8080/job/ApacheCarbonPRBuilder1/1577/

@ravipesala
Copy link
Contributor

SDV Build Success , Please check CI http://144.76.159.231:8080/job/ApacheSDVTests/1978/

SangeetaGulia pushed a commit to SangeetaGulia/incubator-carbondata that referenced this pull request Dec 27, 2017
@jackylk @SangeetaGulia
[CARBONDATA-1832] Table cache should be cleared when dropping table
762d5d5 
Table cache should be cleared when dropping table. This PR drop the index and dictionary cache in drive side.

This closes apache#1585
jatin9896 pushed a commit to jatin9896/incubator-carbondata that referenced this pull request Jan 5, 2018
@jackylk @jatin9896
[CARBONDATA-1832] Table cache should be cleared when dropping table
4b60e43 
Table cache should be cleared when dropping table. This PR drop the index and dictionary cache in drive side.

This closes apache#1585
anubhav100 pushed a commit to anubhav100/incubator-carbondata that referenced this pull request Jun 22, 2018
@jackylk @anubhav100
[CARBONDATA-1832] Table cache should be cleared when dropping table
acede6d 
Table cache should be cleared when dropping table. This PR drop the index and dictionary cache in drive side.

This closes apache#1585
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@jackylk @CarbonDataQA @ravipesala @chenliang613