Navigation Menu

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix MariaDB + Mroonga build failures #144

Closed
kou opened this issue Jun 19, 2017 · 20 comments
Closed

Fix MariaDB + Mroonga build failures #144

kou opened this issue Jun 19, 2017 · 20 comments

Comments

@kou
Copy link
Member

kou commented Jun 19, 2017

http://buildbot.askmonty.org/buildbot/grid?category=main&branch=ks-10.1-mroonga
@kou
Copy link
Member Author

kou commented Jul 13, 2017

これはどう見ればいいんだ。。。
http://buildbot.askmonty.org/buildbot/builders/kvm-bintar-centos5-amd64 とかそれぞれのビルドを見て、最新のビルドがsuccessかfailureかを見ればいいのかしら。

@kou
Copy link
Member Author

kou commented Jul 13, 2017 •
edited by yoku0825

最新ビルドがfailureのもの：

@yoku0825
Copy link
Contributor

これ、mtrが転けてる場合は http://buildbot.askmonty.org/buildbot/builders/kvm-deb-jessie-x86/builds/2575/steps/mtr/logs/mysqld.1.err.4 とかエラーログをつぶさに確認していくしかない…ですよね……？

@kou
Copy link
Member Author

kou commented Jul 27, 2017

いきなりエラーログを見る必要はないです！
stdio http://buildbot.askmonty.org/buildbot/builders/kvm-deb-jessie-x86/builds/2575/steps/mtr/logs/stdio の方の最後に

Failing test(s): funcs_1.is_columns_is funcs_1.is_routines

みたいなのがあるので、その中にMroonga関連の失敗がなければMroongaは関係ないと判断します。

とりあえず、一通り、Mroongaが関係あるやつを絞り込んでからエラーログだったり他のログだったりを確認するのがいいかなぁと思っています。

@yoku0825
Copy link
Contributor

mtrだと

test nm F:delete_use_sou...

のように如何にも「省略しています！ 全部列挙されているかどうかの保証もありません！」って感じの書き方なので、若干不安になりますが了解です。

http://buildbot.askmonty.org/buildbot/builders/kvm-deb-stretch-amd64/builds/1035/steps/mtr

@yoku0825
Copy link
Contributor

上から何個か見てみました。

http://buildbot.askmonty.org/buildbot/builders/kvm-deb-jessie-ppc64le

https://gist.github.com/yoku0825/68dcf3639c07d1ec312b54520d3a448d

( ´-｀).oO（このIssueのリストをチェックボックスにしてもらえると嬉しいです…

@kou
Copy link
Member Author

kou commented Jul 27, 2017

しました！

@yoku0825
Copy link
Contributor

あっこれOwnerじゃないとポチれないのか…！！！ _|￣|○

@kou
Copy link
Member Author

kou commented Jul 27, 2017

Mroonga developersに加えました！ようこそ！

@kou
Copy link
Member Author

kou commented Jul 27, 2017

ポチれるかしら。。。

@yoku0825
Copy link
Contributor

ポチれました！ ありがとうございます！

…(ﾟ∀。) ｱﾚ？

@yoku0825
Copy link
Contributor

yoku0825 commented Jul 27, 2017 •
edited

@yoku0825
Copy link
Contributor

@yoku0825
Copy link
Contributor

yoku0825 commented Jul 27, 2017 •
edited

@yoku0825
Copy link
Contributor

@yoku0825
Copy link
Contributor

@kou 一通り見ました！

コンパイラワーニング2件、mtr1件です。

コンパイラーがワーニング吐いてる

mtrの mroonga/storage.index_multiple_column_unique_datetime_index_read で転けてる

ここかな？

CURRENT_TEST: mroonga/storage.index_multiple_column_unique_datetime_index_read
==12065== Thread 6:
==12065== Conditional jump or move depends on uninitialised value(s)
==12065==    at 0x8354B6: TIME_to_longlong_datetime_packed(st_mysql_time const*) (compat56.cc:242)
==12065==    by 0xBF0A922: mrn::MultipleColumnKeyCodec::decode(unsigned char const*, unsigned int, unsigned char*, unsigned int*) (mrn_multiple_column_key_codec.cpp:294)
==12065==    by 0xBEDC324: ha_mroonga::storage_store_fields_by_index(unsigned char*) (ha_mroonga.cpp:11100)
==12065==    by 0xBED1F89: ha_mroonga::storage_get_next_record(unsigned char*) (ha_mroonga.cpp:9488)
==12065==    by 0xBEC5161: ha_mroonga::storage_index_first(unsigned char*) (ha_mroonga.cpp:7385)
==12065==    by 0xBEC531A: ha_mroonga::index_first(unsigned char*) (ha_mroonga.cpp:7397)
==12065==    by 0x870D7A: handler::ha_index_first(unsigned char*) (handler.cc:2699)
==12065==    by 0x68A573: join_read_first(st_join_table*) (sql_select.cc:19287)
==12065==    by 0x688234: sub_select(JOIN*, st_join_table*, bool) (sql_select.cc:18330)
==12065==    by 0x687AF7: do_select(JOIN*, List<Item>*, TABLE*, Procedure*) (sql_select.cc:17988)
==12065==    by 0x6627B8: JOIN::exec_inner() (sql_select.cc:3221)
==12065==    by 0x65F972: JOIN::exec() (sql_select.cc:2511)
==12065== 
{
   <insert_a_suppression_name_here>
   Memcheck:Cond
   fun:_Z32TIME_to_longlong_datetime_packedPK13st_mysql_time
   fun:_ZN3mrn22MultipleColumnKeyCodec6decodeEPKhjPhPj
   fun:_ZN10ha_mroonga29storage_store_fields_by_indexEPh
   fun:_ZN10ha_mroonga23storage_get_next_recordEPh
   fun:_ZN10ha_mroonga19storage_index_firstEPh
   fun:_ZN10ha_mroonga11index_firstEPh
   fun:_ZN7handler14ha_index_firstEPh
   fun:_ZL15join_read_firstP13st_join_table
   fun:_Z10sub_selectP4JOINP13st_join_tableb
   fun:_ZL9do_selectP4JOINP4ListI4ItemEP5TABLEP9Procedure
   fun:_ZN4JOIN10exec_innerEv
   fun:_ZN4JOIN4execEv
}

@kou
Copy link
Member Author

kou commented Jul 27, 2017

ありがとうございます！

@kou
Copy link
Member Author

kou commented Jul 27, 2017

コンパイラーのウォーニングの前者は最適化できないというやつなので気にしなくてよい。

後者は10.2.4のtar.gzをHA_EXTRA_STARTING_ORDERED_INDEX_SCANでgrepしても見つからないのでたぶん一時的にあっただけだろう。

@kou
Copy link
Member Author

kou commented Jul 27, 2017 •
edited

テストで失敗しているのは 60324a0 で2年前に対応済み。
マージされていないのかしら。

@kou
Copy link
Member Author

kou commented Jul 27, 2017

ということで大丈夫そう！

@kou kou closed this as completed Oct 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kou @yoku0825