You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@carbondata.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2016/09/08 06:53:20 UTC

[jira] [Commented] (CARBONDATA-226) Delete load by ID message when the compacted segment is present is wrong.

    [ https://issues.apache.org/jira/browse/CARBONDATA-226?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15473007#comment-15473007 ] 

ASF GitHub Bot commented on CARBONDATA-226:
-------------------------------------------

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

    https://github.com/apache/incubator-carbondata/pull/135#discussion_r77953863
  
    --- Diff: integration/spark/src/main/scala/org/apache/spark/sql/execution/command/carbonTableSchema.scala ---
    @@ -931,11 +931,11 @@ private[sql] case class DeleteLoadsById(
     
         if (invalidLoadIds.isEmpty) {
     
    -      LOGGER.audit(s"Delete load by Id is successfull for $databaseName.$tableName.")
    +      LOGGER.audit(s"Delete segment by Id is successfull for $databaseName.$tableName.")
         }
         else {
    -      sys.error("Delete load by Id is failed. No matching load id found. SegmentSeqId(s) - "
    -                + invalidLoadIds)
    +      sys.error("Delete segment by Id is failed. Invalid ID is :"
    +                + invalidLoadIds.mkString(",") + ". Please check the log.")
    --- End diff --
    
    remove please check log message.


> Delete load by ID message when the compacted segment is present is wrong.
> -------------------------------------------------------------------------
>
>                 Key: CARBONDATA-226
>                 URL: https://issues.apache.org/jira/browse/CARBONDATA-226
>             Project: CarbonData
>          Issue Type: Bug
>            Reporter: ravikiran
>
> Delete load by ID message when the compacted segment is present is wrong. Need to change the message.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)