[GitHub] storm issue #1939: STORM-1363: TridentKafkaState should handle null values f...
Github user pasalkarsachin1 commented on the issue: https://github.com/apache/storm/pull/1939 Thanks for merge --- 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] storm issue #1939: STORM-1363: TridentKafkaState should handle null values f...
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1939 @pasalkarsachin1 Merged into master via 313e960bedde3b2c47520a74c391425d79271727, and 1.x branch via 4fc55b33504d446ef192a25e5164f861e1495291. Could you close this PR? Thanks in advance. --- 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] storm issue #1939: STORM-1363: TridentKafkaState should handle null values f...
Github user pasalkarsachin1 commented on the issue: https://github.com/apache/storm/pull/1939 @HeartSaVioR Can you merge this so I will close this PR :) Thanks in advance --- 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] storm issue #1939: STORM-1363: TridentKafkaState should handle null values f...
Github user pasalkarsachin1 commented on the issue: https://github.com/apache/storm/pull/1939 @HeartSaVioR Squashed into one commit. Commit sha is 7cc1bdbea562a48aa09cc6dc7fd29edd5bd4eb10 --- 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] storm issue #1939: STORM-1363: TridentKafkaState should handle null values f...
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1939 +1 @pasalkarsachin1 Could you squash commits into one? Thanks in advance! --- 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] storm issue #1939: STORM-1363: TridentKafkaState should handle null values f...
Github user pasalkarsachin1 commented on the issue: https://github.com/apache/storm/pull/1939 @HeartSaVioR I have updated code as per your comment :) --- 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] storm issue #1939: STORM-1363: TridentKafkaState should handle null values f...
Github user pasalkarsachin1 commented on the issue: https://github.com/apache/storm/pull/1939 @HeartSaVioR Can you take a look at it --- 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] storm issue #1939: STORM-1363: TridentKafkaState should handle null values f...
Github user pasalkarsachin1 commented on the issue: https://github.com/apache/storm/pull/1939 Can someone take a look at 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] storm issue #1939: STORM-1363: TridentKafkaState should handle null values f...
Github user pasalkarsachin1 commented on the issue: https://github.com/apache/storm/pull/1939 @HeartSaVioR I did my best. Can you take a look at it now --- 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] storm issue #1939: STORM-1363: TridentKafkaState should handle null values f...
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1939 @pasalkarsachin1 Indentation is still off. Could you please check the diff on PR? https://github.com/apache/storm/pull/1939/files --- 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] storm issue #1939: STORM-1363: TridentKafkaState should handle null values f...
Github user pasalkarsachin1 commented on the issue: https://github.com/apache/storm/pull/1939 @HeartSaVioR Done --- 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] storm issue #1939: STORM-1363: TridentKafkaState should handle null values f...
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1939 @pasalkarsachin1 We don't use 8 spaces for indentation from other java files. If the indentation spaces was 4, it should be 4 unless we set brand new style guide. Code style is important as much as code change. --- 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] storm issue #1939: STORM-1363: TridentKafkaState should handle null values f...
Github user pasalkarsachin1 commented on the issue: https://github.com/apache/storm/pull/1939 I haven't changed anything it was I guess previous one had style. You can ignore white space by appending ?w=1 to url. e.g. https://github.com/apache/storm/pull/1939/files?w=1 This shows only updated code --- 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] storm issue #1939: STORM-1363: TridentKafkaState should handle null values f...
Github user HeartSaVioR commented on the issue: https://github.com/apache/storm/pull/1939 @pasalkarsachin1 Could you preserve the style? I'm sorry we don't have clear style guide but most of changeset is from spaces. --- 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. ---
