Skip to content

Commit

Permalink
[query] Remove jackson string length restriction in Spark/Local backends
Browse files Browse the repository at this point in the history
Resolves #14650
  • Loading branch information
chrisvittal committed Aug 2, 2024
1 parent 84dd26c commit 9e89f35
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 0 deletions.
4 changes: 4 additions & 0 deletions hail/src/main/scala/is/hail/backend/local/LocalBackend.scala
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ import scala.reflect.ClassTag

import java.io.PrintWriter

import com.fasterxml.jackson.core.StreamReadConstraints
import com.google.common.util.concurrent.MoreExecutors
import org.apache.hadoop
import org.json4s._
Expand All @@ -46,6 +47,9 @@ object LocalBackend {
skipLoggingConfiguration: Boolean = false,
): LocalBackend = synchronized {
require(theLocalBackend == null)
StreamReadConstraints.overrideDefaultStreamReadConstraints(
StreamReadConstraints.builder().maxStringLength(Integer.MAX_VALUE).build()
)

if (!skipLoggingConfiguration)
HailContext.configureLogging(logFile, quiet, append)
Expand Down
4 changes: 4 additions & 0 deletions hail/src/main/scala/is/hail/backend/spark/SparkBackend.scala
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,7 @@ import scala.util.control.NonFatal

import java.io.{Closeable, PrintWriter}

import com.fasterxml.jackson.core.StreamReadConstraints
import org.apache.hadoop
import org.apache.hadoop.conf.Configuration
import org.apache.spark._
Expand Down Expand Up @@ -257,6 +258,9 @@ object SparkBackend {
gcsRequesterPaysBuckets: String = null,
): SparkBackend = synchronized {
require(theSparkBackend == null)
StreamReadConstraints.overrideDefaultStreamReadConstraints(
StreamReadConstraints.builder().maxStringLength(Integer.MAX_VALUE).build()
)

if (!skipLoggingConfiguration)
HailContext.configureLogging(logFile, quiet, append)
Expand Down

0 comments on commit 9e89f35

Please sign in to comment.