You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by hu...@apache.org on 2014/02/05 17:46:45 UTC
[1/2] git commit: updated refs/heads/master to adf6026
Updated Branches:
refs/heads/master 6523c0686 -> adf6026d2
SnapshotDetailsVO doesn't have a display field even though it is required in the searchbuilder in ResourceDetailsDaoBase
java.lang.AssertionError: Searching for a field that's not there: display
at com.cloud.utils.db.SearchBase.set(SearchBase.java:219)
at com.cloud.utils.db.SearchBase$Interceptor.intercept(SearchBase.java:475)
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/adf6026d
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/adf6026d
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/adf6026d
Branch: refs/heads/master
Commit: adf6026d2117f42ea625a1d3748493fededcff40
Parents: 8a9d979
Author: Hugo Trippaers <ht...@schubergphilis.com>
Authored: Wed Feb 5 17:44:39 2014 +0100
Committer: Hugo Trippaers <ht...@schubergphilis.com>
Committed: Wed Feb 5 17:46:33 2014 +0100
----------------------------------------------------------------------
.../cloudstack/resourcedetail/ResourceDetailsDaoBase.java | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/adf6026d/engine/schema/src/org/apache/cloudstack/resourcedetail/ResourceDetailsDaoBase.java
----------------------------------------------------------------------
diff --git a/engine/schema/src/org/apache/cloudstack/resourcedetail/ResourceDetailsDaoBase.java b/engine/schema/src/org/apache/cloudstack/resourcedetail/ResourceDetailsDaoBase.java
index 2d08644..60d7f16 100644
--- a/engine/schema/src/org/apache/cloudstack/resourcedetail/ResourceDetailsDaoBase.java
+++ b/engine/schema/src/org/apache/cloudstack/resourcedetail/ResourceDetailsDaoBase.java
@@ -34,7 +34,10 @@ public abstract class ResourceDetailsDaoBase<R extends ResourceDetail> extends G
AllFieldsSearch = createSearchBuilder();
AllFieldsSearch.and("resourceId", AllFieldsSearch.entity().getResourceId(), SearchCriteria.Op.EQ);
AllFieldsSearch.and("name", AllFieldsSearch.entity().getName(), SearchCriteria.Op.EQ);
- AllFieldsSearch.and("display", AllFieldsSearch.entity().isDisplay(), SearchCriteria.Op.EQ);
+ // FIXME SnapshotDetailsVO doesn't have a display field
+ if (_allAttributes.containsKey("display")) {
+ AllFieldsSearch.and("display", AllFieldsSearch.entity().isDisplay(), SearchCriteria.Op.EQ);
+ }
AllFieldsSearch.done();
}
[2/2] git commit: updated refs/heads/master to adf6026
Posted by hu...@apache.org.
Fix camelcasing inconsistency causing an assertion error.
java.lang.AssertionError: Searching for a field that's not there: serviceName
at com.cloud.utils.db.SearchBase.set(SearchBase.java:219)
at com.cloud.utils.db.SearchBase$Interceptor.intercept(SearchBase.java:475)
at com.cloud.network.dao.MonitoringServiceVO$$EnhancerByCGLIB$$84195222.getServiceName(<generated>)
at com.cloud.network.dao.MonitoringServiceDaoImpl.<init>(MonitoringServiceDaoImpl.java:41)
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/8a9d9798
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/8a9d9798
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/8a9d9798
Branch: refs/heads/master
Commit: 8a9d9798d98e5bb1a212f1686ad8f335744d4b20
Parents: 6523c06
Author: Hugo Trippaers <ht...@schubergphilis.com>
Authored: Wed Feb 5 14:55:02 2014 +0100
Committer: Hugo Trippaers <ht...@schubergphilis.com>
Committed: Wed Feb 5 17:46:33 2014 +0100
----------------------------------------------------------------------
.../com/cloud/network/MonitoringService.java | 2 +-
.../network/dao/MonitoringServiceDaoImpl.java | 4 +--
.../cloud/network/dao/MonitoringServiceVO.java | 28 +++++++++-----------
3 files changed, 16 insertions(+), 18 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/8a9d9798/api/src/com/cloud/network/MonitoringService.java
----------------------------------------------------------------------
diff --git a/api/src/com/cloud/network/MonitoringService.java b/api/src/com/cloud/network/MonitoringService.java
index 63f2ec1..b33e8ff 100644
--- a/api/src/com/cloud/network/MonitoringService.java
+++ b/api/src/com/cloud/network/MonitoringService.java
@@ -39,7 +39,7 @@ public interface MonitoringService extends ControlledEntity, Identity, InternalI
String getServiceName();
- String getPidFile();
+ String getServicePidFile();
String getServicePath();
}
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/8a9d9798/engine/schema/src/com/cloud/network/dao/MonitoringServiceDaoImpl.java
----------------------------------------------------------------------
diff --git a/engine/schema/src/com/cloud/network/dao/MonitoringServiceDaoImpl.java b/engine/schema/src/com/cloud/network/dao/MonitoringServiceDaoImpl.java
index c2292a2..fffb42b 100644
--- a/engine/schema/src/com/cloud/network/dao/MonitoringServiceDaoImpl.java
+++ b/engine/schema/src/com/cloud/network/dao/MonitoringServiceDaoImpl.java
@@ -37,10 +37,10 @@ public class MonitoringServiceDaoImpl extends GenericDaoBase<MonitoringServiceVO
AllFieldsSearch = createSearchBuilder();
AllFieldsSearch.and("isDefault", AllFieldsSearch.entity().isDefaultService(), SearchCriteria.Op.EQ);
AllFieldsSearch.and("service", AllFieldsSearch.entity().getService(), SearchCriteria.Op.EQ);
- AllFieldsSearch.and("processname", AllFieldsSearch.entity().getProcessname(), SearchCriteria.Op.EQ);
+ AllFieldsSearch.and("processname", AllFieldsSearch.entity().getProcessName(), SearchCriteria.Op.EQ);
AllFieldsSearch.and("servicename", AllFieldsSearch.entity().getServiceName(), SearchCriteria.Op.EQ);
AllFieldsSearch.and("servicepath", AllFieldsSearch.entity().getServicePath(), SearchCriteria.Op.EQ);
- AllFieldsSearch.and("servicePidFile", AllFieldsSearch.entity().getPidFile(), SearchCriteria.Op.EQ);
+ AllFieldsSearch.and("servicePidFile", AllFieldsSearch.entity().getServicePidFile(), SearchCriteria.Op.EQ);
AllFieldsSearch.done();
}
http://git-wip-us.apache.org/repos/asf/cloudstack/blob/8a9d9798/engine/schema/src/com/cloud/network/dao/MonitoringServiceVO.java
----------------------------------------------------------------------
diff --git a/engine/schema/src/com/cloud/network/dao/MonitoringServiceVO.java b/engine/schema/src/com/cloud/network/dao/MonitoringServiceVO.java
index 1d7ef14..f9967ec 100644
--- a/engine/schema/src/com/cloud/network/dao/MonitoringServiceVO.java
+++ b/engine/schema/src/com/cloud/network/dao/MonitoringServiceVO.java
@@ -31,14 +31,13 @@ import com.cloud.network.MonitoringService;
@Table(name = "monitoring_services")
public class MonitoringServiceVO implements MonitoringService {
- public MonitoringServiceVO(String service, String processname, String serviceName, String servicePath, String pidFile, boolean defaultService) {
+ public MonitoringServiceVO(String service, String processName, String serviceName, String servicePath, String servicePidFile, boolean defaultService) {
this.service = service;
- this.processname = processname;
- this.servicename = serviceName;
+ this.processName = processName;
+ this.serviceName = serviceName;
this.servicePath = servicePath;
- this.servicePidFile = pidFile;
+ this.servicePidFile = servicePidFile;
this.defaultService = defaultService;
-
}
protected MonitoringServiceVO() {
@@ -53,10 +52,10 @@ public class MonitoringServiceVO implements MonitoringService {
String service;
@Column(name = "process_name", updatable = false)
- String processname;
+ String processName;
@Column(name = "service_name", updatable = false)
- String servicename;
+ String serviceName;
@Column(name = "service_path", updatable = false)
private String servicePath;
@@ -77,23 +76,22 @@ public class MonitoringServiceVO implements MonitoringService {
@Override
public String getService() {
- return this.service;
+ return service;
}
@Override
public String getServiceName() {
- return this.servicename; //To change body of implemented methods use File | Settings | File Templates.
+ return serviceName; //To change body of implemented methods use File | Settings | File Templates.
}
@Override
- public String getPidFile() {
- return this.servicePidFile;
-
+ public String getServicePidFile() {
+ return servicePidFile;
}
@Override
public String getServicePath() {
- return this.servicePidFile;
+ return servicePidFile;
}
@Override
@@ -115,7 +113,7 @@ public class MonitoringServiceVO implements MonitoringService {
return defaultService;
}
- public String getProcessname() {
- return processname;
+ public String getProcessName() {
+ return processName;
}
}