diff options
| author | jtc <jtc@ae88bc3d-4319-0410-8dbf-d08b4c9d3795> | 2005-06-09 14:30:37 +0000 |
|---|---|---|
| committer | jtc <jtc@ae88bc3d-4319-0410-8dbf-d08b4c9d3795> | 2005-06-09 14:30:37 +0000 |
| commit | 1af5bce84c70fc730d364611945f928ddf03ddde (patch) | |
| tree | d0e36d570f88bf847347e7ceaebafa3d223a1dc0 /TAO/orbsvcs | |
| parent | be46e6d5b8d7bd5a03b1040788db2bc90029e76f (diff) | |
| download | ATCD-1af5bce84c70fc730d364611945f928ddf03ddde.tar.gz | |
ChangeLogTag: Thu Jun 9 07:24:15 2005 J.T. Conklin <jtc@acorntoolworks.com>
Diffstat (limited to 'TAO/orbsvcs')
| -rw-r--r-- | TAO/orbsvcs/orbsvcs/Log/EventLogConsumer.cpp | 50 | ||||
| -rw-r--r-- | TAO/orbsvcs/orbsvcs/Log/NotifyLogConsumer.cpp | 48 |
2 files changed, 92 insertions, 6 deletions
diff --git a/TAO/orbsvcs/orbsvcs/Log/EventLogConsumer.cpp b/TAO/orbsvcs/orbsvcs/Log/EventLogConsumer.cpp index 04187810399..1b21728ad7f 100644 --- a/TAO/orbsvcs/orbsvcs/Log/EventLogConsumer.cpp +++ b/TAO/orbsvcs/orbsvcs/Log/EventLogConsumer.cpp @@ -37,10 +37,52 @@ ACE_THROW_SPEC (( recList [0].info = data; - // log the RecordList. - this->log_->write_recordlist (recList ACE_ENV_ARG_PARAMETER); - - ACE_CHECK; + // @@ The current revision of the specification (formal/03-07-01) + // states: + // + // * When a push operation is invoked and a log is full, then a + // NO_RESOURCE (sic) SystemException is raised with a LOGFULL + // minor code. + // + // * When a push operation is invoked on a log that is off-duty, + // then a NO_RESOURCE (sic) SystemException is raised with a + // LOGOFFDUTY minor code. + // + // * When a push operation is invoked on a log that is locked, then + // a NO_PERMISSIONS (sic) SystemException is raised with a LOGLOCKED + // minor code. + // + // * When a push operation is invoked on a log that is disabled, + // then a TRANSIENT SystemException is raised with a LOGDISABLED + // minor code. + // + // But neither the Telecom Logging or the CORBA specification define + // the values for these minor codes. + // + // I have submitted a defect report to the OMG for clarification. + // --jtc + ACE_TRY + { + // log the RecordList. + this->log_->write_recordlist (recList ACE_ENV_ARG_PARAMETER); + } + ACE_CATCH (DsLogAdmin::LogFull, ex) + { + ACE_THROW (CORBA::NO_RESOURCES ()); + } + ACE_CATCH (DsLogAdmin::LogOffDuty, ex) + { + ACE_THROW (CORBA::NO_RESOURCES ()); + } + ACE_CATCH (DsLogAdmin::LogLocked, ex) + { + ACE_THROW (CORBA::NO_PERMISSION ()); + } + ACE_CATCH (DsLogAdmin::LogDisabled, ex) + { + ACE_THROW (CORBA::TRANSIENT ()); + } + ACE_ENDTRY; } void diff --git a/TAO/orbsvcs/orbsvcs/Log/NotifyLogConsumer.cpp b/TAO/orbsvcs/orbsvcs/Log/NotifyLogConsumer.cpp index f15f0ba2e96..be15230abb1 100644 --- a/TAO/orbsvcs/orbsvcs/Log/NotifyLogConsumer.cpp +++ b/TAO/orbsvcs/orbsvcs/Log/NotifyLogConsumer.cpp @@ -81,8 +81,52 @@ TAO_Notify_LogConsumer::push recList [0].info = event; - this->log_->write_recordlist (recList ACE_ENV_ARG_PARAMETER); - ACE_CHECK; + // @@ The current revision of the specification (formal/03-07-01) + // states: + // + // * When a push operation is invoked and a log is full, then a + // NO_RESOURCE (sic) SystemException is raised with a LOGFULL + // minor code. + // + // * When a push operation is invoked on a log that is off-duty, + // then a NO_RESOURCE (sic) SystemException is raised with a + // LOGOFFDUTY minor code. + // + // * When a push operation is invoked on a log that is locked, then + // a NO_PERMISSIONS (sic) SystemException is raised with a LOGLOCKED + // minor code. + // + // * When a push operation is invoked on a log that is disabled, + // then a TRANSIENT SystemException is raised with a LOGDISABLED + // minor code. + // + // But neither the Telecom Logging or the CORBA specification define + // the values for these minor codes. + // + // I have submitted a defect report to the OMG for clarification. + // --jtc + ACE_TRY + { + // log the RecordList. + this->log_->write_recordlist (recList ACE_ENV_ARG_PARAMETER); + } + ACE_CATCH (DsLogAdmin::LogFull, ex) + { + ACE_THROW (CORBA::NO_RESOURCES ()); + } + ACE_CATCH (DsLogAdmin::LogOffDuty, ex) + { + ACE_THROW (CORBA::NO_RESOURCES ()); + } + ACE_CATCH (DsLogAdmin::LogLocked, ex) + { + ACE_THROW (CORBA::NO_PERMISSION ()); + } + ACE_CATCH (DsLogAdmin::LogDisabled, ex) + { + ACE_THROW (CORBA::TRANSIENT ()); + } + ACE_ENDTRY; } void |