Kafka
  1. Kafka
  2. KAFKA-768

broker should exit if hitting exceptions durin startup

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 0.8.0
    • Fix Version/s: 0.8.0
    • Component/s: core
    • Labels:

      Description

      A broker hit the following exception, but didn't exit.

      2013/02/20 01:54:21.341 FATAL [KafkaServerStartable] [main] [kafka] [] Fatal error during KafkaServerStable startup. Prepare to shutdown
      kafka.common.KafkaException: Failed to create data directory /export/content/kafka/i001_caches
      at kafka.log.LogManager$$anonfun$createAndValidateLogDirs$2.apply(LogManager.scala:77)
      at kafka.log.LogManager$$anonfun$createAndValidateLogDirs$2.apply(LogManager.scala:72)
      at scala.collection.IndexedSeqOptimized$class.foreach(IndexedSeqOptimized.scala:34)
      at scala.collection.mutable.WrappedArray.foreach(WrappedArray.scala:32)
      at kafka.log.LogManager.createAndValidateLogDirs(LogManager.scala:72)
      at kafka.log.LogManager.<init>(LogManager.scala:60)
      at kafka.server.KafkaServer.startup(KafkaServer.scala:59)
      at kafka.server.KafkaServerStartable.startup(KafkaServerStartable.scala:34)

        Activity

        Neha Narkhede made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Hide
        Swapnil Ghike added a comment -

        Filed KAFKA-770 with a patch.

        Show
        Swapnil Ghike added a comment - Filed KAFKA-770 with a patch.
        Jun Rao made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Fix Version/s 0.8 [ 12317244 ]
        Resolution Fixed [ 1 ]
        Hide
        Jun Rao added a comment -

        Thanks for the review. Committed to 0.8.

        Swapnil,

        I agree. Could you file a separate jira to track that?

        Show
        Jun Rao added a comment - Thanks for the review. Committed to 0.8. Swapnil, I agree. Could you file a separate jira to track that?
        Neha Narkhede made changes -
        Labels p1
        Hide
        Neha Narkhede added a comment -

        +1

        Show
        Neha Narkhede added a comment - +1
        Hide
        Swapnil Ghike added a comment -

        Also to maintain consistency with Producer/ConsumerConfigs, perhaps we should verify KafkaConfig properties in the secondary constructor instead of having a separate verify method.

        Show
        Swapnil Ghike added a comment - Also to maintain consistency with Producer/ConsumerConfigs, perhaps we should verify KafkaConfig properties in the secondary constructor instead of having a separate verify method.
        Hide
        Swapnil Ghike added a comment -

        +1

        Show
        Swapnil Ghike added a comment - +1
        Hide
        Sriram Subramanian added a comment -

        +1

        Show
        Sriram Subramanian added a comment - +1
        Jun Rao made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        Assignee Jun Rao [ junrao ]
        Jun Rao made changes -
        Field Original Value New Value
        Attachment kafka-768.patch [ 12570354 ]
        Hide
        Jun Rao added a comment -

        Attach a patch.

        Show
        Jun Rao added a comment - Attach a patch.
        Jun Rao created issue -

          People

          • Assignee:
            Jun Rao
            Reporter:
            Jun Rao
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development