Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/380
See
[here](https://github.com/jaredjstewart/geode/blob/GEODE-2409/README.md) to
view the updated readme in your browser.
---
If your project is set up for it, you can reply to this email
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/368
+1, this looks much better!
---
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
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/383#discussion_r99235354
--- Diff:
geode-core/src/test/java/org/apache/geode/internal/InternalDataSerializerQuickcheckStringTest.java
---
@@ -0,0 +1,63
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/383#discussion_r99239963
--- Diff:
geode-core/src/test/java/org/apache/geode/internal/InternalDataSerializerQuickcheckStringTest.java
---
@@ -0,0 +1,63
GitHub user jaredjstewart opened a pull request:
https://github.com/apache/geode/pull/393
GEODE-2430: Remove jar and zip files from test resources
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jaredjstewart/geode GEODE-2430
Github user jaredjstewart closed the pull request at:
https://github.com/apache/geode/pull/380
---
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
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/383#discussion_r99268355
--- Diff:
geode-core/src/test/java/org/apache/geode/internal/InternalDataSerializerQuickcheckStringTest.java
---
@@ -0,0 +1,63
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/393
Precheckin started
---
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
GitHub user jaredjstewart opened a pull request:
https://github.com/apache/geode/pull/395
GEODE-2430: Fix failing tests
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jaredjstewart/geode GEODE-2430
Alternatively you can review
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/395
Precheckin started
---
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
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/374
Precheckin passed
---
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
GitHub user jaredjstewart opened a pull request:
https://github.com/apache/geode/pull/380
GEODE-2409: Beautify readme
- Add Apache Geode banner image
- Add badges for the latest version in Maven Central and for the Apache
2.0 license
- Improve formatting for the table
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/374
Precheckin started
---
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
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/374
`HelpCommandsIntegrationTest` failed in precheckin. I've pushed a change
to the properties file used by that test and started a new precheckin.
---
If your project is set up for it, you can
GitHub user jaredjstewart opened a pull request:
https://github.com/apache/geode/pull/361
GEODE-2325: Improve tests for JarDeployer
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jaredjstewart/geode GEODE-2325
Alternatively
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/361
Precheckin passed (other than an unrelated failure in
`PRClientServerRegionFunctionExecutionFailoverDUnitTest.
testServerFailoverWithTwoServerAliveHA`)
---
If your project is set up
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/403
It seems like it might be simpler to add an explicit dependency for
commons-beanutils to geode-core. That way if we later introduce some new
dependency which requires say commons-beanutils
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/403
Can you explain why the version override strategy is necessary? I think
geode-pulse (or any other submodule with an explicit dependency on
commons-beantuils) should end up with version 1.9.3
GitHub user jaredjstewart opened a pull request:
https://github.com/apache/geode/pull/338
GEODE-2272: Fix Pulse data browser export
* Pulse no longer loads all results into the browser before generating a
results file for download.
You can merge this pull request into a Git
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/339
Precheckin started
---
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
GitHub user jaredjstewart opened a pull request:
https://github.com/apache/geode/pull/339
GEODE-2272: Fix Pulse data browser export
* Pulse no longer loads all results into the browser before generating a
results file for download.
You can merge this pull request into a Git
Github user jaredjstewart closed the pull request at:
https://github.com/apache/geode/pull/338
---
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
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/338
Closing for now due precheckin failures
---
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
GitHub user jaredjstewart opened a pull request:
https://github.com/apache/geode/pull/347
GEODE-2313: Fix PulseDataExportTest
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jaredjstewart/geode GEODE-2313
Alternatively you can
Github user jaredjstewart commented on the issue:
https://github.com/apache/incubator-geode/pull/294
Precheckin started
---
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
GitHub user jaredjstewart opened a pull request:
https://github.com/apache/incubator-geode/pull/287
GEODE-2112: Fix UITests failing due to insufficient cleanup
- Extracted setup of the Server and WebDriver for Pulse tests into JUnit
Rules
- Refactored UITests
- Removed
Github user jaredjstewart commented on the issue:
https://github.com/apache/incubator-geode/pull/287
Precheckin started
---
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
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/311#discussion_r91813763
--- Diff:
geode-core/src/main/java/org/apache/geode/distributed/AbstractLauncher.java ---
@@ -668,13 +747,45 @@ public String getJavaVersion
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/310
Oops! I have amended my commit to include the appropriate header.
---
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
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/317#discussion_r92650337
--- Diff:
geode-core/src/test/java/org/apache/geode/test/dunit/rules/GfshShellConnectionRule.java
---
@@ -85,19 +91,17 @@ public void connect(String
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/317#discussion_r92651688
--- Diff:
geode-core/src/test/java/org/apache/geode/test/dunit/rules/ConfigGroup.java ---
@@ -0,0 +1,102 @@
+/*
+ * Licensed to the Apache
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/317#discussion_r92668393
--- Diff:
geode-core/src/test/java/org/apache/geode/test/dunit/rules/GfshShellConnectionRule.java
---
@@ -85,21 +91,16 @@ public void connect(String
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/317
Precheckin passed
---
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
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/305#discussion_r92525359
--- Diff:
geode-core/src/test/java/org/apache/geode/internal/cache/FixedPRSinglehopDUnitTest.java
---
@@ -293,18 +294,23 @@ public void
GitHub user jaredjstewart opened a pull request:
https://github.com/apache/geode/pull/317
[GEODE-2196] Add test for exportClusterConfig.
- Break ClusterConfigDUnitTest out into multiple test classes.
- Refactor ClusterConfigDUnitTest.ExpectedConfig into a standalone
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/317
precheckin started
---
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
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/307
+1
---
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
GitHub user jaredjstewart opened a pull request:
https://github.com/apache/geode/pull/333
Various pulse fixes
GEODE-1114: RemoveGemfireXD references from Pulse
GEODE-2194: Fix Pulse webapp context â¦
* Pulse no longer has redundant URLs like
http://localhost/pulse/pulse
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/333
Precheckin started
---
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
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/332
+1
---
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
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/333
The pulse dataExportTest failed in precheckin. Closing this PR until I've
made the test more resilient.
---
If your project is set up for it, you can reply to this email and have your
reply
Github user jaredjstewart closed the pull request at:
https://github.com/apache/geode/pull/333
---
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
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/335#discussion_r95899936
--- Diff: geode-pulse/src/main/webapp/WEB-INF/web.xml ---
@@ -32,7 +32,7 @@
mvc-dispatcher
-/pulse/*
--- End diff
GitHub user jaredjstewart opened a pull request:
https://github.com/apache/geode/pull/336
GEODE-2194: Fix pulse webapp context
This commit re-adds a line (which turns out to be necessary) that was
removed during the review of PR #335.
You can merge this pull request into a Git
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/335
Precheckin passed
---
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
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/335#discussion_r95897532
--- Diff: geode-pulse/src/main/webapp/WEB-INF/web.xml ---
@@ -32,7 +32,7 @@
mvc-dispatcher
-/pulse/*
--- End diff
GitHub user jaredjstewart opened a pull request:
https://github.com/apache/geode/pull/331
GEODE-958: AgentUtilJunitTest doesn't require env var.
- AgentUtilJunitTest will now set the gemfire.home system property for
AgentUtil to find the pulse war if no GEMFIRE enironment variable
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/331
@kohlmu-pivotal @hiteshk25 Adding you guys since you originally were active
in GEODE-958
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/301
Precheckin passed
---
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
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/429#discussion_r107221701
--- Diff:
geode-core/src/test/java/org/apache/geode/internal/JarDeployerIntegrationTest.java
---
@@ -126,14 +111,16 @@ public void
Github user jaredjstewart closed the pull request at:
https://github.com/apache/geode/pull/429
---
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
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/429
Closing this PR until GEODE-2705 and a few other concerns are addressed
---
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
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/429
Precheckin is started (still running)
---
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
GitHub user jaredjstewart opened a pull request:
https://github.com/apache/geode/pull/431
GEODE-2692: Fix tests relying on file creation time
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/jaredjstewart/geode GEODE-2692
GitHub user jaredjstewart opened a pull request:
https://github.com/apache/geode/pull/428
GEODE-2649: Export logs does not use file creation time
Precheckin started (still running)
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/417
Weâll see if the nightly build passes in the morning :D
> On Mar 7, 2017, at 4:07 PM, Kenneth Howe <notificati...@github.com> wrote:
>
> @pdxrunner commented on th
GitHub user jaredjstewart opened a pull request:
https://github.com/apache/geode/pull/417
GEODE-2621: Reduce time sensitivity of ExportLogsDUnitTest
Precheckin started (still running)
You can merge this pull request into a Git repository by running:
$ git pull https
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/429#discussion_r107194282
--- Diff:
geode-core/src/test/java/org/apache/geode/internal/JarDeployerIntegrationTest.java
---
@@ -126,14 +111,16 @@ public void
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/429#discussion_r107195082
--- Diff:
geode-core/src/test/java/org/apache/geode/internal/JarDeployerIntegrationTest.java
---
@@ -144,40 +131,99 @@ public void run
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/429#discussion_r107192575
--- Diff:
geode-core/src/test/java/org/apache/geode/internal/JarDeployerIntegrationTest.java
---
@@ -39,83 +44,63 @@
@Rule
public
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/429#discussion_r107195612
--- Diff:
geode-core/src/test/java/org/apache/geode/internal/ClassPathLoaderTest.java ---
@@ -65,6 +65,26 @@ public void testLatestExists() throws
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/679#discussion_r130998820
--- Diff:
geode-web/src/test/java/org/apache/geode/management/internal/cli/commands/ConfigCommandDUnitTest.java
---
@@ -0,0 +1,242
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/664
Thanks @jujoramos, this looks good to me. Do you have any thoughts
@jinmeiliao?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/679#discussion_r130998592
--- Diff:
geode-web/src/test/java/org/apache/geode/management/internal/cli/commands/ConfigCommandDUnitTest.java
---
@@ -0,0 +1,242
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/699#discussion_r132585334
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/process/LocalProcessLauncher.java
---
@@ -96,33 +117,55 @@ void close
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/699#discussion_r132562988
--- Diff:
geode-core/src/main/java/org/apache/geode/distributed/AbstractLauncher.java ---
@@ -795,25 +785,25 @@ protected String toString(final Date
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/699#discussion_r132572256
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/process/AttachProcessUtils.java
---
@@ -14,21 +14,28 @@
*/
package
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/699#discussion_r132585045
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/process/FileProcessController.java
---
@@ -112,56 +119,43 @@ private void stop(final File
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/699#discussion_r132570355
--- Diff:
geode-core/src/main/java/org/apache/geode/distributed/LocatorLauncher.java ---
@@ -104,6 +109,8 @@
helpMap.put("bind-ad
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/699#discussion_r132568184
--- Diff:
geode-core/src/main/java/org/apache/geode/distributed/LocatorLauncher.java ---
@@ -1352,11 +1328,13 @@ protected void parseCommand(final
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/699#discussion_r132571788
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/config/ClusterConfigurationNotAvailableException.java
---
@@ -0,0 +1,29
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/699#discussion_r132569537
--- Diff:
geode-core/src/main/java/org/apache/geode/distributed/LocatorLauncher.java ---
@@ -1781,8 +1782,8 @@ protected void validate
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/708#discussion_r132589415
--- Diff: geode-docs/tools_modules/gfsh/command-pages/query.html.md.erb ---
@@ -21,7 +21,11 @@ limitations under the License.
Run queries
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/708#discussion_r132591921
--- Diff: geode-docs/tools_modules/gfsh/command-pages/query.html.md.erb ---
@@ -21,7 +21,11 @@ limitations under the License.
Run queries
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/651#discussion_r129647633
--- Diff:
geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/DiskStoreCommands.java
---
@@ -639,29 +639,34 @@ public Result
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/668
Hi Darren,
It looks like `xercesImpl` may need to be declared as a `testCompile`
dependency rather than `testRuntime` in case people are building with a JDK
which does not include
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/664
Hi Juan,
Thanks for your contribution! Your changes look good to me, but our
[Criteria for Code
Submissions](https://cwiki.apache.org/confluence/display/GEODE/Criteria+for+Code
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/671#discussion_r130668398
--- Diff:
geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/CreateRegionCommand.java
---
@@ -1143,4 +721,26 @@ private
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/672#discussion_r130673499
--- Diff:
geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/DataCommandsUtils.java
---
@@ -0,0 +1,311
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/665#discussion_r130661310
--- Diff:
geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/ExportConfigCommandDUnitTest.java
---
@@ -0,0 +1,181
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/665#discussion_r130658877
--- Diff:
geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/Interceptor.java
---
@@ -0,0 +1,79 @@
+/*
+ * Licensed
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/651#discussion_r130677047
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/process/MBeanProcessController.java
---
@@ -260,6 +265,7 @@ private JMXServiceURL
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/651#discussion_r130677475
--- Diff:
geode-core/src/main/java/org/apache/geode/management/internal/cli/Launcher.java
---
@@ -99,7 +99,6 @@ protected Launcher
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/651#discussion_r130678413
--- Diff:
geode-core/src/main/java/org/apache/geode/management/internal/cli/functions/DataCommandFunction.java
---
@@ -633,8 +631,8 @@ public
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/668#discussion_r131968121
--- Diff:
geode-core/src/test/java/org/apache/geode/internal/cache/xmlcache/CacheXmlParserJUnitTest.java
---
@@ -111,10 +113,31 @@ public void
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/668#discussion_r131966460
--- Diff:
geode-core/src/test/java/org/apache/geode/internal/cache/xmlcache/CacheXmlParserJUnitTest.java
---
@@ -128,9 +128,16 @@ public void
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/668#discussion_r131967266
--- Diff:
geode-core/src/main/java/org/apache/geode/internal/cache/xmlcache/CacheXmlParser.java
---
@@ -2596,6 +2596,18 @@ private void endDeclarable
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/613
Hi Dinesh,
Sorry to be so slow in responding to your PR. Since the changes you are
proposing in GEODE-3151 involve changing a public-facing API (in
DistributionConfig, I think
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/720
Hi Dinesh,
I just commented on your earlier PR, and the same principle applies to this
one as well: Changes to public APIs should first be discussed on
dev@geode.apache.org
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/724#discussion_r134075704
--- Diff:
geode-core/src/test/java/org/apache/geode/internal/process/lang/AvailablePidTest.java
---
@@ -100,7 +105,53 @@ public void
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/724#discussion_r134075750
--- Diff:
geode-core/src/test/java/org/apache/geode/internal/process/lang/AvailablePid.java
---
@@ -30,37 +30,88 @@
*/
public class
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/596
+1
---
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
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/596#discussion_r123817037
--- Diff:
geode-core/src/main/java/org/apache/geode/management/CacheServerMXBean.java ---
@@ -60,48 +61,48 @@
/**
* Returns the port
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/596#discussion_r123819260
--- Diff:
geode-core/src/main/java/org/apache/geode/management/internal/beans/MemberMBean.java
---
@@ -148,7 +148,12 @@ public long
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/596#discussion_r123822645
--- Diff:
geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/FunctionCommands.java
---
@@ -130,31 +125,8 @@ public Result
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/596#discussion_r123818520
--- Diff:
geode-core/src/test/java/org/apache/geode/management/internal/security/DiskStoreMXBeanSecurityJUnitTest.java
---
@@ -57,7 +56,48 @@ public
Github user jaredjstewart commented on a diff in the pull request:
https://github.com/apache/geode/pull/596#discussion_r123820699
--- Diff:
geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/ClientCommands.java
---
@@ -109,12 +107,10 @@ public Result
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/596
Merged as 451d12e
---
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
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/528
Merged as dff937f
---
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
Github user jaredjstewart commented on the issue:
https://github.com/apache/geode/pull/539
Thanks for your contribution, Emily! It looks like this branch will need
to be rebased onto the latest develop to apply cleanly. Would you mind
updating the PR with a rebased version
1 - 100 of 122 matches
Mail list logo