From f7c0e425a933cf868094856dc315bc0001e32959 Mon Sep 17 00:00:00 2001 From: Vadim Ogievetsky Date: Wed, 13 Mar 2024 11:38:51 -0700 Subject: [PATCH] fix URI wording (#16111) --- .../lookup-edit-dialog.spec.tsx.snap | 47 +++++++++---------- .../druid-models/lookup-spec/lookup-spec.tsx | 13 ++--- 2 files changed, 26 insertions(+), 34 deletions(-) diff --git a/web-console/src/dialogs/lookup-edit-dialog/__snapshots__/lookup-edit-dialog.spec.tsx.snap b/web-console/src/dialogs/lookup-edit-dialog/__snapshots__/lookup-edit-dialog.spec.tsx.snap index 213e043bc93..b0edee5ef4c 100644 --- a/web-console/src/dialogs/lookup-edit-dialog/__snapshots__/lookup-edit-dialog.spec.tsx.snap +++ b/web-console/src/dialogs/lookup-edit-dialog/__snapshots__/lookup-edit-dialog.spec.tsx.snap @@ -127,33 +127,28 @@ exports[`LookupEditDialog matches snapshot 1`] = ` }, { "defined": [Function], - "info": -

- URI for the file of interest, specified as a - - file - - , - - hdfs - - , - - - s3 - - , or - - gs - - path -

-

- The URI prefix option is strictly better than URI and should be used instead -

-
, + "info":

+ URI for the file of interest, specified as a + + file + + , + + hdfs + + , + + + s3 + + , or + + gs + + path +

, "issueWithValue": [Function], - "label": "URI (deprecated)", + "label": "URI", "name": "extractionNamespace.uri", "placeholder": "s3://bucket/some/key/prefix/lookups-01.gz", "required": [Function], diff --git a/web-console/src/druid-models/lookup-spec/lookup-spec.tsx b/web-console/src/druid-models/lookup-spec/lookup-spec.tsx index d187c839bc9..b0e0d69fff9 100644 --- a/web-console/src/druid-models/lookup-spec/lookup-spec.tsx +++ b/web-console/src/druid-models/lookup-spec/lookup-spec.tsx @@ -169,7 +169,7 @@ export const LOOKUP_FIELDS: Field[] = [ { name: 'extractionNamespace.uri', type: 'string', - label: 'URI (deprecated)', + label: 'URI', placeholder: 's3://bucket/some/key/prefix/lookups-01.gz', defined: l => oneOfKnown(deepGet(l, 'extractionNamespace.type'), KNOWN_EXTRACTION_NAMESPACE_TYPES, 'uri') && @@ -178,13 +178,10 @@ export const LOOKUP_FIELDS: Field[] = [ !deepGet(l, 'extractionNamespace.uriPrefix') && !deepGet(l, 'extractionNamespace.uri'), issueWithValue: issueWithUri, info: ( - <> -

- URI for the file of interest, specified as a file, hdfs,{' '} - s3, or gs path -

-

The URI prefix option is strictly better than URI and should be used instead

- +

+ URI for the file of interest, specified as a file, hdfs,{' '} + s3, or gs path +

), }, {