Build failed in Jenkins: Geode-nightly-flaky #105

See Changes: [upthewaterspout] GEODE-3513: Removing the use of native sessions session caching [klund] GEODE-3527: increase timeouts for file deletion [klund] GEODE-3527: use consistent value for all

Build failed in Jenkins: Geode-nightly #937

See -- [...truncated 79.27 KB...] at org.apache.geode.test.dunit.rules.gfsh.GfshScript.awaitLoudly(GfshScript.java:133) at

[GitHub] geode-examples pull request #14: GEODE-3440: Add example for function execut...

GitHub user PivotalSarge opened a pull request: https://github.com/apache/geode-examples/pull/14 GEODE-3440: Add example for function execution. You can merge this pull request into a Git repository by running: $ git pull https://github.com/PivotalSarge/geode-examples

[GitHub] geode pull request #747: GEODE-3514: Clean up locator and protobuf related c...

GitHub user pivotal-amurmann opened a pull request: https://github.com/apache/geode/pull/747 GEODE-3514: Clean up locator and protobuf related code * Check for feature flag only once * Simplify TcpServerFactory * Indicate RoundTripLocatorConnection test properly as DUnit

[GitHub] geode pull request #745: GEODE-3436: Restore reverted GFSH command refactori...

Github user PurelyApplied closed the pull request at: https://github.com/apache/geode/pull/745 --- 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] geode pull request #745: GEODE-3436: Restore reverted GFSH command refactori...

GitHub user PurelyApplied reopened a pull request: https://github.com/apache/geode/pull/745 GEODE-3436: Restore reverted GFSH command refactoring Given that this would restore multiple tickets worth of commits, it was unclear if this PR should squash all commits or if the history

[GitHub] geode pull request #730: GEODE-3472: Remove a great deal of commented-out co...

Github user PurelyApplied closed the pull request at: https://github.com/apache/geode/pull/730 --- 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] geode pull request #730: GEODE-3472: Remove a great deal of commented-out co...

GitHub user PurelyApplied reopened a pull request: https://github.com/apache/geode/pull/730 GEODE-3472: Remove a great deal of commented-out code. * Correct imports in touched files * Correct several misspellings * Removed redundant modifier and specifiers * Corrected

[GitHub] geode pull request #730: GEODE-3472: Remove a great deal of commented-out co...

Github user PurelyApplied closed the pull request at: https://github.com/apache/geode/pull/730 --- 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] geode pull request #730: GEODE-3472: Remove a great deal of commented-out co...

GitHub user PurelyApplied reopened a pull request: https://github.com/apache/geode/pull/730 GEODE-3472: Remove a great deal of commented-out code. * Correct imports in touched files * Correct several misspellings * Removed redundant modifier and specifiers * Corrected

[GitHub] geode issue #730: GEODE-3472: Remove a great deal of commented-out code.

Github user PurelyApplied commented on the issue: https://github.com/apache/geode/pull/730 Precheckin returned green. Open/Close PR to trigger Travis again. Hopefully they've pulled `develop` recently... --- If your project is set up for it, you can reply to this email and

