Github user snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/334#issuecomment-127700582
Shouldn't this be done for the master branch, in case we need to release
1.0.3, etc.?
---
If your project is set up for it, you can reply to this email
Github user snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/334#issuecomment-127706740
I think we should merge it to master then from there to 2.1.
---
If your project is set up for it, you can reply to this email and have your
reply appear
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/332
ExportAppTest fixes
Tools should depend on Services not REST, JsonUtils should chill out on the
warnings and ExportAppTest now needs at least 5 writeThreads to keep up with
Usergrid
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/310
Additional fixes to ExportAdmins ImportAdmins
ImportAdmins will now merge duplicate user accounts and give the merged
account a union of the orgs in the duplicates. ExportAdmins
Github user snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/307#issuecomment-121694547
@tnine Please take another look. I made flatmap max observables match write
thread count and use Schedulers.io() instead of a custom readScheduler. But I
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/307
New ExportApp tool, using RxJava
https://issues.apache.org/jira/browse/USERGRID-788
You can merge this pull request into a Git repository by running:
$ git pull https
Github user snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/302#issuecomment-119925798
This is a huge change. It might be a good idea to do a screen-sharing
session with a couple of UG devs and walk through the design and code.
---
If your
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/301
Couple small import/export fixes
Couple more fixes from testing:
1. only export credentials dictionary because it is all we need for admins,
2. use em.get() instead
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/298
Import fixes
More fixes to import/export:
1) ignore empty dictionaries on export
2) counter logic to ensure that import does not exit until all users
imported
3) changes
Github user snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/286#issuecomment-117135031
I did a merge of two-dot-dev into the USERGRID-703 branch and all the REST
tests passed for me. Perhaps you've got a problem in your environment
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/292
ImportAdmins and ExportAdmins fixes
A bunch of changes and improvements made while testing export and import of
Admin Users.
1) Removed code that handled connections
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/287
Fix test problems and re-enable AssetResourceIT tests
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/apache/incubator-usergrid
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/286
USERGRID-703: fixes for SSO-related ManagementResourceIT tests
Tests relied on Codehaus Jackson and tests cannot be run concurrently.
You can merge this pull request into a Git
Github user snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/277#discussion_r32363214
--- Diff:
stack/tools/src/main/java/org/apache/usergrid/tools/ExportAdmins.java ---
@@ -60,213 +82,393 @@ public void runTool( CommandLine line
Github user snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/275#issuecomment-111515415
Apart from the minor things above, this PR looks good. I am merging it now.
---
If your project is set up for it, you can reply to this email and have
Github user snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/275#discussion_r32321377
--- Diff:
stack/tools/src/main/java/org/apache/usergrid/tools/ImportAdmins.java ---
@@ -133,84 +163,129 @@ private void importAdminUsers
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/277
Improve Export Admins performance.
DO NOT MERGE - still testing
Use multiple threads to read data from Cassandra and one thread to write
that data to the two output files
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/268
Small fixes to Admin import/export tool based on results of testing
- ExportAdmins now does not halt when getOrganizationsForAdminUser() has
uniqueness violation
- ImportAdmins
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/261
Add organization export to ExportAdmins tool and organization import â¦
â¦to ImportAdmins tool.
This PR adds organization export and import to the existing ExportAdmins
Github user snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/250#discussion_r30591226
--- Diff:
stack/services/src/main/java/org/apache/usergrid/services/assets/data/S3BinaryStore.java
---
@@ -149,38 +150,52 @@ public void write
Github user snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/250#discussion_r30591214
--- Diff:
stack/rest/src/main/java/org/apache/usergrid/rest/management/ManagementResource.java
---
@@ -701,6 +710,50 @@ private JsonNode
Github user snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/250#discussion_r30605417
--- Diff:
stack/services/src/main/java/org/apache/usergrid/services/assets/data/S3BinaryStore.java
---
@@ -134,10 +134,11 @@ public void write
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/250
Upgrade JClouds, add connection pool for Central SSO
This PR upgrades from to JClouds 1.7.1 and because JClouds no longer
supports async calls to AWS, I had to change code
Github user snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/250#discussion_r30548737
--- Diff:
stack/services/src/main/java/org/apache/usergrid/services/assets/data/S3BinaryStore.java
---
@@ -134,10 +134,11 @@ public void write
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/244
Remove Jackson 2 dependency
Remove Jackson 2 dependency that snuck in with new AWS S3 store because it
breaks the rest tests.
You can merge this pull request into a Git repository
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/245
HTTP client connection pool for SSO token validation calls
Adds connection pool for HTTP client connections from SSO client system to
SSO central.
You can merge this pull request
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/242
central SSO fixes
A couple of small fixes:
1) user's created in the SSO client system are activated
2) fix and simplify only superuser can login to SSO client system logic
You
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/241
Allow /management/me calls when central SSO is enabled.
Portal needs /management/me to work when central SSO is enabled.
You can merge this pull request into a Git repository
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/238
Add metrics reporting to Usergrid SSO endpoint /externaltokens
Adds metrics for:
- Count number of validated Admin Tokens
- Count number of rejected Admin Tokens
- Count
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/234
SSO / external tokens Admin User create fix
Fix to ensure that local Admin Users created by SSO / external tokens get
at least one organization.
Should have been part
Github user snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/170#issuecomment-94434546
This has been merged.
---
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
Github user snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/228#issuecomment-94051396
Also need to be able to configure the Locks keyspace used in
HectorLockManagerImpl.
Currently it is hardcoded into our Spring config.
---
If your
Github user snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/112#issuecomment-94070043
I'm going to close this. It is obsolete because of the many changes in
two-dot-o-dev.
Also, conventional wisdom seems to be that having source
Github user snoopdave closed the pull request at:
https://github.com/apache/incubator-usergrid/pull/112
---
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 snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/230
Ensure that when external token validation is enabled, Admin Users cannot
sign-up
Or be activated or be confirmed... all that should happen via Usergrid
Central.
You can merge
Github user snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/224#discussion_r28521479
--- Diff:
stack/rest/src/test/java/org/apache/usergrid/rest/management/ManagementResourceIT.java
---
@@ -640,4 +644,139 @@ public void
Github user snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/224#discussion_r28521613
--- Diff:
stack/rest/src/main/java/org/apache/usergrid/rest/management/ManagementResource.java
---
@@ -441,6 +464,229 @@ public Viewable
Github user snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/224#discussion_r28520905
--- Diff:
stack/rest/src/main/java/org/apache/usergrid/rest/security/shiro/filters/OAuth2AccessTokenSecurityFilter.java
---
@@ -104,7 +104,12
Github user snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/225#discussion_r28530186
--- Diff: stack/config/src/main/resources/usergrid-default.properties ---
@@ -62,6 +62,15 @@ cassandra.mutation.flushsize=2000
#Note
Github user snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/224#discussion_r28511357
--- Diff:
stack/rest/src/test/java/org/apache/usergrid/rest/management/ManagementResourceIT.java
---
@@ -640,4 +644,63 @@ public void
Github user snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/225#issuecomment-93823653
what version of Hector adds the new cassandraThriftSocketTimeout parameters?
---
If your project is set up for it, you can reply to this email and have
Github user snoopdave closed the pull request at:
https://github.com/apache/incubator-usergrid/pull/221
---
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 snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/224#discussion_r28463783
--- Diff:
stack/rest/src/main/java/org/apache/usergrid/rest/management/ManagementResource.java
---
@@ -171,6 +199,17 @@ public Response
Github user snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/221#discussion_r28346513
--- Diff:
stack/rest/src/main/java/org/apache/usergrid/rest/management/ManagementResource.java
---
@@ -441,6 +451,181 @@ public Viewable
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/221
Validate External Token (UG Central SSO)
FOR REVIEW ONLY -- NOT READY FOR MERGE
New endpoint that allows Usergrid to use access tokens issued by a central
Usergrid SSO
Github user snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/221#discussion_r28328923
--- Diff:
stack/rest/src/test/java/org/apache/usergrid/rest/test/PropertiesResourceIT.java
---
@@ -36,9 +36,6 @@
public class
Github user snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/221#discussion_r28328971
--- Diff:
stack/services/src/main/java/org/apache/usergrid/security/shiro/Realm.java ---
@@ -235,8 +235,16 @@ protected AuthorizationInfo
Github user snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/221#discussion_r28329002
--- Diff:
stack/rest/src/test/java/org/apache/usergrid/rest/test/PropertiesResourceIT.java
---
@@ -51,25 +48,31 @@ public void
Github user snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/221#discussion_r28356858
--- Diff:
stack/rest/src/main/java/org/apache/usergrid/rest/management/ManagementResource.java
---
@@ -441,6 +451,181 @@ public Viewable
Github user snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/221#issuecomment-93042705
We will also need a configuration variable that will turn-off new Admin
User registrations and logins on UG Dev systems when the Usergrid Central SSO
Github user snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/221#discussion_r28356699
--- Diff:
stack/rest/src/main/java/org/apache/usergrid/rest/management/ManagementResource.java
---
@@ -441,6 +451,181 @@ public Viewable
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/213
Lookup applications using unique value instead of using query
This is a fix for Create app has stopped returning new apps
https://issues.apache.org/jira/browse/USERGRID-491
Github user snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/213#issuecomment-90557687
The rest test is currently not working due to application_info UUID changes
in two-dot-o-dev that impact REST layer authorization logic.
---
If your
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/208
Fix cursor problem that was causing multiple tests to fail.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/snoopdave/incubator
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/209
Fix for https://issues.apache.org/jira/browse/USERGRID-530
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/snoopdave/incubator
Github user snoopdave closed the pull request at:
https://github.com/apache/incubator-usergrid/pull/173
---
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 snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/173#issuecomment-87776891
This has been merged it two-dot-o-dev
---
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 snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/194
EsIndexBufferConsumerImpl: Log error and continue instead of throwing
RuntimeError.
Log error and continue instead of throwing RuntimeError.
You can merge this pull request
Github user snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/173#issuecomment-77878271
Review notes
===
This is a big commit, so here are the key changes to consider. Many files
changed because I moved ApplicationInfo from
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/173
Fix for USERGRID-448: Remove redundant appinfos collections in
ManagementServiceImpl
DO NOT MERGE
Needs careful review.
Changes to merge appinfo and application_info
Github user snoopdave closed the pull request at:
https://github.com/apache/incubator-usergrid/pull/168
---
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 snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/170
Usergrid 361 appdeletetest
DO NOT MERGE
not ready for review
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/snoopdave
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/167
Fix for key names in Credentials.java
Fixes many tests in ApplicationResourceIT.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/166
Exclude Codehaus Jackson JAX-RS dependency, plus...
Exclude Codehaus Jackson JAX-RS dependency, plus some minor logging and
test improvements.
You can merge this pull request
Github user snoopdave commented on commit
bfdacb34ebdf918fcf92307c14808cadd5bdd7fd:
https://github.com/apache/incubator-usergrid/commit/bfdacb34ebdf918fcf92307c14808cadd5bdd7fd#commitcomment-9480825
In
sdks/dotnet/samples/notifications/Usergrid.Notifications/MainPage.xaml.cs
Github user snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/155#issuecomment-71677367
Gotcha. Looks good.
---
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
Github user snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/155#issuecomment-71676204
Do the generated lexer files really belong in source code control? I'd
rather see the lexer classes generated into a temporary directory like
target
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/152
USERGRID-312: Implement App Delete
DO NOT MERGE.
This work is ready for review.
Implements application delete as described here:
https://issues.apache.org/jira
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/143
Remove CDDL/GPL licensed file needed for CSS handling in Grizzly.
For USERGRID-314
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/144
Point Launcher to a copy of Portal deployed to our project website.
Now we point the launcher to
http://usergrid.incubator.apache.org/v101-portal-demo/ for the Portal
Related
Github user snoopdave closed the pull request at:
https://github.com/apache/incubator-usergrid/pull/120
---
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 snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/112#discussion_r21322936
--- Diff:
stack/corepersistence/queryindex/src/main/java/org/apache/usergrid/persistence/index/impl/IndexingUtils.java
---
@@ -151,69 +151,99
Github user snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/113#issuecomment-65717017
Thanks Ryan, looks good so I pulled it in to two-dot-o.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user snoopdave commented on a diff in the pull request:
https://github.com/apache/incubator-usergrid/pull/112#discussion_r21209642
--- Diff:
stack/corepersistence/queryindex/src/main/java/org/apache/usergrid/persistence/index/impl/IndexingUtils.java
---
@@ -151,69 +151,99
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/112
No source in es
DO NOT MERGE. I'm creating this for review purposes.
Changes to prevent us from storing source document or source fields in
ElasticSearch
See also: https
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/82
Fix USERGRID-225 by allowing ${blah} expressions in paths.
This new regex allows paths to be specified like so:
/users/${user}/**
See also: https
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/67
Entity serialization fixes
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/snoopdave/incubator-usergrid two-dot-o
Alternatively
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/68
Fixes for new tests brought in for master.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/snoopdave/incubator-usergrid two-dot-o
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/65
Merge from master
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/snoopdave/incubator-usergrid two-dot-o
Alternatively you can
Github user snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/47#issuecomment-53298568
Thanks!
---
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 snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/42#issuecomment-52926952
Thanks! This looks like a great new feature and I'm sure the community can
work to address those problems you mentioned.
I think we should merge
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/57
Fixes to ES refresh logic better thread and timeout settings for running
JUnit tests.
You can merge this pull request into a Git repository by running:
$ git pull https
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/48
Fixes to get Core, Services and REST tests working consistently with Core
Persistence.
You can merge this pull request into a Git repository by running:
$ git pull https
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/49
AWSCluster and test fixes.
This PR contains two fixes:
1) AWSCluster now sets Tomcat threads and memory based on AWS instance type
2) Cassandra and Hystrix connection
GitHub user snoopdave opened a pull request:
https://github.com/apache/incubator-usergrid/pull/34
Fixes to stack/awscluster that allow successful launch of two-dot-o
cluster.
Series of fixes to Usergrid property handling and awscluster scripts that
enable successful creation
Github user snoopdave commented on the pull request:
https://github.com/apache/incubator-usergrid/pull/27#issuecomment-49427782
I see code here from two authors that I know to be Pooja Ramesh Jain and
Harish Rajagopal, both of whom have ICLA agreements on file with ASF. I'll
review
86 matches
Mail list logo