[CARBONDATA-3119] The getOrCreateCarbonSession method 'storePath' parameter only checks for null,Causes the data store path to be an empty string and no exception is reported.Results in no results for the final select data

• Type: Improvement
• Status: Closed
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: 1.5.0
• Fix Version/s: 1.5.0
• Component/s: spark-integration
• Labels: pull-request-available
• Flags: Important
• Assignee: 张源
• Reporter: 张源
• Created: 22/Nov/18 02:46
• Updated: 28/Dec/18 02:50
• Resolved: 28/Dec/18 02:50

Description

val carbon = SparkSession.builder().config(sc.getConf).getOrCreateCarbonSession("");

if (storePath != null)

{ carbonProperties.addProperty(CarbonCommonConstants.STORE_LOCATION, storePath) // In case if it is in carbon.properties for backward compatible }

else if (carbonProperties.getProperty(CarbonCommonConstants.STORE_LOCATION) == null)

{ carbonProperties.addProperty(CarbonCommonConstants.STORE_LOCATION, session.sessionState.conf.warehousePath) }

explain:

The getOrCreateCarbonSession method storePath parameter only checks for null,Causes the data store path to be an empty string and no exception is reported.Results in no results for the final select data.

Please see the attachment for examples

Issue Links

• is a child of: CARBONDATA-3125 Find minor bug or raise some small requirements/features, and then implement them
• links to: GitHub Pull Request #2959
• links to: GitHub Pull Request #2961
• links to: GitHub Pull Request #2983