[PR] [ISSUE-4338] Fix checkstyle issue in sdk module [carbondata]

2023-11-06 Thread via GitHub
QiangCai opened a new pull request, #4339: URL: https://github.com/apache/carbondata/pull/4339 ### Why is this PR needed? There is a checkstyle issue in sdk module ### What changes were proposed in this PR? not import static method in CarbonSchemaReader ### Does th

Re: [PR] Fix TestInsertAndOtherCommandConcurrent testcasee issue and enble testcase [carbondata]

2023-11-06 Thread via GitHub
chenliang613 merged PR #4337: URL: https://github.com/apache/carbondata/pull/4337 -- 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. To unsubscribe, e-mail: dev-unsubscr...@carbon

Re: [PR] Fix TestInsertAndOtherCommandConcurrent testcasee issue and enble testcase [carbondata]

2023-11-06 Thread via GitHub
chenliang613 commented on PR #4337: URL: https://github.com/apache/carbondata/pull/4337#issuecomment-1794808481 looks good to me. -- 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 comm

[PR] Fix TestInsertAndOtherCommandConcurrent testcasee issue and enble testcase [carbondata]

2023-11-06 Thread via GitHub
QiangCai opened a new pull request, #4337: URL: https://github.com/apache/carbondata/pull/4337 ### Why is this PR needed? 1. Fix TestInsertAndOtherCommandConcurrent testcasee issue 2. Using a proper target path in testcase to avoid generating new files during test 3. Enable spar