Github user asfgit closed the pull request at:
https://github.com/apache/storm/pull/2467
---
Github user omkreddy commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r159409982
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/schema/builder/RestJsonSchemaBuilderV2.java
---
@@ -0,0 +1,127 @@
+/**
+ *
Github user omkreddy commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r159409508
--- Diff: external/storm-solr/README.md ---
@@ -171,7 +194,7 @@ Querying Solr for these patterns, you will see the
values that have been indexe
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r159340988
--- Diff: external/storm-solr/README.md ---
@@ -97,6 +97,29 @@ field separates each value with the token % instead of
the default | . To use th
Github user hmcl commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r159120269
--- Diff: external/storm-solr/README.md ---
@@ -97,6 +97,29 @@ field separates each value with the token % instead of
the default | . To use th
Github user hmcl commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r159121022
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/schema/builder/RestJsonSchemaBuilderV2.java
---
@@ -0,0 +1,127 @@
+/**
+ * Licensed
Github user hmcl commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r159120398
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/bolt/SolrUpdateBolt.java
---
@@ -153,4 +160,15 @@ private void failQueuedTuples(List
Github user hmcl commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r159120315
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/bolt/SolrUpdateBolt.java
---
@@ -88,9 +92,11 @@ private int capacity() {
@Override
Github user hmcl commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r159120430
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/schema/SolrFieldTypeFinder.java
---
@@ -70,13 +73,22 @@ public String toString() {
Github user hmcl commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r159120320
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/bolt/SolrUpdateBolt.java
---
@@ -128,6 +134,7 @@ private void fail(Tuple tuple, Exception e)
Github user hmcl commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r159120287
--- Diff: external/storm-solr/README.md ---
@@ -171,7 +194,7 @@ Querying Solr for these patterns, you will see the
values that have been indexe
Github user hmcl commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r159120311
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/bolt/SolrUpdateBolt.java
---
@@ -88,9 +92,11 @@ private int capacity() {
@Override
Github user hmcl commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r159120406
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/config/SolrConfig.java
---
@@ -54,4 +65,7 @@ public int getTickTupleInterval() {
Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r158913106
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/bolt/SolrUpdateBolt.java
---
@@ -74,6 +75,9 @@ public SolrUpdateBolt(SolrConfig
Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r158913266
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/schema/builder/RestJsonSchemaBuilderV2.java
---
@@ -0,0 +1,127 @@
+/**
+ *
Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r158912237
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/mapper/SolrMapper.java
---
@@ -26,6 +26,7 @@
import java.util.List;
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r158899660
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/mapper/SolrMapper.java
---
@@ -26,6 +26,7 @@
import java.util.List;
Github user omkreddy commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r158812503
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/mapper/SolrMapper.java
---
@@ -26,6 +26,7 @@
import java.util.List;
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r158739084
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/mapper/SolrMapper.java
---
@@ -26,6 +26,7 @@
import java.util.List;
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r158658318
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/mapper/SolrMapper.java
---
@@ -26,6 +26,7 @@
import java.util.List;
Github user omkreddy commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r158236702
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/mapper/SolrMapper.java
---
@@ -26,6 +26,7 @@
import java.util.List;
Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r157942171
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/mapper/SolrJsonMapper.java
---
@@ -93,6 +93,10 @@ private SolrJsonMapper(Builder builder)
Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r157944148
--- Diff: external/storm-solr/pom.xml ---
@@ -46,19 +46,24 @@
org.apache.solr
solr-solrj
-
Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r158011321
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/schema/builder/RestJsonSchemaBuilderV2.java
---
@@ -0,0 +1,123 @@
+/**
+ *
Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r157942068
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/mapper/SolrMapper.java
---
@@ -26,6 +26,7 @@
import java.util.List;
Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r157941248
--- Diff: external/storm-solr/README.md ---
@@ -97,6 +97,29 @@ field separates each value with the token % instead of
the default | . To use th
Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r157943487
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/schema/builder/RestJsonSchemaBuilderV2.java
---
@@ -0,0 +1,123 @@
+/**
+ *
Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r157941206
--- Diff: external/storm-solr/README.md ---
@@ -97,6 +97,29 @@ field separates each value with the token % instead of
the default | . To use th
Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r157943956
--- Diff: external/storm-solr/pom.xml ---
@@ -46,19 +46,24 @@
org.apache.solr
solr-solrj
-
Github user satishd commented on a diff in the pull request:
https://github.com/apache/storm/pull/2467#discussion_r157941502
--- Diff:
external/storm-solr/src/main/java/org/apache/storm/solr/bolt/SolrUpdateBolt.java
---
@@ -88,11 +92,14 @@ private int capacity() {
GitHub user omkreddy opened a pull request:
https://github.com/apache/storm/pull/2467
STORM-2860: Add Kerberos support to Solr bolt
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/omkreddy/storm kerberos-solr-bolt
Alternatively
31 matches
Mail list logo
