[GitHub] geode pull request #720: Feature/geode 3462
GitHub user dineshakhand opened a pull request: https://github.com/apache/geode/pull/720 Feature/geode 3462 Thank you for submitting a contribution to Apache Geode. In order to streamline the review of the contribution we ask you to ensure the following steps have been taken: ### For all changes: - [X ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message? - [ X] Has your PR been rebased against the latest commit within the target branch (typically `develop`)? - [X ] Is your initial contribution a single, squashed commit? - [ X] Does `gradlew build` run cleanly? - [ ] Have you written or updated unit tests to verify your changes? - [ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under [ASF 2.0](http://www.apache.org/legal/resolved.html#category-a)? ### Note: Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible. If you need help, please send an email to dev@geode.apache.org. You can merge this pull request into a Git repository by running: $ git pull https://github.com/dineshakhand/geode feature/GEODE-3462 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/geode/pull/720.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #720 commit 34f2a388f4643f8a5dd5374a2f9f2b8178367c13 Author: dineshpune2006 Date: 2017-08-18T09:24:03Z GEODE-3462 Function Stats for JMX Async Qeueu Internal Region are exposed to JMX commit f321c4b4e0d490f93a0a2611a6f219db4a8c1d97 Author: dineshpune2006 Date: 2017-08-18T09:24:49Z GEODE-3462 Function Stats exposed to JMX Async Queue Internal Region are exposed to JMX --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] geode pull request #720: GEODE-3462: FunctionStats Exposed over JMX
Github user dineshakhand commented on a diff in the pull request:
https://github.com/apache/geode/pull/720#discussion_r134197931
--- Diff:
geode-core/src/main/java/org/apache/geode/management/FunctionStatsMXBean.java
---
@@ -0,0 +1,54 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF) under one or more
contributor license
+ * agreements. See the NOTICE file distributed with this work for
additional information regarding
+ * copyright ownership. The ASF licenses this file to You under the Apache
License, Version 2.0 (the
+ * "License"); you may not use this file except in compliance with the
License. You may obtain a
+ * copy of the License at
+ *
+ * http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
distributed under the License
+ * is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF
ANY KIND, either express
+ * or implied. See the License for the specific language governing
permissions and limitations under
+ * the License.
+ */
+package org.apache.geode.management;
+
+import org.apache.geode.cache.asyncqueue.AsyncEventQueue;
+import org.apache.geode.management.internal.security.ResourceOperation;
+import org.apache.geode.security.ResourcePermission.Operation;
+import org.apache.geode.security.ResourcePermission.Resource;
+
+/**
+ * MBean that provides access to an {@link AsyncEventQueue}.
+ *
+ * @since GemFire 7.0
+ *
+ */
+@ResourceOperation(resource = Resource.CLUSTER, operation = Operation.READ)
+public interface FunctionStatsMXBean {
--- End diff --
ok. i will start the discussion. thanks for guiding us , as we are new with
this.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
[GitHub] geode pull request #720: GEODE-3462: FunctionStats Exposed over JMX
Github user dineshakhand closed the pull request at: https://github.com/apache/geode/pull/720 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] geode issue #720: GEODE-3462: FunctionStats Exposed over JMX
Github user dineshakhand commented on the issue: https://github.com/apache/geode/pull/720 Closing the pull rquest after your comments. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
[GitHub] geode pull request #752: GEODE-3544
GitHub user dineshakhand opened a pull request: https://github.com/apache/geode/pull/752 GEODE-3544 JSON parsing failed for short data type & base class setting is not used when key having base class Thank you for submitting a contribution to Apache Geode. In order to streamline the review of the contribution we ask you to ensure the following steps have been taken: ### For all changes: - [ X] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message? - [ X] Has your PR been rebased against the latest commit within the target branch (typically `develop`)? - [ X] Is your initial contribution a single, squashed commit? - [ X] Does `gradlew build` run cleanly? - [ X] Have you written or updated unit tests to verify your changes? - [ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under [ASF 2.0](http://www.apache.org/legal/resolved.html#category-a)? ### Note: Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible. If you need help, please send an email to dev@geode.apache.org. You can merge this pull request into a Git repository by running: $ git pull https://github.com/dineshakhand/geode feature/GEODE-3544 Alternatively you can review and apply these changes as the patch at: https://github.com/apache/geode/pull/752.patch To close this pull request, make a commit to your master/trunk branch with (at least) the following in the commit message: This closes #752 commit 2874f5f3f0429a9e40f4baa41727d14f74034844 Author: dineshpune2006 Date: 2017-08-31T13:18:36Z GEODE-3544 JSON parsing failed for short data type & base class setting is not used when key having base class --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastruct...@apache.org or file a JIRA ticket with INFRA. ---
