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

    https://github.com/apache/carbondata/pull/2852#discussion_r228845130
  
    --- Diff: 
integration/spark2/src/main/scala/org/apache/spark/sql/CarbonDictionaryDecoder.scala
 ---
    @@ -281,9 +281,7 @@ case class CarbonDictionaryDecoder(
                         
.getColumnSchema.getParentColumnTableRelations.isEmpty) {
                     val parentRelationIdentifier = 
carbonDimension.getColumnSchema
                       
.getParentColumnTableRelations.get(0).getRelationIdentifier
    -                val parentTablePath = CarbonMetadata.getInstance()
    -                  .getCarbonTable(parentRelationIdentifier.getDatabaseName,
    -                    parentRelationIdentifier.getTableName).getTablePath
    +                val parentTablePath = parentRelationIdentifier.getTablePath
    --- End diff --
    
    `parentTablePath` is unused. Please check and remove if not required


---

Reply via email to