You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "github-actions[bot] (via GitHub)" <gi...@apache.org> on 2023/04/17 05:00:45 UTC

[GitHub] [doris] github-actions[bot] commented on a diff in pull request #18721: [Improve](GEO)wkb input and output are represented as hexadecimal strings And delete EWKB

github-actions[bot] commented on code in PR #18721:
URL: https://github.com/apache/doris/pull/18721#discussion_r1168166090


##########
be/src/geo/wkb_parse.cpp:
##########
@@ -43,6 +87,36 @@ GeoParseStatus WkbParse::parse_wkb(std::istream& is, bool isEwkb, GeoShape** sha
     return ctx.parse_status;
 }
 
+WkbParseContext* WkbParse::read_hex(std::istream& is, WkbParseContext* ctx) {
+    // setup input/output stream
+    std::stringstream os(std::ios_base::binary | std::ios_base::in | std::ios_base::out);

Review Comment:
   warning: implicit instantiation of undefined template 'std::basic_stringstream<char>' [clang-diagnostic-error]
   ```cpp
       std::stringstream os(std::ios_base::binary | std::ios_base::in | std::ios_base::out);
                         ^
   ```
   **/usr/include/c++/11/iosfwd:107:** template is declared here
   ```cpp
       class basic_stringstream;
             ^
   ```
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org