Uploaded image for project: 'Livy'
  1. Livy
  2. LIVY-986

Need to add null check for SessionInfo

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 0.8.0
    • 0.9.0
    • Server
    • None

    Description

      We need to add a null check for SessionInfo in InteractiveSessionServlet.scala to avoid that might occur during a Livy upgrade.

       new SessionInfo(session.id, session.name.orNull, session.appId.orNull,
            session.owner, session.state.toString, session.kind.toString,
            session.appInfo.asJavaMap, logs.asJava, session.ttl.orNull,
            session.driverMemory.orNull,
            session.driverCores.getOrElse(0), session.executorMemory.orNull,
            session.executorCores.getOrElse(0), session.conf.asJava, session.archives.asJava,
            session.files.asJava, session.heartbeatTimeoutS, session.jars.asJava,
            session.numExecutors.getOrElse(0), session.proxyUser.orNull, session.pyFiles.asJava,
            session.queue.orNull)
      

      This code can throw exception when session.conf is not null.

      Attachments

        Activity

          People

            asifkhatri Asif Khatri
            asifkhatri Asif Khatri
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0h
                0h
                Logged:
                Time Spent - 0.5h
                0.5h