DLog Namespace#openLog should declare LogNotFoundException
Other methods such as delete do, so openLog should declare it to be
consistent and to give users a hint that it can be caught.
Reviewers: Enrico Olivelli <eolivelli@gmail.com>
This closes #2072 from ivankelly/dl-openlog-ex
diff --git a/stream/distributedlog/core/src/main/java/org/apache/distributedlog/api/namespace/Namespace.java b/stream/distributedlog/core/src/main/java/org/apache/distributedlog/api/namespace/Namespace.java
index 712295d..1b2a787 100644
--- a/stream/distributedlog/core/src/main/java/org/apache/distributedlog/api/namespace/Namespace.java
+++ b/stream/distributedlog/core/src/main/java/org/apache/distributedlog/api/namespace/Namespace.java
@@ -128,10 +128,11 @@
* name of the log
* @return distributedlog manager instance.
* @throws InvalidStreamNameException if log name is invalid.
+ * @throws LogNotFoundException if log doesn't exist.
* @throws IOException when encountered issues with backend.
*/
DistributedLogManager openLog(String logName)
- throws InvalidStreamNameException, IOException;
+ throws InvalidStreamNameException, LogNotFoundException, IOException;
/**
* Open a log named <i>logName</i> with specific log configurations.
@@ -149,13 +150,14 @@
* dynamic log configuration
* @return distributedlog manager instance.
* @throws InvalidStreamNameException if log name is invalid.
+ * @throws LogNotFoundException if log doesn't exist.
* @throws IOException when encountered issues with backend.
*/
DistributedLogManager openLog(String logName,
Optional<DistributedLogConfiguration> logConf,
Optional<DynamicDistributedLogConfiguration> dynamicLogConf,
Optional<StatsLogger> perStreamStatsLogger)
- throws InvalidStreamNameException, IOException;
+ throws InvalidStreamNameException, LogNotFoundException, IOException;
/**
* Check whether the log <i>logName</i> exist.