第9回Go勉強会の課題@luccafort #50
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Streamなレスポンスを受け取ったときにどうRubyで扱うのかわからなかったのでチュートリアルで実装を参考にした https://github.com/grpc/grpc.io/blob/master/content/docs/tutorials/basic/ruby.md
luccafort
commented
Apr 19, 2020
| http_response = http_client.say_hello(request_opts) | ||
| puts http_response | ||
|
|
||
| puts '=====================' |
There was a problem hiding this comment.
ファイルを分割しても良かったんですが元々Go側のコードも分割はしてなかったので1つのファイルで実行しています。
There was a problem hiding this comment.
この標準出力は通常のレスポンスとStreamなレスポンスをわかりやすくする境界線として出力しているだけで特に意味はないです。
luccafort
commented
Apr 19, 2020
|
|
||
| request_opts = Sample::HelloRequest.new(first_name: '太郎', last_name: '兼進') | ||
| stream_request = stream_client.pereodic_hello(request_opts) | ||
| stream_request.each { |req| puts req } |
There was a problem hiding this comment.
今回、検証はしてないけどこの block 内で実行されているコード内で処理を抜けたい場合は条件式などで抜け出す条件を記載して return することになるのかな？ 🤔
block 内で break すると意図しない挙動になるってチェリー本に書いてたのは記憶してるが詳細は忘れた。
luccafort
commented
Apr 19, 2020
| # Generated by the protocol buffer compiler. DO NOT EDIT! | ||
| # Source: proto/helloworld.proto for package 'sample' | ||
|
|
||
| require 'grpc' | ||
| require 'proto/helloworld_pb' | ||
| require './proto/helloworld_pb' |
luccafort
commented
Apr 19, 2020
| # Generated by the protocol buffer compiler. DO NOT EDIT! | ||
| # Source: proto/notification.proto for package 'sample' | ||
|
|
||
| require 'grpc' | ||
| require 'proto/notification_pb' | ||
| require './proto/notification_pb' |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
課題
https://mf.esa.io/posts/131091#%E8%AA%B2%E9%A1%8C
ハマりどころ
make setupでgem install grpc-toolsがSystemのほうのGemを参照していた少し困った。解決策は書いたが環境系の問題に引っかかると面倒。
https://moneyforward.slack.com/archives/CVB3XK04U/p1587301291079800
対応としてあっているのかわからないが自動生成されたコードの
require 'proto/xxxx'というファイルのパスが正しくないためにclient.rbがLoad Errorになり続けて原因究明するのに時間がかかった。最終的に
requireしていないgrpc関係ないコードを書く → OKrequireを復活させてgrpc関係ないコードを書く → NGrequireさせてgrpc関係ないコードを書く → OKだったので
requireが正常に機能していないことを確認してパスを書き換えたらうまくいった……が自動生成されたものの読み込みに問題があるのを疑うのはなかなか厳しい。