@@ -59,14 +59,14 @@ export class OutputFormat {
    *
    * @see https://svn.apache.org/repos/infra/websites/production/hive/content/javadocs/r3.1.3/api/org/apache/hadoop/hive/ql/io/avro/AvroContainerOutputFormat.html
    */
-  public static readonly AVRO = new InputFormat('org.apache.hadoop.hive.ql.io.avro.AvroContainerOutputFormat');
+  public static readonly AVRO = new OutputFormat('org.apache.hadoop.hive.ql.io.avro.AvroContainerOutputFormat');
 
   /**
    * OutputFormat for Orc files.
    *
    * @see https://svn.apache.org/repos/infra/websites/production/hive/content/javadocs/r3.1.3/api/org/apache/hadoop/hive/ql/io/orc/OrcOutputFormat.html
    */
-  public static readonly ORC = new InputFormat('org.apache.hadoop.hive.ql.io.orc.OrcOutputFormat');
+  public static readonly ORC = new OutputFormat('org.apache.hadoop.hive.ql.io.orc.OrcOutputFormat');
 
   /**
    * OutputFormat for Parquet files.
 
@@ -100,7 +100,7 @@ export class Database extends Resource implements IDatabase {
   /**
    * Location URI of this database.
    */
-  public locationUri?: string;
+  public readonly locationUri?: string;
 
   private readonly _catalog?: ICatalog;
 
 
@@ -185,7 +185,7 @@ const writePermissions = [
   'glue:UpdatePartition',
 ];
 
-function renderColumns(columns?: Array<Column | Column>) {
+function renderColumns(columns?: Column[]) {
   if (columns === undefined) {
     return undefined;
   }
 
@@ -345,7 +345,7 @@ function createBucket(table: S3Table, props: S3TableProps) {
   };
 }
 
-function renderColumns(columns?: Array<Column | Column>) {
+function renderColumns(columns?: Column[]) {
   if (columns === undefined) {
     return undefined;
   }
 
@@ -0,0 +1,20 @@
+import { InputFormat, OutputFormat } from '../lib';
+
+// Regression test: OutputFormat.AVRO and OutputFormat.ORC were previously
+// constructed as `InputFormat` instances. That compiled in TypeScript because
+// the two classes are structurally identical, but in jsii's nominally-typed
+// languages (Java, C#, Go, Python) it made these members unusable wherever an
+// `OutputFormat` is required. At runtime the classes have distinct prototypes,
+// so `instanceof` distinguishes the bug (would fail below) from the fix.
+describe('OutputFormat static members are OutputFormat instances', () => {
+  test.each([
+    ['AVRO', OutputFormat.AVRO, 'org.apache.hadoop.hive.ql.io.avro.AvroContainerOutputFormat'],
+    ['ORC', OutputFormat.ORC, 'org.apache.hadoop.hive.ql.io.orc.OrcOutputFormat'],
+    ['HIVE_IGNORE_KEY_TEXT', OutputFormat.HIVE_IGNORE_KEY_TEXT, 'org.apache.hadoop.hive.ql.io.HiveIgnoreKeyTextOutputFormat'],
+    ['PARQUET', OutputFormat.PARQUET, 'org.apache.hadoop.hive.ql.io.parquet.MapredParquetOutputFormat'],
+  ])('OutputFormat.%s is an OutputFormat with an unchanged class name', (_name, format, className) => {
+    expect(format).toBeInstanceOf(OutputFormat);
+    expect(format).not.toBeInstanceOf(InputFormat);
+    expect(format.className).toEqual(className);
+  });
+});
Original file line number	Diff line number	Diff line change
`@@ -185,7 +185,7 @@ const writePermissions = [`
`185`	`185`	`'glue:UpdatePartition',`
`186`	`186`	`];`
`187`	`187`
`188`		`-function renderColumns(columns?: Array<Column \| Column>) {`
	`188`	`+function renderColumns(columns?: Column[]) {`
`189`	`189`	`if (columns === undefined) {`
`190`	`190`	`return undefined;`
`191`	`191`	`}`
Original file line number	Diff line number	Diff line change
`@@ -345,7 +345,7 @@ function createBucket(table: S3Table, props: S3TableProps) {`
`345`	`345`	`};`
`346`	`346`	`}`
`347`	`347`
`348`		`-function renderColumns(columns?: Array<Column \| Column>) {`
	`348`	`+function renderColumns(columns?: Column[]) {`
`349`	`349`	`if (columns === undefined) {`
`350`	`350`	`return undefined;`
`351`	`351`	`}`