feat(api): validate and reject external datasets in document update (#21783)
This commit is contained in:
@@ -211,6 +211,9 @@ class DocumentAddByFileApi(DatasetApiResource):
|
|||||||
if not dataset:
|
if not dataset:
|
||||||
raise ValueError("Dataset does not exist.")
|
raise ValueError("Dataset does not exist.")
|
||||||
|
|
||||||
|
if dataset.provider == "external":
|
||||||
|
raise ValueError("External datasets are not supported.")
|
||||||
|
|
||||||
indexing_technique = args.get("indexing_technique") or dataset.indexing_technique
|
indexing_technique = args.get("indexing_technique") or dataset.indexing_technique
|
||||||
if not indexing_technique:
|
if not indexing_technique:
|
||||||
raise ValueError("indexing_technique is required.")
|
raise ValueError("indexing_technique is required.")
|
||||||
@@ -301,6 +304,9 @@ class DocumentUpdateByFileApi(DatasetApiResource):
|
|||||||
if not dataset:
|
if not dataset:
|
||||||
raise ValueError("Dataset does not exist.")
|
raise ValueError("Dataset does not exist.")
|
||||||
|
|
||||||
|
if dataset.provider == "external":
|
||||||
|
raise ValueError("External datasets are not supported.")
|
||||||
|
|
||||||
# indexing_technique is already set in dataset since this is an update
|
# indexing_technique is already set in dataset since this is an update
|
||||||
args["indexing_technique"] = dataset.indexing_technique
|
args["indexing_technique"] = dataset.indexing_technique
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user