Github user zzcclp commented on a diff in the pull request:

    https://github.com/apache/carbondata/pull/2961#discussion_r237783794
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/CarbonSession.scala ---
    @@ -248,7 +251,7 @@ object CarbonSession {
     
             session = new CarbonSession(sparkContext, None, !enableInMemCatlog)
             val carbonProperties = CarbonProperties.getInstance()
    -        if (storePath != null) {
    +        if (storePath != null && StringUtils.isNotBlank(storePath)) {
    --- End diff --
    
    remove 'storePath != null', StringUtils.isNotBlank already includes this 
judgement.


---

Reply via email to