This isn't introduced in this diff, but is worth fixing: all the `try`/`catch` checks here should have an `Assert.fail` after the line where an exception is expected, e.g.:

    try {
      stringDatum1.asByte();
      Assert.fail("Expected exception")
    } catch (NumberFormatException e) {
      // expected
    }