[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #661 was SUCCESSFUL (with 2027 tests)

--- Spring Data GemFire > Nightly-ApacheGeode > #661 was successful. --- Scheduled 2029 tests in total. https://build.spring.io/browse/SGF-NAG-661/ -- This

[GitHub] geode pull request #721: GEODE-3448: Implement and expose parallel snapshot ...

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/721 --- 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

[GitHub] geode issue #730: GEODE-3472: Remove a great deal of commented-out code.

Github user PurelyApplied commented on the issue: https://github.com/apache/geode/pull/730 Travis failures appear related to `f41ca9d7d2fa7c045ec439df9478335233f1d95e`. Is there a way to force Travis to rerun without applying a new commit? --- If your project is set up for it, you

[GitHub] geode issue #746: GEODE-3529 move new client/server security classes to a di...

Github user bschuchardt commented on the issue: https://github.com/apache/geode/pull/746 Travis check failed in an unrelated unit test. This test passes on my branch when I run it individually or as part of the "build" task. --- If your project is set up for it, you can reply to

[GitHub] geode pull request #746: GEODE-3529 move new client/server security classes ...

GitHub user bschuchardt opened a pull request: https://github.com/apache/geode/pull/746 GEODE-3529 move new client/server security classes to a different pac… @kohlmu-pivotal @hiteshk25 @galen-pivotal @pivotal-amurmann @metatype The new classes/interfaces have been moved

[GitHub] geode pull request #745: GEODE-3436: Restore reverted GFSH command refactori...

Github user PurelyApplied commented on a diff in the pull request: https://github.com/apache/geode/pull/745#discussion_r135641458 --- Diff: geode-core/src/test/java/org/apache/geode/management/internal/security/TestCommand.java --- @@ -171,28 +171,37 @@ private static void init()

[GitHub] geode pull request #745: GEODE-3436: Restore reverted GFSH command refactori...

Github user pdxrunner commented on a diff in the pull request: https://github.com/apache/geode/pull/745#discussion_r135639047 --- Diff: geode-core/src/test/java/org/apache/geode/management/internal/security/TestCommand.java --- @@ -171,28 +171,37 @@ private static void init() {

[GitHub] geode pull request #745: GEODE-3436: Restore reverted GFSH command refactori...

Github user jinmeiliao commented on a diff in the pull request: https://github.com/apache/geode/pull/745#discussion_r135635795 --- Diff: geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/ShowMetricsCommand.java --- @@ -794,7 +84,7 @@ public Result

[GitHub] geode pull request #745: GEODE-3436: Restore reverted GFSH command refactori...

Github user PurelyApplied commented on a diff in the pull request: https://github.com/apache/geode/pull/745#discussion_r135635781 --- Diff: geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/ShowMetricsCommand.java --- @@ -794,7 +84,7 @@ public Result

[GitHub] geode pull request #745: GEODE-3436: Restore reverted GFSH command refactori...

Github user pdxrunner commented on a diff in the pull request: https://github.com/apache/geode/pull/745#discussion_r135633281 --- Diff: geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/ExecuteFunctionCommand.java --- @@ -156,7 +132,7 @@ public Result

[GitHub] geode pull request #745: GEODE-3436: Restore reverted GFSH command refactori...

Github user PurelyApplied commented on a diff in the pull request: https://github.com/apache/geode/pull/745#discussion_r135632737 --- Diff: geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/CreateRegionCommand.java --- @@ -247,7 +207,7 @@ public Result

[GitHub] geode pull request #745: GEODE-3436: Restore reverted GFSH command refactori...

Github user pdxrunner commented on a diff in the pull request: https://github.com/apache/geode/pull/745#discussion_r135629055 --- Diff: geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/CreateRegionCommand.java --- @@ -247,7 +207,7 @@ public Result

[GitHub] geode pull request #730: GEODE-3472: Remove a great deal of commented-out co...

Github user PurelyApplied commented on a diff in the pull request: https://github.com/apache/geode/pull/730#discussion_r135626288 --- Diff: geode-core/src/main/java/org/apache/geode/management/internal/cli/CliUtil.java --- @@ -212,8 +211,7 @@ public static void

[GitHub] geode pull request #730: GEODE-3472: Remove a great deal of commented-out co...

Github user PurelyApplied commented on a diff in the pull request: https://github.com/apache/geode/pull/730#discussion_r135625955 --- Diff: geode-core/src/main/java/org/apache/geode/management/internal/cli/shell/Gfsh.java --- @@ -158,7 +156,7 @@ private Thread runner;

[GitHub] geode pull request #730: GEODE-3472: Remove a great deal of commented-out co...

Github user PurelyApplied commented on a diff in the pull request: https://github.com/apache/geode/pull/730#discussion_r135625773 --- Diff: geode-core/src/main/java/org/apache/geode/management/internal/cli/domain/DataCommandResult.java --- @@ -415,9 +413,8 @@ public Result

[GitHub] geode pull request #734: GEODE-3508: Remove unused internal deprecated class...

Github user PurelyApplied closed the pull request at: https://github.com/apache/geode/pull/734 --- 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] geode issue #739: GEODE-3385: Change GetAllRequest to return list of errors.

Github user galen-pivotal commented on the issue: https://github.com/apache/geode/pull/739 Rebased on develop. --- 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] geode issue #734: GEODE-3508: Remove unused internal deprecated classes.

Github user kirklund commented on the issue: https://github.com/apache/geode/pull/734 I'll pull this in. --- 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] geode pull request #734: GEODE-3508: Remove unused internal deprecated class...

Github user kirklund commented on a diff in the pull request: https://github.com/apache/geode/pull/734#discussion_r135605423 --- Diff: geode-core/src/main/java/org/apache/geode/internal/process/ClusterConfigurationNotAvailableException.java --- @@ -1,33 +0,0 @@ -/* - *

[GitHub] geode pull request #745: GEODE-3436: Restore reverted GFSH command refactori...

Github user kirklund commented on a diff in the pull request: https://github.com/apache/geode/pull/745#discussion_r135605262 --- Diff: geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/CreateDefinedIndexesCommand.java --- @@ -0,0 +1,155 @@ +/* + *

[GitHub] geode pull request #745: GEODE-3436: Restore reverted GFSH command refactori...

Github user PurelyApplied commented on a diff in the pull request: https://github.com/apache/geode/pull/745#discussion_r135599224 --- Diff: geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/CreateDefinedIndexesCommand.java --- @@ -0,0 +1,155 @@ +/*

[GitHub] geode pull request #745: GEODE-3436: Restore reverted GFSH command refactori...

Github user pdxrunner commented on a diff in the pull request: https://github.com/apache/geode/pull/745#discussion_r135597926 --- Diff: geode-core/src/main/java/org/apache/geode/management/internal/cli/commands/CreateDefinedIndexesCommand.java --- @@ -0,0 +1,155 @@ +/* +

[GitHub] geode issue #719: GEODE-3447 Implement client authorization for the new prot...

Github user bschuchardt commented on the issue: https://github.com/apache/geode/pull/719 @metatype After reviewing StreamAuthenticator I get your point. I think these new interfaces and classes need to be in a different package. --- If your project is set up for it, you can reply

[GitHub] geode pull request #745: GEODE-3436: Restore reverted GFSH command refactori...

GitHub user PurelyApplied opened a pull request: https://github.com/apache/geode/pull/745 GEODE-3436: Restore reverted GFSH command refactoring Given that this would restore multiple tickets worth of commits, it was unclear if this PR should squash all commits or if the history

Re: Query mechanism

DefaultQuery is where the processing starts for a query. CompiledSelect will most likely be the first node in processing the query. The IndexManager class will contain the list of indexes for a region as well as the methods that help find indexes to use with a query. Specific index classes:

[GitHub] geode pull request #740: GEODE-3513: Removing the use of native sessions ses...

Github user asfgit closed the pull request at: https://github.com/apache/geode/pull/740 --- 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

[GitHub] geode pull request #719: GEODE-3447 Implement client authorization for the n...

Github user bschuchardt commented on a diff in the pull request: https://github.com/apache/geode/pull/719#discussion_r135575597 --- Diff: geode-core/src/main/java/org/apache/geode/security/StreamAuthenticator.java --- @@ -46,6 +44,14 @@ void receiveMessage(InputStream inputStream,

Re: Review Request 61895: GEDOE-3516: TXManagerImpl.tryResume call may add itself again into the waiting thread queue

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61895/#review183945 ---

[GitHub] geode pull request #719: GEODE-3447 Implement client authorization for the n...

Github user metatype commented on a diff in the pull request: https://github.com/apache/geode/pull/719#discussion_r135573704 --- Diff: geode-core/src/main/java/org/apache/geode/security/StreamAuthenticator.java --- @@ -46,6 +44,14 @@ void receiveMessage(InputStream inputStream,

[GitHub] geode pull request #719: GEODE-3447 Implement client authorization for the n...

GitHub user bschuchardt reopened a pull request: https://github.com/apache/geode/pull/719 GEODE-3447 Implement client authorization for the new protocol Implementation of authorization checks for the new protocol. This will have to be merged with the Locator Protobuf

[GitHub] geode pull request #719: GEODE-3447 Implement client authorization for the n...

Github user bschuchardt closed the pull request at: https://github.com/apache/geode/pull/719 --- 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

Re: Review Request 61950: GEODE-3519 servers are not locking on remove or invalidate ops initiated by clients

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61950/#review183950 --- Ship it! Ship It! - Hitesh Khamesra On Aug. 28, 2017, 4:09

Re: [GitHub] geode issue #719: GEODE-3447 Implement client authorization for the new prot...

A hash is not guaranteed unique so is not suitable as a security token. -- Mike Stolz Principal Engineer, GemFire Product Manager Mobile: +1-631-835-4771 On Fri, Aug 25, 2017 at 4:49 PM, galen-pivotal wrote: > Github user galen-pivotal commented on the issue: > >

Re: [Discuss] Enable region set operations to start JTA transactions

+1 for revising set ops to bootstrap by default. Document the behavior change in release notes along with property to revert to old behavior. -- Mike Stolz Principal Engineer, GemFire Product Manager Mobile: +1-631-835-4771 On Fri, Aug 25, 2017 at 6:41 PM, Anilkumar Gingade

Re: Indxes and hints

Hi Roi, Answers are below the questions... Question 1. Is it true to say, that the query as it is will load all the data values from the file, since the field C is part of the value, which is already persisted to file? Depending on if an index is used or not, if an index is used, the values

Review Request 61950: GEODE-3519 servers are not locking on remove or invalidate ops initiated by clients

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61950/ --- Review request for geode, Alexander Murmann, Galen O'Sullivan, Hitesh Khamesra,

[GitHub] geode pull request #719: GEODE-3447 Implement client authorization for the n...

Github user pivotal-amurmann commented on a diff in the pull request: https://github.com/apache/geode/pull/719#discussion_r135550580 --- Diff: geode-core/src/main/java/org/apache/geode/security/StreamAuthenticator.java --- @@ -46,6 +44,14 @@ void receiveMessage(InputStream

Query mechanism

Hi, I am looking into the internals of how the query process works, and how indexes/hints affect it, Can anyone point me to the most important classes, the main mechanism "wheels" etc.? Thanks, Roi -Original Message- From: Roi Apelker Sent: Sunday, August 27, 2017 12:55 PM To:

Bug while parsing the JSON "key which having short data type" in locate command

Hi Team, I have found one bug in geode 1.2 . If in the key we having the short data type Example: public class EmpData implements Serializable{ private short empid; public short getEmpid() { return empid; } public void setEmpid(short empid) { this.empid = empid; } EmpData d1

DISCUSS : Monitor the neighbour JVM using neihbour's member-timeout (GEODE-3411)

Hi Team, We have a requirement to configure different member timeout for different members as we need some members to survive in the view for longer time than the other the members before being kicked out of the view in case they aren't responding. 1. Now with the current monitoring

[GitHub] geode issue #717: Feature/geode 3411 Monitor the neighbour JVM using neihbou...

Github user aravindmusigumpula commented on the issue: https://github.com/apache/geode/pull/717 thanks @upthewaterspout for pointing out these issues , I have added this new parameter in two methods toDataPre_GFE_7_1_0_0, toDataPre_GFE_9_0_0_0. And I understand that adding a